[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