Page 1 of 1
Probleme beim Artikel speichern seit 1.7.7
Posted: Fri Feb 07, 2014 10:42 pm
by MarioH
Hallo Zusammen!
Ich habe gestern von 1.7.4 auf 1.7.7 upgedatet. Seitdem habe ich folgendes Problem. Wenn ich einen Artikel schreibe und vor dem Speichern die Vorschau anzeige, wird der Artikel danach nicht gespeichert (auch nicht als Entwurf). Er zeigt zwar an, dass der Artikel gespeichert wurde, Trackbacks erstellt usw, der Artikel ist aber danach nicht da. Auch die Ankündigung bei Twitter über das Microblogging-Plugin funktioniert dann nicht.
Speichere ich einmal als Entwurf ab, ändere dann etwas, sehe die Vorschau an und speichere dann, werden die Änderungen gespeichert. Auch eine direkte Veröffentlichung ohne vorherige Vorschau funktioniert.
Ich benutze den CK-Editor in der aktuellen Version.
Re: Probleme beim Artikel speichern seit 1.7.7
Posted: Sat Feb 08, 2014 9:30 am
by Timbalu
Ja das kann ich bestätigen!
Das Entry endet als /archives/0-test-entry.html und ist auch nicht in der DB enthalten.
Und das hat definitiv genau so in 1.7.4 funktioniert, ja? (Mein Schnell-Test sagt "ja", aber ohne CKE.)
We will have to investigate this for a quick 1.7.8, I presume!
Re: Probleme beim Artikel speichern seit 1.7.7
Posted: Sat Feb 08, 2014 10:27 am
by Timbalu
Ein Schnellfix - solange - wäre diese Zeile
https://github.com/s9y/Serendipity/blob ... c.php#L138
so zu ändern:
Code: Select all
$hidden .= ' <input type="hidden" id="entryid" name="serendipity[id]" value="' . ((isset($entry['id']) && (int)$entry['id'] > 0) ? (int)$entry['id'] : '') . '" />' . $n;
Ich frage mich, Garvin, wo entry ID da plötzlich eine "0" her bekommt...? Aber das ist der Kasus! (
Am neuen typecast liegt es IMHO nicht.)
Re: Probleme beim Artikel speichern seit 1.7.7
Posted: Sat Feb 08, 2014 4:07 pm
by garvinhicking
Hi!
Das kann gut sein dass vorher da ohne das (int) casting immer ein leerstring erschien. Timbalu, dein vorschlag da einen > 0 Vergleich reinzusetzen, scheint mir sinnvoll. Bitte committe das gerne in die 1.7 und 2.0 trees und dann können wir eine 1.7.8 nachschieben, wenn Mario bestätigen kann das es so klappt?
Grüße,
Garvin
Re: Probleme beim Artikel speichern seit 1.7.7
Posted: Sat Feb 08, 2014 4:11 pm
by Timbalu
Ich habe das heute morgen aber ohne den cast ausprobiert, wenn ich mich recht erinnere. Mit dem selben Ergebnis (0). Von irgendwo muss die ja (durch die sec fixes?) generiert werden.
Re: Probleme beim Artikel speichern seit 1.7.7
Posted: Sat Feb 08, 2014 4:14 pm
by garvinhicking
Timbalu wrote:Ich habe das heute morgen aber ohne den cast ausprobiert, wenn ich mich recht erinnere. Mit dem selben Ergebnis (0). Von irgendwo muss die ja (durch die sec fixes?) generiert werden.
Verstehe nicht was du meinst?
Die 0 kommt daher, wenn $entry['id'] zwar gesetzt (isset) ist, aber ein leerer string drinsteht. Früher gabs da kein (int) vorher, d.h. ein leeres $entry['id'] hatte einen Leerstring erzeugt, und jetzt kommt da ne "0" rein, die wenn man es submittet dann immer auch ein isset() = true erzeugt. Aus dem Grund vermutete ich, dass dein vorgeschlagener Fix das Problem auch beheben sollte.
Grüße,
Garvin
Re: Probleme beim Artikel speichern seit 1.7.7
Posted: Sat Feb 08, 2014 4:39 pm
by Timbalu
Hast recht! Commited. Aber kein upport nötig, oder?
Bitte testen und Bescheid geben. File zum kopieren:
https://raw.github.com/s9y/Serendipity/ ... in.inc.php
Danke
Re: Probleme beim Artikel speichern seit 1.7.7
Posted: Sat Feb 08, 2014 6:14 pm
by garvinhicking
Timbalu wrote:Hast recht! Commited. Aber kein upport nötig, oder?
Ah, weil das da über Smarty läuft und dort escape genutzt wird? Ja stimmt.
Mario, wenn Dein Feedback positiv ist zum Fix würde ich die 1.7.8 dann herausgeben (sofern nicht noch was anderes dazukommt)
Grüße,
Garvin
Re: Probleme beim Artikel speichern seit 1.7.7
Posted: Sat Feb 08, 2014 9:26 pm
by MarioH
Ich habe den Patch soeben getestet.
Eintrag schreiben, Vorschau, als Entwurf speichern, funktioniert.
Eintrag schreiben, Vorschau, speichern zum Veröffentlichen, funktioniert auch.
Also soweit alles OK.
