felixpfeiffer / Contao-Subcolumns

Subcolumns version 2
5 stars 9 forks source link

Seiten kopieren geht nicht #4

Closed psi-4ward closed 11 years ago

psi-4ward commented 12 years ago

Kopiert man eine Seite, die einen Artikel mit Spaltenset beinhaltet erreicht man einen PHP-Error in tl_subcolumnsCallback.php Z 112.

Das Array $childs[$parent]['sc_childs'] ist null, da es keine $newChilds gibts. Ich hab den Fehler durch

<?php 
if(is_array($childs[$parent]['sc_childs'])) sort($childs[$parent]['sc_childs']); 

behoben, ohne ihn genauer untersucht zu haben.

felixpfeiffer commented 12 years ago

Sorry, habe das jetzt erst gesehen.

Du beziehst dich aber auf die Version im Repository, oder?

Das muss ich mal checken, da ist aber eh einiges im Argen beim Kopieren von Seiten.

Ich werde wohl auch im Laufe der nächsten Woche diese Version hier ins Repository schieben.

MacKP commented 12 years ago

Hallo felixpfeiffer, ich habe aktuell auch noch ein Problem beim kopieren.... ich versuch mich gerade an einem ganzen Seitenbaum ^^ Fehlermledung:

Warning: sort() expects parameter 1 to be array, null given in /var/www/vhosts/*/system/modules/subcolumns/tl_subcolumnsCallback.php on line 127

0 [internal function]: __error(2, 'sort() expects ...', '/var/www/vhosts...', 127, Array)

1 /var/www/vhosts/_/system/modules/subcolumns/tl_subcolumnsCallback.php(127): sort(NULL)

2 /var/www/vhosts/_/system/modules/subcolumns/tl_subcolumnsCallback.php(78): tl_subcolumnsCallback->copyCheck('3859')

3 /var/www/vhosts/_/system/drivers/DC_Table.php(884): tl_subcolumnsCallback->pageCheck(1310, Object(DC_Table))

4 /var/www/vhosts/_/system/modules/backend/Backend.php(287): DC_Table->copy()

5 /var/www/vhosts/_/contao/main.php(120): Backend->getBackendModule('page')

6 /var/www/vhosts/_/contao/main.php(230): Main->run()

7 {main}

....

Viele Grüße

MacKP commented 12 years ago

Eigentlich hätte ich ein neues Ticket für den Fehler von oben auf machen sollen.... Das kopieren von Seiten geht zumindest mit der aktuellen Version von hier. Zumindest werden keine Fehler ausgeworfen und alles kopierte ist vorhanden. Probleme gibt es dann nur, wenn man ein Spaltenset Startelement noch mal bearbeitet. Die ID wird dann neu generiert und die dazugehörigen Spaltenelemente werden neu angelegt in der kopie. Die jetzt nutzlosen Teile kann man dann entfernen... Ist also noch nicht ganz so rund, wie ich mir das wünschen würde ;-)

Viele Grüße

MacKP commented 12 years ago

OK, weiter geht die Geschichte ^^ Ich habe weiter getestet. Es ist jetzt so, das ich sogar ganze Seiten mit unterseiten kopieren kann... aber nicht immer. Einmal geht es ohne Probleme und bei dem nächsten Punkt wieder nicht. So langsam weiß ich auch nicht mehr weiter :(

Viele Grüße

felixpfeiffer commented 12 years ago

Hallo Marc,

Ich habe die ersten Probleme zumindest lokal schon behoben. Was noch fehlt ist das einzelene Kopieren eines Inhaltselements. Das wollte ich heute abend oder morgen mal angehen. Ich kann dir nachher aber auch erst mal die modigizierte Version schicken, die beim Kopieren von Seiten, Artikeln und mehreren Inhaltselementen die richtige Zuweisung trifft.

Gruß, Felix

MacKP commented 12 years ago

Ah, das klingt schon mal super. Wie ich auf twitter gelesen habe, hast du auch schon nen update gemacht hier? Dann lade ich mir das später mal runter und guck mir das dann an.

Viele Grüße

felixpfeiffer commented 12 years ago

Hi, ja, ganz vergessen das hier zu posten.

Mit der aktuellen Version sollte es ohne Probleme gehen.

Was noch nicht läuft ist das kopieren eines Trenn- oder Endelements und daraus ein neues Spaltenset basteln. Aber wer macht das schon?? ;-)

Gruß, Felix

MacKP commented 12 years ago

Oh, das klingt ja super. Kopieren von einzelnen Elementen hab ich bisher auch noch nicht gemacht XD

Ich frage mich nur gerade, was ich mit den Seiten etc. mache, die ich jetzt schon kopiert habe... ob ich die vllt besser nochmal komplett neu kopiere? Hast du da schon mal tests gemacht gehabt?

Viele Grüße