Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

DC_Table::copy() kopiert nicht rekursive #1360

Closed
aschempp opened this issue Nov 29, 2011 · 4 comments
Closed

DC_Table::copy() kopiert nicht rekursive #1360

aschempp opened this issue Nov 29, 2011 · 4 comments
Assignees
Labels
Milestone

Comments

@aschempp
Copy link
Member

Habe gerade einen komplizierten Fehler in der DC_Table::copy() Funktion gefunden, wenn Child-Records kopiert werden sollen.
Ganz unten in der Funktion findet sich folgender Code:

if ($objInsertStmt->affectedRows && count($cctable[$k]) && $kk != $parentId)
{
    $this->copyChilds($k, $objInsertStmt->insertId, $kk, $parentId);
}

Die Prüfung auf count($cctable[$k]) sollte hier nicht sein. Im Falle der tl_page enthält dies die tl_article, was soweit funktioniert. Falls eine Tabelle aber keine Untertabellen hat, sondern einfach nur eine Baumstruktur ist, wird nur die erste Ebene der Struktur kopiert, und copyChilds() für die zweite Ebene nicht mehr durchgeführt.

--- Originally created on January 5th, 2010, at 11:15am (ID 1360)

@ghost ghost assigned leofeyer Nov 29, 2011
@leofeyer
Copy link
Member

Ich kann mich erinnern, dass speziell das Kopieren von Kinddatensätzen über mehrere Tabellen relativ kompliziert zu implementieren war. Daswegen bin ich mir auch fast sicher, dass das Entfernen der besagten Anweisung zu Problemen führen wird - auch wenn mir auf Anhieb kein Testszenario eingefallen ist. Bitte versuch daher mal folgenden Code:

if ($objInsertStmt->affectedRows && (count($cctable[$k]) || $GLOBALS['TL_DCA'][$k]['list']['sorting']['mode'] == 5) && $kk != $parentId)

--- Originally created on January 8th, 2010, at 01:04pm

@aschempp
Copy link
Member Author

Funktioniert perfekt, habe das jetzt bei mir so eingebaut.

--- Originally created on January 8th, 2010, at 03:07pm

@leofeyer
Copy link
Member

Behoben in 82c9dfe.

--- Originally created on January 8th, 2010, at 07:47pm

@leofeyer
Copy link
Member

--- Originally completed on January 8th, 2010, at 07:47pm

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants