[TYPO3-dam-devel] RFC 8057: 4.2 compatibility

[Michiel Roos]

+1

Patch solves the problem of passing wrong data to arrayToLogString.

On May 5, 2008, at 5:06 PM, Dan Osipov wrote:

> This is a svn patch request
>
> Type: Bugfix
>
> Versions affected: 1.0.13 & trunk
>
> Description: cc_devlog extension causes an error in the list view of  
> the DAM when it tries to write to a log. This wasn't an issue in  
> Typo3 4.1.x, where the parameter type was not enforced, but this  
> makes the list view unusable in 4.2.0
> See the bugtracker entry for more information.
>
> Attached patch solves the issue. Big thanks for Benjamin Mack for  
> this one!!
> -- 
> Dan Osipov
> Calkins Media
> http://phillyburbs.com/
> Index: lib/class.tx_dam_selstorage.php
> ===================================================================
> --- lib/class.tx_dam_selstorage.php	(revision 9058)
> +++ lib/class.tx_dam_selstorage.php	(working copy)
> @@ -278,7 +278,9 @@
> 		$storeControl = t3lib_div::_GP('selStoreControl');
> 		$storeIndex = $storeControl['STORE'];
>
> -		if ($this->writeDevLog) t3lib_div::devLog('Store command:  
> '.t3lib_div::arrayToLogString($storeControl), 'tx_dam_selStorage', 0);
> +		if ($this->writeDevLog) {
> +			t3lib_div::devLog('Store command: '.(is_array($storeControl) ?  
> t3lib_div::arrayToLogString($storeControl) : $storeControl),  
> 'tx_dam_selStorage', 0);
> +		}
>
> 		$msg = '';
> 		$saveSettings = FALSE;
> _______________________________________________
> Before posting to this list, please have a look to the posting rules
> on the following websites:
>
> http://typo3.org/teams/core/core-mailinglist-rules/
> http://typo3.org/development/bug-fixing/diff-and-patch/
> _______________________________________________
> TYPO3-team-dam mailing list
> TYPO3-team-dam at lists.netfielders.de
> http://lists.netfielders.de/cgi-bin/mailman/listinfo/typo3-team-dam


Met vriendelijke groet,

Michiel Roos

Netcreators BV :: creation and innovation
www.netcreators.com

Interesse in werken bij Netcreators?
http://www.netcreators.com/bedrijf/vacatures/