[TYPO3-core] RFC #12245: Code cleanup: Reports: Cleanup API for providers in status report

[Rupert Germann]

hi Francois,

thanks for cleaning this up.

+1 by reading and testing

greets
rupert



Francois Suter schrieb:
> 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
>