CronJob ruft Methode nicht auf

Hallo Forum,

komme gerade bei einem Problem nicht weiter... Ich habe ein neues Modul erstellt, welches momentan eine Methode enthält. Wenn diese aufgerufen wird, wird eine FTP Verbindung aufgebaut und eine Datei heruntergeladen. Funktioniert soweit ganz gut. Wenn ich jedoch class blub extends Backend mache, kann ich kein Objekt erzeugen bzw. es erscheint keine Ausgabe, kann mir vielleicht jemand erklären wieso das so ist? Wenn ich es ohne das extends mache und mir ein Objekt bastle, wird die Datei korrekt heruntergeladen, wenn ich das extends wieder dranpacke und die Klasse / Methode per CronJob aufrufen lassen will, passiert aber nichts. Das error Log sagt aber auch nichts, daher gehe ich schon davon auss, dass die Methode eigentlich aufgerufen werden kann...

Weiß da vielleicht jemand weiter?

Sorry jetzt hab ich es kapiert... Er hat dich Datei ins Hauptverzeichnis gepackt, ich dachte aber die würde irgendwo im Modulordner rauskommen, daher funktionierts nun :-)

Mhh aber was ich jetzt noch lösen muss... Wenn die Klasse Backend extended, kann ich dann nicht selber in der Datei kurz ein Objekt erzeugen und die Methoden aufrufen?

Die __construct()-Methode der Backend Klasse ist ev. protected. Du müsstest diese Funktion also in deiner Klasse hinzufügen und parent::__construct() aufrufen.

Hallo Andreas,

hab das jetzt so:

PHP-Code:

---

public function __construct() {
        parent::__construct();
    } 


---

innerhalb meiner klasse die Backend extendend. Aber wenn ich jetzt ein Objekt erzeuge und eine Methode aufrufe, passiert halt gar nichts... Wie gesagt, extends weg klappt alles wunderbar aber ich muss eben auch ein paar Dinger vom System nutzen (Datenbank etc.)

Hast du mal geschaut welche Fehlermeldung(en) ausgegeben werden?

Im error.log steht nichts, kann auch keine Fehler produzieren und die Ausgabe bleibt immer weiß... Okay vielleicht sollte ich die Sache anders angehen. Muss die Klasse überhaupt Backend extendend? Meine Klasse selbst führt ein paar SQL Statements aus und hat ein import auf eine selbst geschriebene Lib ... Aber ansonsten relativ wenig spektakuläres und soll eben per cronjob ausgeführt werden.

Du solltest immer von etwas erben. Vielleicht kannst du uns ein Codebeispiel liefern?

Hallo Andreas,

anbei mal das Script von mir:

PHP-Code:

---

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

require_once('functions.php');


class blub {
    
    public function bla() {
        return 'blub';
    }
    
}

// Gibt blub zurück
$obj = new blub();
echo $obj->bla();


// gibt nix zurück
$obj2 = new alka;
echo $obj2->alkaGetUpdatedFiles();


class alka extends Backend {
//class alka {
    
    public function __construct() {
        parent::__construct();
        $this->Import('Logtabs');
    }
    
