[TYPO3-core] RFC #6833: Possibility to add inline JS to page

Ingmar Schlecht ingmar at typo3.org
Mon Nov 26 10:11:52 CET 2007


Hi Steffen,

+1 based on reading.

But: You forgot to attach a .txt file with the doc changes for
http://wiki.typo3.org/index.php/Pending_Documentation

Some minor comments about the coding style:


> +		//add additional inline JS

Comments are indented with one more tab than the code it is describing.
(And add a space before the word "add" in the comment.)

> +		// defined in TS with page.additionalInlineJS
> +		if (is_array($GLOBALS['TSFE']->pSetup['additionalInlineJS.']))	{
> +			$GLOBALS['TSFE']->additionalInlineJS[]= $GLOBALS['TSFE']->cObj->cObjGet($GLOBALS['TSFE']->pSetup['additionalInlineJS.'],'additionalInlineJS.');
> +		}
> +		// defined in php
> +		if(is_array($GLOBALS['TSFE']->additionalInlineJS)) {
> +			foreach($GLOBALS['TSFE']->additionalInlineJS as $key=>$val) {
> +				if(!is_array($val)) {
> +					$_scriptCode .= chr(10).$val.chr(10);	
> +				}
> +			}
> +		}
> +		
>  		// Should minify?

Apparently the above comment isn't indented correctly either, so please
just correct it as well.

cheers
Ingmar

-- 
Ingmar Schlecht
TYPO3 Association Active Member


More information about the TYPO3-team-core mailing list