[TYPO3-core] RFC #13998: Introduce Automatic versioning of CSS and JS files
Ernesto Baschny [cron IT]
ernst at cron-it.de
Fri Apr 9 10:19:44 CEST 2010
Stefano Kowalke schrieb am 09.04.2010 10:00:
> Hi Lars,
>
>>> - CGL: comments are intended one addtional tab
>> In t3liv_div, the function above is using same indention, which is one tab and one space afterwards. Is this wrong?
>
> If someone touch exist code or write new code for an bugfix or feature
> he must follow the CGL. So t3lib_div is no good example for following
> the CGL ;-)
>
> To have a first fast overview, please setup your mail client that
> attached files displayed inline.
>
>
> Some CGL nitpicking (line numbers are patch v5 related):
>
> Line 9 and 17:
> Inline comments schould go one line above with one more intention.
> // Boolean. If true, ...
> 'versionNumberInFilename' => FALSE,
config_default.php is parsed by the install too ("All configuration"),
so it needs to be that exact syntax.
> Line 43: Spaces before and after the concatenation dot
> Line 69: Space before the concatenation dot
> Line 210: Spaces before and after the assignment operator
> Line 210: Space before the first concatenation dot
> Line 210: Spaces before and after the second concatenation dot
> Line 223: Space before the first concatenation dot
> Line 223: Spaces before and after the second concatenation dot
> Line 236 and 245: Space before the assignment operator (.=)
Please review by _v6. Apart from the CGL nitpicking, I also expect some
functional review or test.
Cheers,
Ernesto
More information about the TYPO3-team-core
mailing list