Mein 1. Modul funktioniert überhaupt nicht

[m-werk]

Hallo Leute,

erstens: Ich bin kein Programmierer. Ich versuche aber anhand anderer Codes mir selbst einen zu erstellen.

zweitens: Ich habe das Modul 'login_count' bei mir installiert. Hier wird in der Datenbank unter tl_member -> lu_login_count der Login des Mitgliedes hochgezählt.

Nun wollte ich ein Modul schreiben, wo ich dann die Anzahl der Logins Im Mitgliederbereich gleich sehe (dort wo alle Mitglieder aufgelistet sind).

Es funktioniert aber nicht. Ich habe den Code und alles was noch dazugehört in einen Ordner gegeben. Nur wenn ich im BE auf Mitglieder klicke, erscheint NICHTS. Nur eine weiße Seite.

Anbei mein tl_member.php Code, den ich im Ordner list_login_count -> dca liegen habe:

<?php if (!defined('TL_ROOT')) die('You can not access this file directly!');
......
......
/**
 * Listing
 */
$GLOBALS['TL_DCA']['tl_member']['list']['label']['label_callback'] = array('tl_member_list_login_count','addList');


class tl_member_list_login_count extends Backend
{

    public function addList($row, $label)
    {
        //addon
        if(version_compare(VERSION . '.' . BUILD, '2.8.0', '<'))
        {
            // Code für Versionen < 2.8.0
            $listLoginCount = '';
        }
        else
        {
            // Code für Versionen ab 2.8.0
            $listLoginCount = '['.$GLOBALS['TL_LANG']['MSC']['tl_member_list_login_count']['list_login_count'].']';
        }
        $objUsers = $this->Database->prepare("SELECT tlm.id"
                                          . " FROM tl_member tlm, tl_session tls")
                                          . " WHERE tlm.id = tls.pid AND tlm.id=? AND tls.name=?")
                                    ->execute($row['id'],time()-300,'FE_USER_AUTH');
        //{
            if(version_compare(VERSION . '.' . BUILD, '2.7.6', '>'))
            {
                if ($row['lu_login_count'] >0 )
                {
                    $listLoginCount = '['.$GLOBALS['TL_LANG']['MSC']['tl_member_list_login_count']['list_login_count'].' '.$row['lu_login_count']).']';
                }
            } 
        //addon
        return sprintf($this->getTheme(), $label, $listLoginCount);
    }
}

Möge mich bitte jemand dabe unterstützen?

[BugBuster]

Deine Datei müsste tl_member_list_login_count.php heißen, sonst findet TL deine Class nicht.

[m-werk]

Oh, danke.

Jetzt sehe ich im Mitgliederbereich wieder die Mitglieder, ABER leider ohne dem gewünschten Ergebniss. Ich sehe den Inhalt der lu_login_count nicht. Es steht nichts drinn.

Ach übrigens, der Code bzw. die Codeteile habe ich von deinem Modul "Backend User Online".
Denn ich wollte auch die Infos so drinn stehen haben, wie du es mit diesem Modul gemacht hast. (Letzter Login)

[p7ha]

Eine weiße Seite heißt übrigens, dass ein fataler Fehler aufgetreten ist. Die Fehlermeldung kannst du sehen, wenn du im Backend bei "Einstellungen" "Fehler anzeigen" anklickst. Diese Fehlermeldung wird dich schon um einiges weiterbringen.
-> das ist glaub der einzigste Beitrag den ich bisher immer geben konnte :-)

[BugBuster]

Ach übrigens, der Code bzw. die Codeteile habe ich von deinem Modul "Backend User Online".
Denn ich wollte auch die Infos so drinn stehen haben, wie du es mit diesem Modul gemacht hast. (Letzter Login)

Habe ich erkannt
Das wird aber nicht gehen, derzeit.
Problem ist, das man keine Kette von Aufrufen im label_callback registrieren kann.
TL liest erst alle configs ein.
Ist nun Dein Modul alphabetisch nach meinem, dann überschreibst du mein Aufruf, du siehst dann Deine Zahl, aber meine Infos nicht mehr.

Ich habe schon seit einiger Zeit im Kopf das so zu ändern das erst das originale aufgerufen wird und dann selber die Änderungen eingearbeitet werden.
http://dev.typolight-forge.org/issues/show/510

Das war aber nur dazu gedacht, das ich nicht immer den originalen Code von Leo kopieren muss.
Eine funktionierende Verkettung ist damit noch nicht gelöst.

[m-werk]

Ok, kann ich das denn sonst irgendwie lösen? bzw. müsste ich das row in deinem Code einbauen. oder?

Nachtrag:
Ich habe bei dir aber auch gesehen, dass dein Dateiname tl_member.php heißt. Warum geht es bei dir?

[BugBuster]

weil ich so frech war und die original Methode überschrieben habe. (glaub ich)

[xtra]

Etwas aehnliches hab ich letztens erst beschrieben.
Wenn einer den Thread noch findet betr. der Kaskadierung dieser Funktionsaufrufe...
Einfach mal ein bisschen suchen.

Gruss
Chris

[BugBuster]

Hier, das isser
labelCallback-Überschreiben

Aber so ganz gerafft habe ich das damals nicht.

[andreas.schempp]

Deine Datei müsste tl_member_list_login_count.php heißen, sonst findet TL deine Class nicht.

Ehm, sorry, aber ich glaube das ist nicht korrekt. Die Datei muss schon tl_member heissen, damit sie das entsprechende DCA überschreibt.

Dann würde ich erst mal die ganzen Prüfungen für alte Contao-Versionen entfernen. Die brauchst du ja für dich nicht, oder? Dann verstehe ich nicht warum du $objUser abfragst, denn das wird ja gar nirgends benutzt?

Hier mal meine (vereinfachte) Variante, basierend auf deinem:

<?php if (!defined('TL_ROOT')) die('You can not access this file directly!'); 

/** 
 * Listing 
 */ 
$GLOBALS['TL_DCA']['tl_member']['list']['label']['label_callback'] = array('tl_member_list_login_count','addList'); 


class tl_member_list_login_count extends Backend 
{ 

    public function addList($row, $label) 
    {  
        return $label . '['.$GLOBALS['TL_LANG']['MSC']['tl_member_list_login_count']['list_login_count'].' '.$row['lu_login_count'].']';
    } 
}

Ob die Sprachvariablen so existiert und wie sie aussieht, weis ich natürlich nicht. Die weisse Seite kommt überigens daher, weil in der 7letzten Zeile nach $row['lu_login_count'] eine schliessende Klammer ist, welche nirgends geöffnet wird.

[m-werk]

Hallo, eigentlich hat sich das Modul für mich schon fast erledigt.

Es gibt eine neue Version dieses Modules login_count. Wenn man beim Mitglied auf Details klickt, findet man nun die Info "Anzahl der Logins".

Es wäre jetzt nur schön, dies gleich bei der Liste anzuzeigen, aber vorerst hilft dies schon weiter.

Danke aber trotzdem für die Infos und Hilfen