http://forge.typo3.org/http://forge.typo3.org/themes/typo3_forge/favicon/favicon.png?17058661692020-07-30T13:51:10ZTYPO3 ForgeTYPO3 Core - Bug #91680: Record with sub-items is doubled in workspace after movinghttp://forge.typo3.org/issues/91680?journal_id=4282322020-07-30T13:51:10ZChristian Kuhnlolli@schwarzbu.ch
<ul></ul><p>Thanks for your report. It would be great if you could manage your tone a bit, though.</p>
<p>Reproduce with styleguide in master:</p>
<ul>
<li>Have ext:styleguide loaded (it's a dev dependency for core, so it should be there already if doing core development)</li>
<li>Have a workspace on pid 0</li>
<li>Create a page in live workspace</li>
<li>Add a styleguide "Form engine - inline 1:n foreign field" record on this page, add an inline record named "1 inline el 1" to it</li>
<li>Add a second styleguide "Form engine - inline 1:n foreign field" record on this page, add an inline record named "2 inline el 1" to it</li>
<li>-> Ends up with 4 records: 2 "Form engine - inline 1:n foreign field" and two children "Form engine - inline 1:n foreign field child" </li>
<li>Switch to workspace, select page, list module</li>
<li>Resort the "Form engine - inline 1:n foreign field" (first down, or second up)</li>
<li>-> One of the children will be duplicated, this also persists if the changes are published</li>
</ul>
<p>Confirmed in current master, not tested in v10 (reporter says it happens there), NOT confirmed in v9. I'd thus suspect the pid=-1 change to be a likely candidate for this regression.</p> TYPO3 Core - Bug #91680: Record with sub-items is doubled in workspace after movinghttp://forge.typo3.org/issues/91680?journal_id=4287172020-08-09T17:57:47ZGerrit Code Review
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>Under Review</i></li></ul><p>Patch set 1 for branch <strong>master</strong> of project <strong>Packages/TYPO3.CMS</strong> has been pushed to the review server.<br />It is available at <a class="external" href="https://review.typo3.org/c/Packages/TYPO3.CMS/+/65230">https://review.typo3.org/c/Packages/TYPO3.CMS/+/65230</a></p> TYPO3 Core - Bug #91680: Record with sub-items is doubled in workspace after movinghttp://forge.typo3.org/issues/91680?journal_id=4287182020-08-09T18:04:47ZChristian Kuhnlolli@schwarzbu.ch
<ul></ul><p>As far as I found out, the children duplication happens only on children that are NOT workspace aware, if the parent is workspace aware.</p>
<p>In my test case, the parent table 'tx_styleguide_inline_1n' <strong>is</strong> workspace aware in TCA, while the child 'tx_styleguide_inline_1n_child' is <strong>not</strong>.</p>
<p>If both tables are workspace aware, I'm unable to reproduce a 'duplicate children are created' scenario. The workspace related irre functional tests show this scenario is ok, too.</p>
Thus, I narrowed that down for the styleguide scenario now:
<ul>
<li>In v9, the core <strong>also</strong> tries to duplicate the child record, but that fails with styleguide since the record is tried to be inserted on pid=-1, but the children pid db field is unsigned. This actually triggers an error by the DataHandler, which is also shown as red box.</li>
<li>In v10 & master, the core no longer uses pid=-1 for versioned records, so the insert does not fail and the record is basically duplicated (because there are no workspace related fields).</li>
</ul>
<p>I pushed a patch to fix this, by suppressing child table processing in workspace context if the child in not workspace aware.<br />We'll have to see if we'll want to cover that with tests - the core tests work with inline_tutorial tables and those are all workspace aware ... I'm currently not sure on how to cover this without adding further tables to the tests scenarios. I'll discuss that with others and the patch is marked WIP for now for that reason.</p> TYPO3 Core - Bug #91680: Record with sub-items is doubled in workspace after movinghttp://forge.typo3.org/issues/91680?journal_id=4287202020-08-09T19:07:49ZGerrit Code Review
<ul></ul><p>Patch set 2 for branch <strong>master</strong> of project <strong>Packages/TYPO3.CMS</strong> has been pushed to the review server.<br />It is available at <a class="external" href="https://review.typo3.org/c/Packages/TYPO3.CMS/+/65230">https://review.typo3.org/c/Packages/TYPO3.CMS/+/65230</a></p> TYPO3 Core - Bug #91680: Record with sub-items is doubled in workspace after movinghttp://forge.typo3.org/issues/91680?journal_id=4287212020-08-09T19:08:15ZGerrit Code Review
<ul></ul><p>Patch set 3 for branch <strong>master</strong> of project <strong>Packages/TYPO3.CMS</strong> has been pushed to the review server.<br />It is available at <a class="external" href="https://review.typo3.org/c/Packages/TYPO3.CMS/+/65230">https://review.typo3.org/c/Packages/TYPO3.CMS/+/65230</a></p> TYPO3 Core - Bug #91680: Record with sub-items is doubled in workspace after movinghttp://forge.typo3.org/issues/91680?journal_id=4287552020-08-10T12:41:34ZGerrit Code Review
<ul></ul><p>Patch set 4 for branch <strong>master</strong> of project <strong>Packages/TYPO3.CMS</strong> has been pushed to the review server.<br />It is available at <a class="external" href="https://review.typo3.org/c/Packages/TYPO3.CMS/+/65230">https://review.typo3.org/c/Packages/TYPO3.CMS/+/65230</a></p> TYPO3 Core - Bug #91680: Record with sub-items is doubled in workspace after movinghttp://forge.typo3.org/issues/91680?journal_id=4287562020-08-10T12:50:25ZGerrit Code Review
<ul></ul><p>Patch set 5 for branch <strong>master</strong> of project <strong>Packages/TYPO3.CMS</strong> has been pushed to the review server.<br />It is available at <a class="external" href="https://review.typo3.org/c/Packages/TYPO3.CMS/+/65230">https://review.typo3.org/c/Packages/TYPO3.CMS/+/65230</a></p> TYPO3 Core - Bug #91680: Record with sub-items is doubled in workspace after movinghttp://forge.typo3.org/issues/91680?journal_id=4287892020-08-10T16:41:36ZGerrit Code Review
<ul></ul><p>Patch set 1 for branch <strong>10.4</strong> of project <strong>Packages/TYPO3.CMS</strong> has been pushed to the review server.<br />It is available at <a class="external" href="https://review.typo3.org/c/Packages/TYPO3.CMS/+/65252">https://review.typo3.org/c/Packages/TYPO3.CMS/+/65252</a></p> TYPO3 Core - Bug #91680: Record with sub-items is doubled in workspace after movinghttp://forge.typo3.org/issues/91680?journal_id=4287912020-08-10T17:00:05ZChristian Kuhnlolli@schwarzbu.ch
<ul><li><strong>Status</strong> changed from <i>Under Review</i> to <i>Resolved</i></li><li><strong>% Done</strong> changed from <i>0</i> to <i>100</i></li></ul><p>Applied in changeset <a class="changeset" title="[BUGFIX] Skip processing of inline children which aren't workspace-aware When dealing with inlin..." href="http://forge.typo3.org/projects/typo3cms-core/repository/1749/revisions/84cbe10d85e7d05cf134582e1a07b4eab8c38659">84cbe10d85e7d05cf134582e1a07b4eab8c38659</a>.</p> TYPO3 Core - Bug #91680: Record with sub-items is doubled in workspace after movinghttp://forge.typo3.org/issues/91680?journal_id=4287922020-08-10T17:32:01ZGerrit Code Review
<ul><li><strong>Status</strong> changed from <i>Resolved</i> to <i>Under Review</i></li></ul><p>Patch set 1 for branch <strong>9.5</strong> of project <strong>Packages/TYPO3.CMS</strong> has been pushed to the review server.<br />It is available at <a class="external" href="https://review.typo3.org/c/Packages/TYPO3.CMS/+/65253">https://review.typo3.org/c/Packages/TYPO3.CMS/+/65253</a></p> TYPO3 Core - Bug #91680: Record with sub-items is doubled in workspace after movinghttp://forge.typo3.org/issues/91680?journal_id=4287932020-08-10T17:52:55ZGerrit Code Review
<ul></ul><p>Patch set 2 for branch <strong>9.5</strong> of project <strong>Packages/TYPO3.CMS</strong> has been pushed to the review server.<br />It is available at <a class="external" href="https://review.typo3.org/c/Packages/TYPO3.CMS/+/65253">https://review.typo3.org/c/Packages/TYPO3.CMS/+/65253</a></p> TYPO3 Core - Bug #91680: Record with sub-items is doubled in workspace after movinghttp://forge.typo3.org/issues/91680?journal_id=4287942020-08-10T18:00:04ZChristian Kuhnlolli@schwarzbu.ch
<ul><li><strong>Status</strong> changed from <i>Under Review</i> to <i>Resolved</i></li></ul><p>Applied in changeset <a class="changeset" title="[BUGFIX] Skip processing of inline children which aren't workspace-aware When dealing with inlin..." href="http://forge.typo3.org/projects/typo3cms-core/repository/1749/revisions/d079793ba0e125fcac294c03105e983f31e49e11">d079793ba0e125fcac294c03105e983f31e49e11</a>.</p> TYPO3 Core - Bug #91680: Record with sub-items is doubled in workspace after movinghttp://forge.typo3.org/issues/91680?journal_id=4323732020-09-29T15:07:20ZBenni Mackbenni@typo3.org
<ul><li><strong>Status</strong> changed from <i>Resolved</i> to <i>Closed</i></li></ul>