    /* Via FTP die aktuellen Daten von Alka laden */
    public function alkaGetUpdatedFiles() {
        
        // Variablen
        $remote_file    = 'plalkawws.txt';
        $remote_path    = 'pl';
        $local_file        = 'alka_update.txt';
        $local_path        = 'files/import/alka';
        
        // Verbindung herstellen
        $connection = ftp_connect("-");
        
        // Am FTP Server anmelden
        if (@ftp_login($connection, "-", "-")) {
            //log_message('FTP-Verbindung erfolgreich', 'alka.log');
            $this->Logtabs->addentry("FTP-Verbindung erfolgreich hergestellt", "importexport_alka", 'complete');
        }
        
        else {
            //log_message('FTP-Verbindung nicht erfolgreich', 'alka.log');
            $this->Logtabs->addentry("FTP-Verbindung konnte nicht hergestellt. Benutzername und Passwort überprüfen.", "importexport_alka", 'error');
        }
        
        // Verzeichnis wechseln
        if(ftp_chdir($connection, $remote_path)) {
            //log_message('Verzeichnis erfolgreich gewechselt', 'alka.log');
            $this->Logtabs->addentry("FTP Verzeichniswechsel erfolgreich.", "importexport_alka", 'complete');
        }
        
        else {
            //log_message('Verzeichnis nicht erfolgreich gewechselt', 'alka.log');
            $this->Logtabs->addentry("Konnte FTP Verzeichnis nicht wechseln.", "importexport_alka", 'error');
        }
        
        // Datei löschen
        unlink($local_file);
        
        // Datei erstellen
        $handle = fopen($local_file, 'w');
        
        // Datei herunterladen und Inhalt speichern
        if(ftp_fget($connection, $handle, $remote_file, FTP_ASCII,0)) {
            //log_message('Alka Datei erfolgreich erzeugt', 'alka.log');
            $this->Logtabs->addentry("Lokale ALKA Datei erfolgreich erzeugt.", "importexport_alka", 'complete');
        }
        
        else {
            $this->Logtabs->addentry("Lokale ALKA Datei konnte nicht erzeugt werden.", "importexport_alka", 'error');
        }
        
        // FTP Verbindung schließen
        ftp_close($connection);
        
        // Datei verschieben
        // Kommt noch
        
        return 'test';

    }

    public function alkaImportUpdatedFiles() {
        // Product Array
        $products = array();
        
        
        // Datei öffnen
        $file = @fopen("alka_update.txt", "r");
        
        if($file) {
            $this->Logtabs->addentry("Lokale ALKA Datei erfolgreich geöffnet.", "importexport_alka", 'complete');
        }
        
        else {
            $this->Logtabs->addentry("Lokale ALKA Datei kann nicht geöffnet werden.", "importexport_alka", 'error');
        }

        // Datei durchlaufen und product Array füllen
        if ($file)
        {
            while(!feof($file))
            {
                $zeile = fgets($file,1000);
                   $products[] = explode("\t", $zeile); // TAB getrennt
                //$products[] = explode(";", $zeile); // HCBS
            }
            
            // Over and out
            fclose($file);
        }
        
        // Datensätze in der Wawi aktualisieren
        foreach($products as $key => $val) {
            if(count($val) < 24) continue;
            if($val[9] == 0)     continue;
                                
            // Build Price Array
            if($val[17] != '' && $val[18] != '' && $val[19] == '' && $val[20] == '')
            {
                //$arrGroup = array(getSpNumber($db,$val[18]),"-1");
                $arrValue = array(changeCtoP($val[12]),changeCtoP($val[17]));
            }
            
            elseif($val[17] != '' && $val[18] != '' && $val[19] != '' && $val[20] != '')
            {
                //$arrGroup = array(getSpNumber($db,$val[18]),getSpNumber($db,$val[20]),"-1");
                $arrValue = array(changeCtoP($val[12]),changeCtoP($val[17]),changeCtoP($val[19]));
            }
            
            else
            {
                $arrGroup = array("-1");
                $arrValue = array(changeCtoP($val[12]));
            }
                    
    
            // Build Price Cost Array
            if($val[13] != '' && $val[14] != '' && $val[15] == '' && $val[16] == '')
            {
                //$arrGroupCost = array(getSpNumber($db,$val[14]),"-1");
                $arrValueCost = array(changeCtoP($val[11]),changeCtoP($val[13]));
            }

            elseif($val[13] != '' && $val[14] != '' && $val[15] != '' && $val[16] != '')
            {
                //$arrGroupCost = array(getSpNumber($db,$val[14]),getSpNumber($db,$val[16]),"-1");
                $arrValueCost = array(changeCtoP($val[11]),changeCtoP($val[13]),changeCtoP($val[15]));
            }

            else
            {
                $arrGroupCost = array("-1");
                $arrValueCost = array(changeCtoP($val[11]));
            }
                    
            // Verkaufspreise erzeugen
            $price = array("group" => $arrGroup, "value" => $arrValue);
            $price = serialize($price);
            
            // Einkaufspreise erzeugen
            $priceCost = array("group" => $arrGroupCost, "value" => $arrValueCost);
            $priceCost = serialize($priceCost);
            //$priceCost = array("group" => array("-1"), "value" => array($val[11]));
            
            
                    
            $title = $val[6].' '.$val[7];
            $title = utf8_encode($title);
            
            
            //$checkSql = $this->Database->prepare("SELECT id, alka_nummer FROM tl_warenwirtschaft_products WHERE alka_nummer=?")->limit(1)->execute($val[0]);
            
            if($checkQuery->numRows > 0) {
                
                $this->Database->prepare("UPDATE tl_warenwirtschaft_products SET title=?, singlePrice=?, priceCost=?, alka_ean=? WHERE alka_nummer=?")->limit(1)->execute(addslashes($title), $price, $priceCost, addslashes($val[1]), $val[0]);
                
                //$updateSql = "UPDATE tl_warenwirtschaft_products SET title = '".addslashes($title)."', singlePrice = '".$price."', priceCost = '".$priceCost."', alka_ean = '".addslashes($val[1])."' WHERE alka_nummer = '".$val[0]."' LIMIT 1;";
                //$db->dbQuery($updateSql);
                //echo $val[0].' upgedated<br />';                    
            }
            
            else {
                //    echo $val[0].' nicht upgedated<br />';
            }
        }
    }
}


