Probleme beim Artikel speichern seit 1.7.7

Hier können Probleme und alles andere in Deutscher Sprache gelöst werden.
Post Reply
MarioH
Regular
Posts: 235
Joined: Mon Jul 20, 2009 10:53 pm
Contact:

Probleme beim Artikel speichern seit 1.7.7

Post by MarioH » Fri Feb 07, 2014 10:42 pm

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.

User avatar
Timbalu
Regular
Posts: 4598
Joined: Sun May 02, 2004 3:04 pm

Re: Probleme beim Artikel speichern seit 1.7.7

Post by Timbalu » Sat Feb 08, 2014 9:30 am

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!
Regards,
Ian

Serendipity Styx Edition and additional_plugins @ https://ophian.github.io/ @ https://github.com/ophian

User avatar
Timbalu
Regular
Posts: 4598
Joined: Sun May 02, 2004 3:04 pm

Re: Probleme beim Artikel speichern seit 1.7.7

Post by Timbalu » Sat Feb 08, 2014 10:27 am

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.)
Regards,
Ian

Serendipity Styx Edition and additional_plugins @ https://ophian.github.io/ @ https://github.com/ophian

User avatar
garvinhicking
Core Developer
Posts: 30020
Joined: Tue Sep 16, 2003 9:45 pm
Location: Cologne, Germany
Contact:

Re: Probleme beim Artikel speichern seit 1.7.7

Post by garvinhicking » Sat Feb 08, 2014 4:07 pm

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
# Garvin Hicking (s9y Developer)
# Did I help you? Consider making me happy: http://wishes.garv.in/
# or use my PayPal account "paypal {at} supergarv (dot) de"
# My "other" hobby: http://flickr.garv.in/

User avatar
Timbalu
Regular
Posts: 4598
Joined: Sun May 02, 2004 3:04 pm

Re: Probleme beim Artikel speichern seit 1.7.7

Post by Timbalu » Sat Feb 08, 2014 4:11 pm

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.
Regards,
Ian

Serendipity Styx Edition and additional_plugins @ https://ophian.github.io/ @ https://github.com/ophian

User avatar
garvinhicking
Core Developer
Posts: 30020
Joined: Tue Sep 16, 2003 9:45 pm
Location: Cologne, Germany
Contact:

Re: Probleme beim Artikel speichern seit 1.7.7

Post by garvinhicking » Sat Feb 08, 2014 4:14 pm

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
# Garvin Hicking (s9y Developer)
# Did I help you? Consider making me happy: http://wishes.garv.in/
# or use my PayPal account "paypal {at} supergarv (dot) de"
# My "other" hobby: http://flickr.garv.in/

User avatar
Timbalu
Regular
Posts: 4598
Joined: Sun May 02, 2004 3:04 pm

Re: Probleme beim Artikel speichern seit 1.7.7

Post by Timbalu » Sat Feb 08, 2014 4:39 pm

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
Regards,
Ian

Serendipity Styx Edition and additional_plugins @ https://ophian.github.io/ @ https://github.com/ophian

User avatar
garvinhicking
Core Developer
Posts: 30020
Joined: Tue Sep 16, 2003 9:45 pm
Location: Cologne, Germany
Contact:

Re: Probleme beim Artikel speichern seit 1.7.7

Post by garvinhicking » Sat Feb 08, 2014 6:14 pm

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
# Garvin Hicking (s9y Developer)
# Did I help you? Consider making me happy: http://wishes.garv.in/
# or use my PayPal account "paypal {at} supergarv (dot) de"
# My "other" hobby: http://flickr.garv.in/

MarioH
Regular
Posts: 235
Joined: Mon Jul 20, 2009 10:53 pm
Contact:

Re: Probleme beim Artikel speichern seit 1.7.7

Post by MarioH » Sat Feb 08, 2014 9:26 pm

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

Post Reply