[TYPO3-mvc] Setting storagePid in controller

Jochen Rau jochen.rau at typoplanet.de
Tue Aug 4 12:21:05 CEST 2009


Hi.

Bastian Waidelich wrote:
> Jochen Rau wrote:
>
>> No. We both were unhappy with setRespectStoragePageState() and
>> setCheckVisibilityState(). But we didn't find an appropriate naming.
>> Maybe someone could help us ;-).
>
> At least you could remove the "State". That does not make it clearer in
> my eyes. Why not:
> setRespectStoragePage(TRUE);
> and
> setCheckVisibility(FALSE);

The simple, the best;-)

> And instead of "checkVisibility" maybe "respectEnableFields" or
> "useEnableFields".
> I don't like the name "enable fields" but every TYPO3 dev knows it..

+1 for setUseEnableFields()

Regards
Jochen


-- 
Every nit picked is a bug fixed



More information about the TYPO3-project-typo3v4mvc mailing list