?>

---

Das kann nicht funktionieren, weil du die system/initizalize.php nicht eingebunden hast. Die functions.php muss du nicht einbinden (oder ist die von dir?).

Hallo Andreas,

okay, das habe ich bislang aber noch bei keinem Modul gesehen... Ich dachte wenn ich das Backend extende wird das auch geladen... Die functions.php ist von mir, aber wenn ich jetzt noch ein require_once require_once('../../initialize.php'); mache kommt:

Fatal error: Class 'alka' not found in /usr/local/zend/apache2/htdocs/spatzel/www/system/modules/importexport/alka.php on line 8

Deines scheint mir auch kein Modul zu sein. Ein Modul erbt von der Klasse "Module" oder "BackendModule" und wird über's System eingebunden.

Welche Funktion hat deine Klasse, die Datei wird ja direkt aufgerufen. Ev. ein CronJob?

Hallo Andreas,

okay vllt. habe ich mich falsch ausgedrückt was ich eigentlich machen möchte :-) Ich hab zwar ein Modulordner innerhalb von Modules erstellt, aber die Datei die ich gepostet hab hat keine Logik die dann z.B. über Contao (Frontend Module, Backend Module) eingebunden werden soll, das ist eigentlich eine autarke Datei / Klasse, welche aber zumindest die Datenbankfunktionalität von Contao nutzen soll, mehr aber nicht (und die Datei wird per CronJob aufgerufen was aber funktioniert d.h. das macht contao korrekt jede Stunde). Daher ja die Frage ob ich überhaupt ein extends Backend machen muss oder ob ich irgendwie anders dann auf die DB Funktionalität zurückgreifen kann

Wenn du den CronJob verwendest, dann brauchst du doch in der Datei keine run()-Funktion mehr auszuführen. Wie hast du den CronJob angelegt?

Bezüglich der Fehlermeldung, der Name der Datei muss mit dem Klassennamen identisch sein! Nur eine Klasse pro Datei.

Hallo Andreas,

