Телеграмм чат группы symfony

Size: a a a

symfony

2021 April 11

зачем править контроллер? Я предлагаю вместо ($user->isCanView == true) сделать проверку ($user->postPublished >= $min_pposts_published) и всё.

источник

00:23пожаловаться #1

Вадим in symfony

Но это детали реализации. Я к тому, что сама логика выполнилась в момент записи, а на чтении вы только читаете флаг и тупо передаёте его в шаблон, и этот же флаг вы проверяете в методе публикации комента.

источник

00:23пожаловаться #2

✨

✨Basic_Instinct✨ in symfony

сомнительно, если условие n вдруг поменяется

источник

00:24пожаловаться #3

GamIet [UA, Odessa] in symfony

Ну т.е. ДВА модуля. правильно?

источник

00:24пожаловаться #4

✨

✨Basic_Instinct✨ in symfony

ну допустим так

источник

00:24пожаловаться #5

GamIet [UA, Odessa] in symfony

ну норм. но это уже детали.. вопрос бы не об этом

источник

00:24пожаловаться #6

Шурик in symfony

ты же сам говорил, что требования по доступам могут меняться часто. и если в сотне мест ты сделал проверку на $user->postPublished >= $min_pposts_published, а потом выяснится, что кроме количества постов нужно проверять еще и их рейтинг - что делать?

источник

00:25пожаловаться #7

GamIet [UA, Odessa] in symfony

у модели users сделать метод isCanViewPost() и внутри реализовать всю логику и менять в одном месте.

источник

00:27пожаловаться #8

GamIet [UA, Odessa] in symfony

но в базе хранить не флаг, а число постов)

источник

00:27пожаловаться #9

Вадим in symfony

Все опять же зависит от того как это важно для бизнеса, если это константа в коде которая редко меняется, то поменяли н сделали миграцию которая пересчитает все и проставит флаги по новой, если будет требование что это должно изменяется с админки, тогда возможно будет другое решение

источник

00:27пожаловаться #10

Шурик in symfony

а их рейтинг где хранить?

источник

00:27пожаловаться #11

GamIet [UA, Odessa] in symfony

там же в базе в Users. если это единый рейтинг пользователя

источник

00:27пожаловаться #12

Шурик in symfony

и ты же сам сказал обратное. вот твои слова
Я предлагаю вместо ($user->isCanView == true) сделать проверку ($user->postPublished >= $min_pposts_published) и всё.

источник

00:28пожаловаться #13

✨

✨Basic_Instinct✨ in symfony

уходим от сути

источник

00:28пожаловаться #14

GamIet [UA, Odessa] in symfony

или сделать UsersDetails для всяких экстра-счетчиков

источник

00:28пожаловаться #15

✨

✨Basic_Instinct✨ in symfony

бизнес зло?

источник

00:28пожаловаться #16

Вадим in symfony

Детали -- это что именно хранить, количество постов или булеан. Основной посыл был в том, что логика не отрабатывает на каждое чтение, а проверяется только результат

источник

00:29пожаловаться #17

Шурик in symfony

а зачем каждый раз пересчитывать права доступа? не лучше ли при наступлении события давать/отнимать права?

источник

00:29пожаловаться #18

✨

✨Basic_Instinct✨ in symfony

для модуля это все ровно бизнес, не имеет значения пересчитал ты, или прочел флаг

источник

00:30пожаловаться #19

✨

✨Basic_Instinct✨ in symfony

условие если пользователь имеет n постов, даем доступ

источник

00:30пожаловаться #20