[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