Nach Dashboard-Update keine Kommentare mehr

[Lux]

Hi,

ich habe am Sonntag das Dashboard-Plugin aktualisiert. Seit dem sehe ich "zukünftige Einträge" und von den Kommentaren nur noch die Überschriften "Zu bewilligende" und "Kommentar".

Da läuft vielleicht noch etwas schief.

Gruss

Dirk

[Timbalu]

Hmm...,

• Du benutzt die Version 0.6.4?
• Erinnerst du dich, von welcher Version du upgedated hast?
• Vorher haben wirklich "zukünftige Einträge" nicht, dafür aber "Zu bewilligende" und "Kommentar" Kommentar Einträge beide funktioniert?
• Du hast jetzt tatsächlich auch noch Einträge dafür?
• Läuft das ganze auf Mysql?
• Welche Serendipity Version hast du, genau?

Teste mal bitte, ob ein rename von $this->showElementcommentlist zu $this->showElementCommentlist einen Erfolg bringt (~425 und ~437).

Eventuell kann auch ein Leerzeichen vor AND nötig sein:

Code: Select all

$this->showElementCommentlist(" AND status = 'approved'", $lim));

dann müssten aber mindestens die Pending Kommentare angezeigt werden. Die haben das Leerzeichen.

[Lux]

Hmm...,

• Du benutzt die Version 0.6.4?
• Erinnerst du dich, von welcher Version du upgedated hast?
• Vorher haben wirklich "zukünftige Einträge" nicht, dafür aber "Zu bewilligende" und "Kommentar" Kommentar Einträge beide funktioniert?
• Du hast jetzt tatsächlich auch noch Einträge dafür?
• Läuft das ganze auf Mysql?
• Welche Serendipity Version hast du, genau?

Ich aktualisiere ein Mal wöchentlich alle Plugins.

Aktuell ist Version 0.6.5, ich weiss nicht von welcher Version ich aktualisiert habe, "Zu bewilligende" und "Kommentar" haben funktioniert, "zukünftige Einträge" gab es nicht (oder waren nicht aktiviert).

Ja, ich habe Unmengen an Kommentaren. Es ist Serendipity 1.6 auf MySQL in sechs verschiedenen Blogs.

Teste mal bitte, ob ein rename von $this->showElementcommentlist zu $this->showElementCommentlist einen Erfolg bringt (~425 und ~437).

Eventuell kann auch ein Leerzeichen vor AND nötig sein:

Code: Select all

$this->showElementCommentlist(" AND status = 'approved'", $lim));

dann müssten aber mindestens die Pending Kommentare angezeigt werden. Die haben das Leerzeichen.

Änderungen durchgeführt, aber ohne Änderung des Verhaltens.

Gruss

Dirk

[Timbalu]

Ja sorry, ich meinte auch die 0.6.5.
Bitte deaktiviere (wie im Beispiel unten) diese 4 Zeilen ca ~118 und lasse dir mit dem angehängten echo pre mal das Ergebnis ausgeben. Von Interesse ist nur, ob es überhaupt ein Ergebnis gibt. Wenn ja, hat eventuell bei deinen Update auf S9y 1.6 etwas nicht geklappt (in der serendipity_fetchComments()). Wird die Datei include/functions_comments.inc.php bei einem Integrity Check eventuell angemeckert?

Code: Select all

