Probleme beim Artikel speichern seit 1.7.7

[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.

[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!

[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.)

[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

[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.

[garvinhicking]

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

[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

[garvinhicking]

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

[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.