[TYPO3-core] RFC: #2278 - Hook request for t3lib_TCEmain::checkModifyAccessList()

[Michael Stucki]

Hi Peter,

>> I am not sure if the name of the hook is well-chosen: Instead of calling
>> it "recheckModifyAccessList" I would propose "checkModifyAccessList_post"
>> to make it clear where the hook is placed in the code. Do you agree with
>> this?
> 
> pls rename it like you proposed.
> 
> [...]
>> +            foreach ($hookObjectsArr as $hookObj)   {
>> +                    if (method_exists($hookObj, 'recheckModifyAccessList')) {
>> +                            $hookObj->recheckModifyAccessList($table, $this->cmdmap, $this,
>> &$res);
> [...]
> 
> recheck is something pretty special for this case, postfixing it by
> '_post' is more general. So this ^^ should still be changed, shouldn't it?

You mean like this? See attachment.

- michael
-- 
Use a newsreader! Check out
http://typo3.org/community/mailing-lists/use-a-news-reader/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: bug_2278_v2.diff
Type: text/x-diff
Size: 1081 bytes
Desc: not available
Url : http://lists.netfielders.de/pipermail/typo3-team-core/attachments/20061107/76b438b5/attachment.bin