[TYPO3-core] RFC: Cleanup #13365: Table headers in t3lib_fullsearch

[Christian Kuhn]

Committed attached patch to:
- trunk rev. 6870
- 4.3 rev. 6871

After my +1 on reading and testing.

I committed this patch to 4.3 because it makes the code a bit cleaner 
without side effects, I bet Olly won't hurt me for that.

Changes in latest version:
- Use th instead of td for last column
- Sum up content in an array and implode with chr(10) to make html more 
readable
- Added reasonable method header
- Renamed some variables for better reading.

Final patch attached.

Thanks
Christian


Georg Ringer wrote:
> This is an SVN patch request.
> 
> Type: Cleanup
> 
> Bugtracker references:
> http://bugs.typo3.org/view.php?id=13365
> 
> Branches: Trunk
> 
> Problem:
> This function renders the headers in t3lib_fullsearch which is used in
> sysexts sys_actions and lowlevel (BE module DB check > full search).
> Table headers are needed to style those cells independed from all others.
> 
> Solution:
> Instead of <td><strong>foo</strong></td> use <th>fo</th>. All other
> changes are to apply CGL to this function.
> 
> Additional information:
> This bug maybe looks like it is not worth a patch but I need this change
> for redoing the taskcenter which btw looks already really wonderful. PM
> for more details.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 13365_04.diff
Type: text/x-patch
Size: 2362 bytes
Desc: not available
URL: <http://lists.typo3.org/pipermail/typo3-team-core/attachments/20100207/3a33953f/attachment.bin>