Bug #28882
closed
Workspace module - diff-view should only show fields configured in TCA 'showRecordFieldList'
100%
Description
Hi,
i found a bug in the detailed diff-view of the workspace module.
If a tt_news record (or another extension-record) is listed as a new version in the workspace module, the diff-view shows all modified DB fields (uid, pid, crdate, l18n_diffsource, etc.). The correct behaviour would be that it just shows the modified fields configured in $TCA[tablename][interface][showRecordFieldList].
There is such a query in the sources but it expects that the whole TCA Array for every table is loaded. You have to load the TCA for each table with t3lib_div::loadTCA(tablename).
I attached a patch file to fix this bug.
Regards,
Simon
Files
Updated by Tolleiv Nietsch over 13 years ago
- Status changed from New to Needs Feedback
Pretty obvious. Would you mind submitting it to Gerrit?
Updated by Mr. Hudson over 13 years ago
Patch set 1 of change Ia6bf921726d7dd6da85a666365ce48c0ac6c16ae has been pushed to the review server.
It is available at http://review.typo3.org/4463
Updated by Tolleiv Nietsch over 13 years ago
- Status changed from Needs Feedback to Under Review
Updated by Mr. Hudson over 13 years ago
Patch set 1 of change I56b098accc2dd31db69d7690cd5d29f53f74e093 has been pushed to the review server.
It is available at http://review.typo3.org/4464
Updated by Simon Apold over 13 years ago
Thanks for submitting the change. Last week I was on vacation so I wasn't able to.
Updated by Simon Apold over 13 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset commit:0bf9fb36e3ba20a600b6aed799a9a3c53fd191fd.
Updated by Michael Stucki almost 11 years ago
- Category changed from Bugs to Workspaces
Updated by Michael Stucki almost 11 years ago
- Project changed from 624 to TYPO3 Core
- Category changed from Workspaces to Workspaces
Updated by