Общее Исключение

а как максимум? ???

Ну еще не стоит игнорировать то, что метод предка возвращает. Никто в этом треде это не игнорирует. Просто мне встречался код где эти методы или не вызывались, или результат их работы нигде не использовался.

to Charger, а в каком случае parent::beforeAction($action) может вернуть фолс из-за чего его надо обязательно проверять на тру?

Допустим вы объявили BeforeAction в Controller (проверка доступа) и потом в классе наследника UserController (где проверяете не забанен ли конкретный юзер, или не включен ли режим обслуживания). Вот во втором случае наверное данная проверка будет актуальной, сначала проверить родительский класс, потом делать свои проверки. Или наоборот сначала свои, потом родительские.

Эта проверка более актуальная в методах модели beforeSave, помню сталкивался с несколькими багами из-за того что не проверял (не передавал дальше), но подробности сейчас не вспомню.

Он не может вернуть false, если вы его не трогали. А так это действительно просто хороший тон. А если изменили, то лучше проверить, чтобы избежать двойных проверок и непонятных ошибок (ошибку в этом месте можно искать очень долго, в свое время убил целый день на подобное в afterSave)

Хороший тон, на случай копипасты кода в место, где такая проверка окажется к месту :)

Да и кстати ошибку так и не нашел :)

Переписал все на триггеры в бд, и удалил эти странные методы. Честно говоря кроме beforesave стараюсь больше ничего не использовать. KISS в действии. ::)

ну раз с проверкой - хороший тон и страховка: значит будем проверять

вот так работает при использовании фильтра =


if (Yii::app()->controller->action->id!='error')

{

  throw new CHttpException(403, 'preFilter($filterChain) доступ запрещен');

}