[TYPO3-core] RFC #13623 : Bug: Deprecation log for IMAGE.alttext does not work
Susanne Moog
typo3 at susanne-moog.de
Tue Mar 2 10:39:17 CET 2010
Hi,
Sebastian Gebhard wrote:
> This is an SVN patch request.
>
> Type: bugfix, tweak
>
> Bugtracker references: http://bugs.typo3.org/view.php?id=13623
>
> Branches: 4.3, trunk
>
> Problem:
>
> Due to a typo no entry in the deprecation log is done when using the
> deprecated IMAGE.alttext property.
>
> Solution:
>
> I corrected the property names in the IF-Condition to lowercase.
>
> Additionally I merged the two nested IF-Clauses to one. This also solves
> the little issue that $conf['altText'] = $conf['alttext']; was assigned
> even if there is no $conf['alttext'] - an IF-Check is done anyway, so
> why not include the assignement in there. This should tweak performance
> a little bit.
>
> I was not sure if I should include the tweak into this RFC. That's why I
> attached also a bugfix only patch.
>
> Notes:
>
> Steps to test:
>
> - Set up an IMAGE Object with an alttext property and call it in the FE
> - Open localconf/deprecation_*.log - Nothing about IMAGE.alttext in there
> - Apply patch and call FE again
> - deprecation_*.log will have an entry about IMAGE.alttext
Your patch works as expected, although I think there is a conceptional
error in the code (even before your patch). Why is alttext only
deprecated if not used together with altText? If I use the following
code - for whatever reason - no entry in the deprecation log is
triggered, though I'd say alttext is still deprecated.
page.130 = IMAGE
page.130 {
file = fileadmin/user_upload/CIMG1597.JPG
alttext = sometext
altText = someOtherText
}
What do you think?
Best regards,
Susanne
More information about the TYPO3-team-core
mailing list