[TYPO3-core] RFC #12245: Code cleanup: Reports: Cleanup API for providers in status report
Francois Suter
fsu-lists at cobweb.ch
Thu Oct 15 21:24:46 CEST 2009
This is an SVN patch request.
Type: Code cleanup
Bugtracker references:
http://bugs.typo3.org/view.php?id=12245
Branches:
Trunk
Problem:
Currently the list of status providers for the status report is
alongside the title, description and report properties in
$GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['reports']['tx_reports']['status'].
This makes it necessary to filter the title, description and report
properties when looping over the status providers. This makes for messy
code, hard to understand for developers. Furthermore it means that any
new property added to reports must also be filtered out (like a custom
icon, see RFC #12167).
Solution:
Move the list of status providers to a "providers" sub-array.
Notes:
1) I was meaning to do this clean up at some point, but now it appears
necessary for RFC #12167 (custom icon for reports), which in turn is
necessary for RFC #12178 (installed services report).
2) ExtBase also registers a status provider, but I can't change it here
as it is included using svn:externals. So with this patch, you will see
the extbase status disappear. I will file an issue in the ExtBase
project as soon as this patch is in.
Cheers
--
Francois Suter
Cobweb Development Sarl - http://www.cobweb.ch
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: rfc12245.diff
Url: http://lists.netfielders.de/pipermail/typo3-team-core/attachments/20091015/8f3fa0d7/attachment.txt
More information about the TYPO3-team-core
mailing list