Bug #57291
RelationHandler->purgeItemArray() should use isTableWorkspaceEnabled check
| Status: | Resolved | Start date: | 2014-03-25 | |
|---|---|---|---|---|
| Priority: | Must have | Due date: | ||
| Assigned To: | - | % Done: | 100% |
|
| Category: | Workspaces | Spent time: | - | |
| Target version: | next-patchlevel | |||
| TYPO3 Version: | 6.2 | Is Regression: | Yes | |
| PHP Version: | 5.4 | Sprint Focus: | ||
| Complexity: | no-brainer |
Description
The purgeItemArray method produces database errors ("Unknown column 't3ver_oid' in 'field list'") if there are mm relations without workspace support.
A check for BackendUtility::isTableWorkspaceEnabled solves the problem.
(See http://forge.typo3.org/issues/57169)
RelationHandler.patch
(1.3 kB)
Related issues
Associated revisions
[BUGFIX] Check WorkspaceEnabled for MM field related table
If you try to edit a MM field associated to a table which doesn't use
versioning (t3ver_* fields are missing) you get a database error.
This patch adds a check if versioning for the table is enabled.
Resolves: #57291
Releases: 6.2
Change-Id: I3fc5dfeb77ce9a67a6e4494a3b2247c3007d0d25
Reviewed-on: https://review.typo3.org/28783
Reviewed-by: Oliver Hader
Tested-by: Oliver Hader
History
#1 Updated by Gerrit Code Review over 1 year ago
- Status changed from New to Under Review
Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/28783
#2 Updated by Alexander Opitz over 1 year ago
Pushed the added patch to gerrit.
#3 Updated by Gerrit Code Review over 1 year ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/28783
#4 Updated by Gerrit Code Review over 1 year ago
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/28783
#5 Updated by Gerrit Code Review over 1 year ago
Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/28783
#6 Updated by Gerrit Code Review over 1 year ago
Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/28783
#7 Updated by Gerrit Code Review over 1 year ago
Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/28783
#8 Updated by Gerrit Code Review over 1 year ago
Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/28783
#9 Updated by Oliver Hader over 1 year ago
- Is Regression changed from No to Yes
#10 Updated by Oliver Hader over 1 year ago
- Priority changed from Should have to Must have
#11 Updated by Oliver Hader over 1 year ago
- Target version set to next-patchlevel
#12 Updated by Gerrit Code Review over 1 year ago
Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/28783
#13 Updated by Sebastian Lechenbauer over 1 year ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 022a3fc0fceaf3ceac862be9ca1c56414d1da9c9.