#        if (version_compare(substr($serendipity['version'], 0, 3), '1.6') >= 0) {
#            $comments = serendipity_fetchComments(null, $limit, 'co.id DESC', true, 'NORMAL', $where);
#        } else {
            $comments = serendipity_db_query("SELECT c.*, e.title FROM {$serendipity['dbPrefix']}comments c
LEFT JOIN {$serendipity['dbPrefix']}entries e ON (e.id = c.entry_id)
WHERE 1 = 1 " . $where
                                        . (!serendipity_checkPermission('adminEntriesMaintainOthers') ? 'AND e.authorid = ' . (int)$serendipity['authorid'] : '') . "
ORDER BY c.id DESC LIMIT $limit");
#        }
echo '<pre>';print_r($comments);echo '</pre>';

[Lux]

Hi Ian,

hier das Ergebnis:

Code: Select all

SELECT c.*, e.title FROM comments c
                                        LEFT JOIN entries e ON (e.id = c.entry_id)
                                        WHERE 1 = 1  AND status IN ('pending','confirm')
                                        ORDER BY c.id DESC LIMIT 5

 / Table 'lx_logbuch.comments' doesn't exist

bzw.

Code: Select all

SELECT c.*, e.title FROM comments c
                                        LEFT JOIN entries e ON (e.id = c.entry_id)
                                        WHERE 1 = 1  AND status = 'approved'
                                        ORDER BY c.id DESC LIMIT 5

 / Table 'lx_logbuch.comments' doesn't exist

Mir scheint da fehlt der Prefix, bei mir heisst die Tabelle "lx_logbuch.serendipity_comments".

Gruss

Dirk

[Timbalu]

HA! Recht hast du.
Es fehlte das wiedereinzufügende global

Code: Select all

    function showElementCommentlist($where, $limit) {
        global $serendipity;

Ist commited und müsste ab morgen Früh zur Verfügungs stehen.

[Lux]

Danke!

Dirk

[Lux]

Ist commited und müsste ab morgen Früh zur Verfügungs stehen.

Ist da etwas schief gelaufen?

Das aktualisierte Plugin ist noch nicht da.

Gruss

Dirk

[Timbalu]

Definitiv Ja! (Wobei ich mir nicht sicher bin, ob SourceForge nicht noch einmal 24 Std dranhängt...)

Der automatische XML sync funktioniert nicht, ...
Hoffentlich liest Garvin hier mit, sonst müssten wir ihm mal eine Mail schicken.

Garvin, möglicherweise was mit den headers $Id$ expansion... in den bayes cz lang files?

[garvinhicking]

Hi!

Ich weiß nicht, evtl hängt es noch mit der Reihenfolge der Git-Import, CSV commit, XML compile, XML uploadsache zusammen. Lasst uns noch einen Tag warten, ich denke grundsätzlich müsste es klappen.

Leider ist der ganze Vorgang eh schon komplex genug (siehe emerge.sh, emerge_spartacus.php), da habe ich gerade nicht so den Nerv das detailgenau nochmal zu prüfen.

Die emerge.sh wird bei mir auf meinem linuxserver zuhause täglich im 3:30 nachts ausgeführt.

Grüße,
Garvin

[Timbalu]

Tut ja normalerweise auch, heute Nacht allerdings nicht, d.h. es muss irgendwas die Ausführung behindern. Ich habe auch weder die tägliche Email darüber bekommen, noch ist es in der GitHub Commit list zu sehen.

[garvinhicking]

Hi!

Ah, in meinem cronjob-Mailscript steht ein "ssh_exchange_identification" bug. Irgendwo konnte dann wohl etwas nicht abgeholt werden, evtl gab es temporäre Netzprobleme bei Github oder sourceforge (eher letzteres).

Grüße,
Garvin

[garvinhicking]

Hi!

Der CVS commit scheint wohl erst heute nacht um 4:31 gelaufen zu sein, also vor dem 03:30 spartacus lauf, scheint mir. D.h. die XML-Syncsache mit Github wäre dann erst im nächsten Lauf gestartet.

Ich hab das jetzt einmalig manuell ausgeführt, mal schauen was passiert.

[Timbalu]

Ja - ist da!
(Merkwürdig ist der commit timestamp Wed Jan 11 03:31:23 2012 UTC aber doch)

Da jetzt aber seitdem die CVS commit statements immer so 'gitclone.sh autocommit' aussehen, sollte man vielleicht den Link ViewCVS unter http://spartacus.s9y.org/index.php?mode ... s_event_en eher auf gitHub setzen..., oder?

[garvinhicking]

Hi!

Das ist eine sehr gute Idee.

Habe ich geändert!

Grüße,
Garvin