Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Seiten kopieren geht nicht #4

Closed
psi-4ward opened this issue Jul 2, 2012 · 8 comments
Closed

Seiten kopieren geht nicht #4

psi-4ward opened this issue Jul 2, 2012 · 8 comments

Comments

@psi-4ward
Copy link

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
Copy link
Owner

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
Copy link

MacKP commented Oct 19, 2012

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
Copy link

MacKP commented Oct 19, 2012

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
Copy link

MacKP commented Oct 19, 2012

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
Copy link
Owner

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
Copy link

MacKP commented Oct 22, 2012

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
Copy link
Owner

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
Copy link

MacKP commented Oct 22, 2012

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

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

No branches or pull requests

3 participants