von der Logik an sich funktioniert doch alles :-) Wenn der Cronjob die Datei 1 mal Pro Stunde aufruft, läuft alles korrekt durch. Das Problem ist aber, dass ich natürlich gewisse Parts selbst testen muss und daher kurz ein Objekt erzeuge um zu schauen, ob alles passt, das ist ja gerade meine Frage.
(Die zwei Klassen oben waren einfach exemplarisch denk dir die blub weg, Datei ist alka.php die Klasse alka). Welche run() funktion meinst du ich sehe da keine.
Cronjob rufe ich via config/config.php in meinem Modul importexport auf

PHP-Code:

---

$GLOBALS['TL_CRON']['hourly'][]  = array('alka', 'alkaGetUpdatedFiles');
$GLOBALS['TL_CRON']['hourly'][]  = array('alka', 'alkaImportUpdatedFiles'); 


---

Das funktioniert wie gesagt auch

Das hier brauchst du nicht:

PHP-Code:

---

$obj2 = new alka; 
echo $obj2->alkaGetUpdatedFiles(); 


---

Hi Andreas,

klaro brauche ich das nicht weil ja der CronJob jede Stunde die angegebenen Methoden aufruft... Nur wie kann ich denn ohne den CronJob die Methoden direkt aufrufen? Doch eigentlich nur indem ich mir ein Objekt erstelle? Wie gesagt mir geht's nicht darum, wie ich schaffe dass der CronJob die Methoden aufruft sondern wie ich (um zu testen dass alles passt) mir ein Objekt erstellen kann um die entsprechenden Methoden auszuführen :-)

Jetzt versteh ich. Nun, beim CronJob brauchst du auch die initialize.php nicht. Ich würde dir empfehlen, in der localconfig.php den Timestamp für den CronJob anzupassen und dann einfach die cron.php aufzurufen.

Hallo Andreas,

okay danke dir... Aber es muss doch möglich sein ein Objekt zu erzeugen auch wenn die Klasse Backend extended? Ich meine lasse ich das weg kann ich mir das Objekt ja auch basteln... Immer den Timestamp runtersetzen ist jetzt nicht gerade die idealste Lösung :-)

Ja, dann brauchst du halt die initialize.php

aber wenn ich die einbinde bekomm ich ja class alka not found obwohl ich sonst nichts ändere

vermutlich müsstest du die instanziierungs-Zeilen ganz unten einfügen, denn oberhalb der Klasse ist diese ja noch nicht definiert.

Dann verschwindet zwar der error, dass die Klasse nicht vorhanden wäre, aber es erfolgt wieder eine leere Seite als Ausgabe

Jetzt wirds schwierig zu helfen

Was ist die Klasse Logtabs? Versuchs mit einem echo ganz zu beginn der Funktion und/oder am Ende der Datei, ob überhaupt was gemacht wird. Ein normales "PHP-Debugging" mit var_dump halt ;)

Hallo Andreas,

also ich glaube das ist ein allgemeines Verständnisproblem von mir vielleicht ist es ja gar nicht so angedacht was ich eigentlich machen möchte... Mal ein ganz simples Beispiel:

PHP-Code:

---

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

class test  {
    public function test () {
        return 'Hello World';
    }
}



$obj = new test();
echo $obj->test();


?>

---

Rufe ich die Datei auf erhalte ich "Hello World".

PHP-Code:

---

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

class test extends Backend {
    public function test () {
        return 'Hello World';
    }
}



$obj = new test();
echo $obj->test();


?>

---

Rufe ich die Datei auf, kommt ne leere Seite selbst wenn ich noch die initialize.php includiere ...

Die Fehlermeldung scheint dir nicht angezeigt zu werden. Hast du die Fehleranzeige im System aktiviert?

Meinst du das

PHP-Code:

---

$GLOBALS['TL_CONFIG']['displayErrors']  = true; 


---

oder gibt's da noch eine andere Option?

Ja das mein ich. Wenn das nichts nützt, musst du vermutlich manuell die Fehleranzeige-Routine von Contao temporär ausschalten/entfernen.