Bug #89555
closed
Querying records in workspaces does not return all proper values due to "pid=-1" database
100%
Description
Versioned records have "pid=-1" due to heavy legacy reasons before we had Doctrine DBAL and Restrictions, where one could accidentally query versioned records in their third party extension.
"pid=-1" has several drawbacks on versioned records:
- One could never query results properly by doing SQL queries on versioned records (as only equivalent live records were selected, and then overlaid)
- Unnecessary logic regarding moving and sorting due to versioned records
I propose to migrate to the proper "pid" of the live version, and look at "t3ver_oid" (live record) as all queries in the Core are already work based on this field.
An upgrade wizard then sets the "pid" field properly for all existing versions.
Updated by Gerrit Code Review about 5 years ago
- Status changed from New to Under Review
Patch set 27 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61699
Updated by Gerrit Code Review about 5 years ago
Patch set 28 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61699
Updated by Gerrit Code Review about 5 years ago
Patch set 29 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61699
Updated by Gerrit Code Review about 5 years ago
Patch set 30 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61699
Updated by Gerrit Code Review about 5 years ago
Patch set 31 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61699
Updated by Gerrit Code Review about 5 years ago
Patch set 32 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61699
Updated by Gerrit Code Review almost 5 years ago
Patch set 33 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61699
Updated by Gerrit Code Review almost 5 years ago
Patch set 34 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61699
Updated by Gerrit Code Review almost 5 years ago
Patch set 35 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61699
Updated by Gerrit Code Review over 4 years ago
Patch set 36 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61699
Updated by Gerrit Code Review over 4 years ago
Patch set 37 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61699
Updated by Gerrit Code Review over 4 years ago
Patch set 38 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61699
Updated by Gerrit Code Review over 4 years ago
Patch set 39 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61699
Updated by Gerrit Code Review over 4 years ago
Patch set 40 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61699
Updated by Gerrit Code Review over 4 years ago
Patch set 41 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61699
Updated by Gerrit Code Review over 4 years ago
Patch set 42 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61699
Updated by Gerrit Code Review over 4 years ago
Patch set 43 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61699
Updated by Gerrit Code Review over 4 years ago
Patch set 44 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61699
Updated by Gerrit Code Review over 4 years ago
Patch set 45 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61699
Updated by Benni Mack over 4 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 01c6cdc1b58be4451a07998ccc1b943d04bb6d7a.
Updated by Andreas Kienast over 4 years ago
- Related to Bug #91144: Copying records in pid 0 is not possible anymore added
Updated by Christian Kuhn over 4 years ago
- Related to Bug #91851: Page module translation wizard shows duplicate records if in workspace added
Updated by Christian Kuhn over 4 years ago
- Related to Bug #91144: Copying records in pid 0 is not possible anymore added
Updated by Christian Kuhn over 4 years ago
- Related to deleted (Bug #91144: Copying records in pid 0 is not possible anymore)
Updated by Christian Kuhn over 4 years ago
- Due date set to 2019-09-12
- Start date changed from 2019-10-31 to 2019-09-12
- Follows Bug #89122: Inconsistent evaluation of pid=-1 with workspace records added
Updated by Christian Kuhn over 4 years ago
- Due date changed from 2019-09-12 to 2019-09-13
- Start date changed from 2019-09-12 to 2019-09-13
- Follows Bug #91881: Massive load when deleting sys_workspace record on pid 0 added
Updated by Christian Kuhn over 4 years ago
- Follows deleted (Bug #91881: Massive load when deleting sys_workspace record on pid 0)
Updated by Christian Kuhn over 4 years ago
- Precedes Bug #91881: Massive load when deleting sys_workspace record on pid 0 added
Updated by Christian Kuhn over 4 years ago
- Related to Bug #91943: Workspace preview shows correct items for sub menus added
Updated by Christian Kuhn almost 3 years ago
- Related to Task #96279: Remove v9->v10 upgrade wizards added
Updated by Christian Kuhn about 2 years ago
- Related to Task #98475: Avoid negative pid's in SQL schema added