http://forge.typo3.org/http://forge.typo3.org/themes/typo3_forge/favicon/favicon.png?17058661692018-09-04T14:28:25ZTYPO3 ForgeTYPO3 Core - Task #86141: Remove superfluous database contraint in DataMapProcessorhttp://forge.typo3.org/issues/86141?journal_id=3724732018-09-04T14:28:25ZGerrit Code Review
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>Under Review</i></li></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/58049">https://review.typo3.org/58049</a></p> TYPO3 Core - Task #86141: Remove superfluous database contraint in DataMapProcessorhttp://forge.typo3.org/issues/86141?journal_id=3724932018-09-04T19:20:08ZGerrit 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/58049">https://review.typo3.org/58049</a></p> TYPO3 Core - Task #86141: Remove superfluous database contraint in DataMapProcessorhttp://forge.typo3.org/issues/86141?journal_id=3730382018-09-09T22:30:18ZNicole Cordestypo3@cordes.co
<ul><li><strong>File</strong> <a href="/attachments/33708">issue86141_0.1.0_201809092229.zip</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/33708/issue86141_0.1.0_201809092229.zip">issue86141_0.1.0_201809092229.zip</a> added</li></ul><p>Steps to reproduce the current pending patch and its bugfixes:</p>
<p>- install attached extension "issue86141" <br />- add a tt_content element in default language as well as a content element for "Related Content" (tab "Extended")<br />- translate the default record to language_1<br />- open the new content element of language_1 and remove the attached record in "Related Content", switch from "Value of default language" to "Custom value" and add another new record in that field<br />- translate the language_1 record to language_2<br />- add a new "Related Content" element to language_1 record<br />- translate the language_2 record to language_3</p>
<p>Expected behavior:</p>
<p>- Two new tt_content elements should be created for language_3</p>
<p>Actual behavior:</p>
<p>- An exception ("Child record was not processed", 1486233164) is thrown and prevents the overlay from being removed</p> TYPO3 Core - Task #86141: Remove superfluous database contraint in DataMapProcessorhttp://forge.typo3.org/issues/86141?journal_id=3730432018-09-10T00:10:23ZGerrit 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/58049">https://review.typo3.org/58049</a></p> TYPO3 Core - Task #86141: Remove superfluous database contraint in DataMapProcessorhttp://forge.typo3.org/issues/86141?journal_id=3730662018-09-10T16:06:05ZOliver Haderoliver.hader@typo3.org
<ul></ul><p>The issue is a bit more complex... in the first run I was wondering, why there are localized records that are not having any language parent-relation (<code>l10n_parent</code>).</p>
<p>Since <a class="external" href="https://review.typo3.org/#/c/50022/">https://review.typo3.org/#/c/50022/</a> (which allowed chained translations from an existing localized record), <code>l10n_parent</code> is only set for the default language.<br />In a consequence, records that are only available for a given language and are localized from there into a different language, <code>l10n_parent</code> will stay empty.<br />The behavior for free-mode localizations (<code>copyToLanguage</code> command in DataHandler) have a similar effect, <code>l10n_parent</code> is set to <code>0</code>.</p>
<blockquote>
<p>Great, in this case we cannot distinguish between connected relations (that were not created from the default language) and free-mode localizations anymore. That's the first issue...</p>
</blockquote>
<p><code>DataMapProcessor</code> can only handle items that are connected (thus, has accordant ancestor relationship - in terms of <strong>a chain of translations</strong>) - synchronizing values between items, that are not connected to each other, is simply wrong.</p>
<blockquote>
<p>Awesome, <code>DataMapProcessor</code> is currently written with the assumption that connected elements always have <code>l10n_parent</code> defined - which is not always the case as written above. That's the second issue...</p>
</blockquote>
<p>In order to resolve connected translations chains in a reliable way, the <code>l10n_source</code> values have to be resolved to their relative ancestors. However, since there is no way to <strong>distinguish between connected and free-mode localizations</strong> that have not been created from some <strong>record in default language</strong> the information simply is not available.</p>
<blockquote>
<p>Great, this issue cannot be solved without enriching the information that is persisted currently. Besides that, <code>l10n_source</code> is not available for all tables. Thus, this is not generic. And that's the third and most severe issue...</p>
</blockquote>
<p><strong>Conclusion: Cannot be solved in a generic way with the current information that is available in the database.</strong></p> TYPO3 Core - Task #86141: Remove superfluous database contraint in DataMapProcessorhttp://forge.typo3.org/issues/86141?journal_id=3730672018-09-10T16:07:36ZOliver Haderoliver.hader@typo3.org
<ul><li><strong>Related to</strong> <i><a class="issue tracker-1 status-5 priority-4 priority-default closed" href="/issues/78059">Bug #78059</a>: Checks in DataHandler localize mismatch with new Localization Wizard</i> added</li></ul> TYPO3 Core - Task #86141: Remove superfluous database contraint in DataMapProcessorhttp://forge.typo3.org/issues/86141?journal_id=3730732018-09-10T16:42:34ZGerrit Code Review
<ul></ul><p>Patch set 6 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/58049">https://review.typo3.org/58049</a></p> TYPO3 Core - Task #86141: Remove superfluous database contraint in DataMapProcessorhttp://forge.typo3.org/issues/86141?journal_id=3730852018-09-10T17:55:52ZGerrit Code Review
<ul></ul><p>Patch set 7 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/58049">https://review.typo3.org/58049</a></p> TYPO3 Core - Task #86141: Remove superfluous database contraint in DataMapProcessorhttp://forge.typo3.org/issues/86141?journal_id=3730882018-09-10T18:09:27ZTymoteusz Motylewskit.motylewski@gmail.com
<ul></ul><p>@Olly, you mean that there is no way to detect default language ancestor except for walkin the chain up? <br />Can you sum up which combinations are/can be supported and which not (in terms of chains of translations)?</p> TYPO3 Core - Task #86141: Remove superfluous database contraint in DataMapProcessorhttp://forge.typo3.org/issues/86141?journal_id=3731412018-09-12T08:50:22ZOliver Haderoliver.hader@typo3.org
<ul></ul><p>@Tymek I'll do and create a new ticket...</p> TYPO3 Core - Task #86141: Remove superfluous database contraint in DataMapProcessorhttp://forge.typo3.org/issues/86141?journal_id=3731432018-09-12T08:51:35ZOliver Haderoliver.hader@typo3.org
<ul><li><strong>Related to</strong> <i><a class="issue tracker-1 status-1 priority-4 priority-default" href="/issues/86231">Bug #86231</a>: Distinguish between free-mode localization and chained translation</i> added</li></ul> TYPO3 Core - Task #86141: Remove superfluous database contraint in DataMapProcessorhttp://forge.typo3.org/issues/86141?journal_id=3731502018-09-12T09:31:28ZOliver Haderoliver.hader@typo3.org
<ul></ul><p>@Tymek Here we go: <a class="external" href="https://forge.typo3.org/issues/86231">https://forge.typo3.org/issues/86231</a></p> TYPO3 Core - Task #86141: Remove superfluous database contraint in DataMapProcessorhttp://forge.typo3.org/issues/86141?journal_id=3742332018-09-25T16:19:12ZGerrit Code Review
<ul></ul><p>Patch set 8 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/58049">https://review.typo3.org/58049</a></p> TYPO3 Core - Task #86141: Remove superfluous database contraint in DataMapProcessorhttp://forge.typo3.org/issues/86141?journal_id=3742342018-09-25T16:23:11ZGerrit Code Review
<ul></ul><p>Patch set 9 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/58049">https://review.typo3.org/58049</a></p> TYPO3 Core - Task #86141: Remove superfluous database contraint in DataMapProcessorhttp://forge.typo3.org/issues/86141?journal_id=3742412018-09-25T17:07:57ZGerrit Code Review
<ul></ul><p>Patch set 10 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/58049">https://review.typo3.org/58049</a></p> TYPO3 Core - Task #86141: Remove superfluous database contraint in DataMapProcessorhttp://forge.typo3.org/issues/86141?journal_id=3742452018-09-25T17:11:02ZGerrit Code Review
<ul></ul><p>Patch set 11 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/58049">https://review.typo3.org/58049</a></p> TYPO3 Core - Task #86141: Remove superfluous database contraint in DataMapProcessorhttp://forge.typo3.org/issues/86141?journal_id=3922202018-12-14T10:57:07ZGerrit Code Review
<ul></ul><p>Patch set 12 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/58049">https://review.typo3.org/58049</a></p> TYPO3 Core - Task #86141: Remove superfluous database contraint in DataMapProcessorhttp://forge.typo3.org/issues/86141?journal_id=4139192020-01-16T13:34:34ZGerrit Code Review
<ul></ul><p>Patch set 13 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/+/58049">https://review.typo3.org/c/Packages/TYPO3.CMS/+/58049</a></p> TYPO3 Core - Task #86141: Remove superfluous database contraint in DataMapProcessorhttp://forge.typo3.org/issues/86141?journal_id=4180942020-03-07T21:24:37ZGerrit Code Review
<ul></ul><p>Patch set 14 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/+/58049">https://review.typo3.org/c/Packages/TYPO3.CMS/+/58049</a></p> TYPO3 Core - Task #86141: Remove superfluous database contraint in DataMapProcessorhttp://forge.typo3.org/issues/86141?journal_id=4438102021-05-05T09:18:03ZGerrit Code Review
<ul></ul><p>Patch set 15 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/+/58049">https://review.typo3.org/c/Packages/TYPO3.CMS/+/58049</a></p> TYPO3 Core - Task #86141: Remove superfluous database contraint in DataMapProcessorhttp://forge.typo3.org/issues/86141?journal_id=4438192021-05-05T09:56:39ZGerrit Code Review
<ul></ul><p>Patch set 16 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/+/58049">https://review.typo3.org/c/Packages/TYPO3.CMS/+/58049</a></p> TYPO3 Core - Task #86141: Remove superfluous database contraint in DataMapProcessorhttp://forge.typo3.org/issues/86141?journal_id=4438202021-05-05T10:03:13ZGerrit Code Review
<ul></ul><p>Patch set 17 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/+/58049">https://review.typo3.org/c/Packages/TYPO3.CMS/+/58049</a></p> TYPO3 Core - Task #86141: Remove superfluous database contraint in DataMapProcessorhttp://forge.typo3.org/issues/86141?journal_id=4438242021-05-05T10:23:35ZGerrit 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/+/69030">https://review.typo3.org/c/Packages/TYPO3.CMS/+/69030</a></p> TYPO3 Core - Task #86141: Remove superfluous database contraint in DataMapProcessorhttp://forge.typo3.org/issues/86141?journal_id=4438252021-05-05T10:39:26ZGerrit Code Review
<ul></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/+/69031">https://review.typo3.org/c/Packages/TYPO3.CMS/+/69031</a></p> TYPO3 Core - Task #86141: Remove superfluous database contraint in DataMapProcessorhttp://forge.typo3.org/issues/86141?journal_id=4438262021-05-05T10:40:09ZGerrit 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/+/69031">https://review.typo3.org/c/Packages/TYPO3.CMS/+/69031</a></p> TYPO3 Core - Task #86141: Remove superfluous database contraint in DataMapProcessorhttp://forge.typo3.org/issues/86141?journal_id=4438352021-05-05T11:47:40ZGerrit Code Review
<ul></ul><p>Patch set 3 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/+/69031">https://review.typo3.org/c/Packages/TYPO3.CMS/+/69031</a></p> TYPO3 Core - Task #86141: Remove superfluous database contraint in DataMapProcessorhttp://forge.typo3.org/issues/86141?journal_id=4438462021-05-05T13:35:40ZNicole Cordestypo3@cordes.co
<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] Return correct source record for relations This change addresses a flaw when resolving ..." href="http://forge.typo3.org/projects/typo3cms-core/repository/1749/revisions/e5a3bb11ec4ea0efb16fbb735ea0b8a1245b2b35">e5a3bb11ec4ea0efb16fbb735ea0b8a1245b2b35</a>.</p> TYPO3 Core - Task #86141: Remove superfluous database contraint in DataMapProcessorhttp://forge.typo3.org/issues/86141?journal_id=4441992021-05-11T16:01:13ZBenni Mackbenni@typo3.org
<ul><li><strong>Status</strong> changed from <i>Resolved</i> to <i>Closed</i></li></ul>