[TYPO3-core] RFC #11813: Code cleanup: refactoring t3lib_db to match CGLs

Susanne Moog typo3 at susannemoog.de
Fri Oct 9 15:57:44 CEST 2009


Hey,

+1 by reading with these remarks (referring to the lines in the patch):

l. 69 strict string comparison (before ==, now ===)
l. 77 + 82 probably missing brackets around the whole statement
l. 725 line length (121 chars) compare with 763/764
l. 785 missing space --> $this->debug('admin_query',$query);
l. 911 missing space --> t3lib_div::inList('ALL',$explain_output[0]['type'])

Best regards,

Susanne


Christian Kuhn schrieb:
> Hey,
> 
> Felix Oertel wrote:
>> sorry, missed one bracket ... and now patched against root.
> 
> +1 on reading and testing (everything seems to work like before).
> 
> Attached a new version with some minor things:
> - Another bracket
> - Some indentation changes
> - Some forgotten whitespaces and tab fixes
> 
> Good to get a better readable version of this class.
> 
> Thanks
> Christian
> 


-- 
Susanne Moog
NEUSTA GmbH - www.neusta.de


More information about the TYPO3-team-core mailing list