Page 1 of 1

Nach Dashboard-Update keine Kommentare mehr

Posted: Mon Jan 09, 2012 1:01 pm
by 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

Re: Nach Dashboard-Update keine Kommentare mehr

Posted: Mon Jan 09, 2012 2:37 pm
by 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.

Re: Nach Dashboard-Update keine Kommentare mehr

Posted: Mon Jan 09, 2012 3:30 pm
by Lux
Timbalu wrote: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.
Timbalu wrote: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

Re: Nach Dashboard-Update keine Kommentare mehr

Posted: Mon Jan 09, 2012 3:59 pm
by 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>';

Re: Nach Dashboard-Update keine Kommentare mehr

Posted: Mon Jan 09, 2012 7:36 pm
by 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

Re: Nach Dashboard-Update keine Kommentare mehr

Posted: Mon Jan 09, 2012 7:48 pm
by 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.

Re: Nach Dashboard-Update keine Kommentare mehr

Posted: Mon Jan 09, 2012 9:48 pm
by Lux
Danke!

Dirk

Re: Nach Dashboard-Update keine Kommentare mehr

Posted: Wed Jan 11, 2012 5:09 am
by Lux
Timbalu wrote: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

Re: Nach Dashboard-Update keine Kommentare mehr

Posted: Wed Jan 11, 2012 10:20 am
by 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?

Re: Nach Dashboard-Update keine Kommentare mehr

Posted: Wed Jan 11, 2012 12:24 pm
by 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

Re: Nach Dashboard-Update keine Kommentare mehr

Posted: Wed Jan 11, 2012 12:53 pm
by 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.

Re: Nach Dashboard-Update keine Kommentare mehr

Posted: Wed Jan 11, 2012 1:00 pm
by 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

Re: Nach Dashboard-Update keine Kommentare mehr

Posted: Wed Jan 11, 2012 1:05 pm
by 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.

Re: Nach Dashboard-Update keine Kommentare mehr

Posted: Wed Jan 11, 2012 8:10 pm
by 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?

Re: Nach Dashboard-Update keine Kommentare mehr

Posted: Thu Jan 12, 2012 11:06 am
by garvinhicking
Hi!

Das ist eine sehr gute Idee. :-)

Habe ich geändert!

Grüße,
Garvin