[TYPO3-core] RFC: #15172: Add wrapper files for list/filelist modules

Ingo Renner ingo at typo3.org
Thu Jul 29 15:55:27 CEST 2010


Steffen Kamper wrote:

Hey Steffen,

> I committed v3 to trunk rev 8273
>
> v3 adds the referer to deprecation log entry.
> (there was also a type error in file_list.php which is corrected now)

Thanks for the wrappers, however I just saw now that there is the method 
t3lib_extMgm::createListViewLink().

Now I'm wondering, why should the EM be responsible for creating links 
to modules / certain views? It's not in its domain. Could you please 
think about moving it to a better location?


Also a note for everyone else here: When reviewing patches please also 
consider the naming of APIs and whether they make sense in the place 
where they are, especially for new ones.


best
Ingo

-- 
Ingo Renner
TYPO3 Core Developer, Release Manager TYPO3 4.2



More information about the TYPO3-team-core mailing list