Isotope getProduct() liefert null

[Ernestopheles]

Im Integrationstest meiner extension habe ich den folgenden Testcase. Das Item ist mit dem Produkt id 88 verknüpft und $objItem wird noch richtig geholt. Dass das Produkt nicht geholt wird, liegt vielleicht an meiner Deklaration ( $GLOBALS['TL_MODELS']['tl_iso_product'] = Product::class; )? Wenn ich Standard::class verwende statt Product::class, ist es eigentlich richtiger, oder? Weil die Objekte ja nur von Standard gebildet werden können. Aber auch das führt leider zum selben Fehler.

<?php
declare(strict_types=1);

namespace Nahati\ContaoIsotopeStockBundle\Tests\Integration\EventListener;

use Contao\CoreBundle\Framework\ContaoFramework;

use Contao\CoreBundle\Framework\Adapter;

use Contao\Database;

use Contao\TestCase\FunctionalTestCase;
use Isotope\Model\Product;
use Isotope\Model\Product\Standard;
use Isotope\Model\ProductCollection;
use Isotope\Model\ProductCollection\Cart;
use Isotope\Model\ProductCollectionItem;
use Nahati\ContaoIsotopeStockBundle\EventListener\UpdateItemInCollectionListener;

/**
 * Test the UpdateItemInCollectionListener class.
 */
class UpdateItemInCollectionListenerTest extends FunctionalTestCase
{
    private ContaoFramework $framework;
    private Adapter $databaseAdapter;

    private ProductCollectionItem $objItem;
    private Cart $objCart;
    private mixed $arrSet;
    private Standard $objProduct;

    private mixed $return;


    // In constructor we initialize the neccessary environment once for all tests
    public function __construct()
    {
        parent::__construct();

        static::bootKernel();

        $this->framework = static::getContainer()->get('contao.framework');
        $this->framework->initialize();

        /** @varAdapter<Database> $databaseAdapter */
        $this->databaseAdapter = $this->framework->getAdapter(Database::class);

        // Drop all tables
        foreach ($this->databaseAdapter->getInstance()->listTables() as $table) {
            $sql = 'DROP TABLE IF EXISTS ' . $table;
            $this->databaseAdapter->getInstance()->execute($sql);
        };

        // Create tables and insert data
        $this->loadFixture('zfr00_cleverquir-4-Produkte-Im Warenkorb.sql');

        // These assignments link the tables with the model classes. Now we can use the model classes to access and manipulate the data in the tables.
        $GLOBALS['TL_MODELS']['tl_iso_product'] = Product::class;
        $GLOBALS['TL_MODELS']['tl_iso_product_collection'] = ProductCollection::class;
        $GLOBALS['TL_MODELS']['tl_iso_product_collection_item'] = ProductCollectionItem::class;
    }

    private function loadFixture(string $fileName): void
    {
        $sql = file_get_contents(__DIR__ . '/..' . '/Fixtures/' . $fileName);

        $this->databaseAdapter->getInstance()->execute($sql);
        // $this->connection->executeStatement($sql);
    }

    /**
     * setup() is called for each Testcase and contains the basic setup for the tests.
     * Override this method if you need to change the basic setup.
     */
    public function setUp(): void
    {
        parent::setUp();

        // Instantiate a Cart object with id '263'
        $this->objCart = Cart::findByPk('263');
    }

    public function testUpdateItemInCollectionListenerReturnsUnchangedQuantityWhenProductIsNotAVariantAndProductHasUnlimitedQuantity(): void
    {
        // Instantiate the Item with id '3105' of this Cart
        $this->objItem = ProductCollectionItem::findByPk('3105');

        /** @varStandard|null $objProduct */
        $objProduct = $this->objItem->getProduct() ?? null;

        // ...

    }
}

Die sourcen finden sich auf github: https://github.com/nahati/contao-iso...tree/2.0.1-dev

[Ernestopheles]

Ich bin begeistert über so viel Hilfe

[Spooky]

Da wirst du wohl deine Erwartungshaltung anpassen müssen.

[Ernestopheles]

Mein Engagement wird weder gewürdigt noch gibt es konkrete Unterstützung. Schade um die eingesetzte Zeit und Energie und den Ärger den ich mit überheblichen Äusserungen hatte

Damit habt ihr mich erfolgreich vertrieben.

[Spooky]

Niemand kann sich erwarten für jedes Problem sofortige, kostenlose Unterstützung zu bekommen. Es wird nicht jeder immer Zeit und Lust dafür haben und es kann auch schlicht und einfach sein, dass niemand die Lösung zu deinem Problem weiß. Ich finde es auch sehr befremdlich, dass du dir das überhaupt erwartest.

Wenn du Zeitnahe Unterstützung brauchst, dann ist es wohl besser, wenn du jemanden dafür bezahlst.

[Ernestopheles]

Das wäre ja traurig wenn niemand seine extension mit Integrationstests testet... ich denke das machen einige und das es für die auch einfach wäre mir zu helfen.


Meine Arbeit für Contao war ehrenamtlich. Die Erwartung einer finanziellen Zuwendung habe ich bisher - passend zu meiner quasi Null Einkommenslage - mit einer Contao Association Mitgliedschaft erfüllt.

Mit euren Erwartungen vergrault ihr sicherlich so manchen, aber das scheint euch ja gleich zu sein. Hauptsache, ihr lobt euch weiter gegenseitig hoch in den Himmel...

[Spooky]

Das wäre ja traurig wenn niemand seine extension mit Integrationstests testet... ich denke das machen einige und das es für die auch einfach wäre mir zu helfen.

Was hat das mit den bisherigen Kommentaren zu tun?

Die Erwartung einer finanziellen Zuwendung habe ich bisher - passend zu meiner quasi Null Einkommenslage - mit einer Contao Association Mitgliedschaft erfüllt.

Niemand ist verpflichtet dir im Forum Hilfe zu geben - oder verpflichtet überhaupt zu Wissen, wie man dir Helfen kann.

Mit euren Erwartungen

Wenn meinst du mit "euren" und von welchen Erwartungen sprichst du? In den bisherigen Kommentaren geht es ja um deine fehlgeleiteten Erwartungen.

[Ernestopheles]

Wie ich sehe, kommen genau die Reaktionen wie so oft von dir. Damit kann ich nichts anfangen und ich halte es auch für sinnlos weiter zu reden.

[mlweb]

Was meinst Du wieviel ehrenamtliche Arbeit im Core und in einigen Erweiterungen steckt, die viele von uns nutzen - wahrscheinlich auch Du?
Was meinst wieviel Zeit ehrenamtlich aufgebracht wird, um im Forum zu unterstützen (ganz besonders auch von @Spooky).
Was meinst Du wieviel Zeit ehrenamtlich von all denen aufgebracht wird, die sich im Hintergrund um Sponsoren bemühen, Events organisieren etc.
Keiner von denen verweist so häufig wie Du (Slack und Forum) auf seine eigene ehrenamtliche Tätigkeit, die er nicht gewürdigt sieht.

[Spooky]

Wie ich sehe, kommen genau die Reaktionen wie so oft von dir.

Was genau meinst du damit?

[derRenner]

So - es ist heiss - die Temperaturen sind hoch - die Lage ist angespannt
wir beruhigen uns nun wieder und lehnen uns mal zurück ... vielleicht ein Kaffee, n Eistee oder ein Bierchen gefällig?


Ich würde sagen, @Ernestopheles Thema aus #1 bleibt mal als unerledigt offen und wartet weiterhin auf mögliche Antwort(en).

@Ernestopheles auch wenn's mal etwas länger dauert - deine Anliegen werden nicht übersehen/überlesen.

[zoglo]

Im Integrationstest meiner extension habe ich den folgenden Testcase...
Die sourcen finden sich auf github: https://github.com/nahati/contao-iso...tree/2.0.1-dev

- Habe es mir gerade geladen und versucht, die Tests zu starten, bekomme aber folgenden Fehler, andere Erweiterungen ermöglichen es mir sofort die Tests laufen zu lassen

PHP Fatal error: Uncaught LogicException: You must set the KERNEL_CLASS environment variable to the fully-qualified class name of your Kernel in phpunit.xml

Zudem scheint es so, als wenn hier eine echte Datenbank genutzt wird:

<env name="DATABASE_URL" value="mysql://root:@127.0.0.1:3306/ContaoIsotopeStockBundleTest" />

Ohne diese Datenbank kann ich hier leider nicht helfen, weiß nicht, wo ich den SQL-Dump außer aus dem Repository herbekommen könnte.

Ich bin begeistert über so viel Hilfe

- Bei kleinen Problemen, bei welchen ich die Antwort auf Anhieb weiß, schreibe ich oft noch am selben Abend zurück
- Bei Entwickler-Fragen, welche sehr speziell sind (zumindest in meinem Fall), opfert man die Zeit an seinem Wochenende

Meine Arbeit für Contao war ehrenamtlich. Die Erwartung einer finanziellen Zuwendung habe ich bisher - passend zu meiner quasi Null Einkommenslage - mit einer Contao Association Mitgliedschaft erfüllt...

- Bei Free Open Source Software ist es nun mal die Erwartungshaltung. Du programmierst für Lau, bitte auch Support für Lau - dann bitte aber schnell - denn jeder ist heutzutage wohl ehrenamtlich unterwegs


Würde heute erstmal das Wochenende genießen, dein Thema ist zumindest interessant genug, dass ich es auch gerne testen wollte, um zu helfen (an Wochenenden, wenn man genug Zeit hat).
Mir fehlen aber leider Ressourcen, bzw. kann ich an dieser Stelle nicht einfach deine Tests ausführen

[Ernestopheles]

Danke für deine Unterstützung. Habe das database export file jetzt hochgeladen.

Integrationstest: vendor/bin/phpunit -c phpunit.xml.integration --colors=always

Unittest:vendor/bin/phpunit --colors=always

Der Int.Test benötigt den Kernel, der ist in der phpunit.xml.integration eingetragen

Der Unittest benötigt weder Kernel noch Datenbank.

[zoglo]

Danke für deine Unterstützung.

Bitte folgende Anpassungen machen:

Damit können nun auch andere Leute die Tests laufen lassen

In deiner Datei phpunit.xml.dist:

<php>
....
    <env name="KERNEL_CLASS" value="Nahati\ContaoIsotopeStockBundle\Tests\Integration\app\AppKernel" />
    <env name="APP_SECRET" value="test" />
</php> 

Und nach Debugging habe ich das Problem gefunden...

Dir fehlt folgende Initialisierung in deinem Konstruktor der UpdateItemInCollectionListenerTest
$GLOBALS['TL_DCA']['tl_iso_product']['attributes'] = [];

        // Create tables and insert data
        $this->loadFixture('ContaoIsotopeStockBundleTest.sql');

        // These assignments link the tables with the model classes. Now you can use the model classes to access and manipulate the data in the tables.
        $GLOBALS['TL_MODELS']['tl_iso_producttype'] = ProductType::class;
        $GLOBALS['TL_MODELS']['tl_iso_product'] = Standard::class;
        $GLOBALS['TL_MODELS']['tl_iso_product_collection'] = ProductCollection::class;
        $GLOBALS['TL_MODELS']['tl_iso_product_collection_item'] = ProductCollectionItem::class;

        $GLOBALS['TL_DCA']['tl_iso_product']['attributes'] = [];
        $GLOBALS['TL_DCA']['tl_iso_product']['fields'] = [
            'inventory_status' => [
                'sql' => "varchar(1) NOT NULL default ''",
            ],
        ]; 

Viel Spaß beim Testen, hat mich jetzt eine Stunde meiner Zeit gekostet (Sowas macht man nicht unter der Woche ), nächstes Mal also geduldiger sein

Edit:

        Product::registerModelType("standard", Standard::class); 

muss natürlich vor $objProduct = $this->objItem->getProduct() ?? null passieren

[Ernestopheles]

Erstmal ganz herzlichen Dank für deine Zeit und deine Hilfe!

In der tat haben deine Vorschläge bewirkt, dass das Produkt jetzt instanziert werden kann über

$objProduct = Standard::findByPk('88'); 

.
Dies habe ich aber nur testhalber eingefügt, und das ging vorher noch nicht. Vermutlich hast du das fehlende

$GLOBALS['TL_DCA']['tl_iso_product']['attributes'] = []; 

beim Debuggen dort bemerkt.
(Ich war gestern auch schon an der Stelle, wusste aber nicht, wie der Fehler behoben werden kann.)

Jedoch, leider, lässt sich immer noch nicht das Produkt über das Cart Item holen:

        $objProduct = $this->objItem->getProduct() ?? null; 

liefert null!

Habe beide Wege verglichen:

In beiden Fällen schlägt das Holen der Klasse für den ModelType "standard" fehl.

Beim Instanzierung des Produkts:

$strClass = static::getClassForModelType($objType->class); 

und beim getProduct():

$strClass = Product::getClassForModelType($this->type); 

Beim Instanzieren geht es trotzdem, da anschliessend ein Fallback auf die aktuelle Klasse erfolgreich ist:

$strClass = \get_called_class(); 

Beim getProduct() wird das nicht versucht (und wäre auch falsch, da die aktuelle Klasse ja ProductCollectionItem ist).

Kurzum:
Ich weiß hier nicht weiter. Leider. Die vermutlich entscheidende Frage ist nun, warum die Klasse für den ModelType nicht geholt werden kann.

Dies ist die betreffende Methode:

   /**     * Get class name for given model type     *     * @param string $strName     *     * @return string|null     */    public static function getClassForModelType($strName)    {        return static::$arrModelTypes[$strName] ?? null;    } 

Debuggen lässt sich das ja nicht mehr.

Registriert werden die ModelTypes in der config.php von Isotope. habe versucht, diese über meine Kopie der AppKernel.php aus dem Contao Core einzubinden.

Das führt zu diesem Fehler (vermutlich weil ich PHP 8 verwende):

Fatal error: Uncaught Error: Call to undefined function array_insert() in /shared/httpd/nahati/contao-isotope-stock/tests/Integration/app/config/config.php:17

Habe daraufhin diese Funktion ersetzt durch

array_splice 

Dann kommt gleich der nächste Fehler:

Fatal error: Uncaught Error: Undefined constant "TL_MODE" in /shared/httpd/nahati/contao-isotope-stock/tests/Integration/app/config/config.php:46

Dies lässt sich nicht im construct der Testklasse setzen, habe daher eine bootstrap.php hinzugezogen.

Dann musste ich noch viel in der config.php auskommentieren.

Dann habe ich die Isotope config also erfolgreich wie es scheint eingebunden. ABER: Immer noch sind die ModelTypes nicht vorhanden.

Was da jetzt noch fehlt, ist völlig unklar. UND: Kann es wirklich sein, dass man um einen Test lauffähig zu machen, so tief in Contao und Isotope eindringen muss und auf Vermutung irgendwelche Dinge ergänzt, die sich nicht von selber erklären und dann auch nicht zum Erfolg führen...

Das ist einfach eine Vorgehensweise, die ich unglaublich ätzend finde. ABER: Da es ja kein funktionierendes Beispiel zu geben scheint - jedenfalls hat man mir das bisher trotz Nachfrage nicht gezeigt - und auch keine Doku dazu, hangele ich mich hier von Hölzchen auf Stöckchen.

[Ernestopheles]

Edit:

        Product::registerModelType("standard", Standard::class); 

muss natürlich vor $objProduct = $this->objItem->getProduct() ?? null passieren

Danke, das hat geholfen.


Für mich war das nicht "natürlich". Habe ein paar lange Stunden andere Wege versucht, siehe mein letzter Post.

Wenn man einmal so weit ist, dass einem sowas als "natürlich" erscheint, braucht man "natürlich" nicht zu fragen. Für die anderen, die wie ich nicht ganz so tief drinstecken, werde ich das Vorgehen dokumentieren. Ihr werdet es dann auf meiner Webseite finden. Ich sehe erstmal davon ab, es im dev Handbuch zu ergänzen, da das erfahrungsgemäß auch kein einfacher und widerspruchsfreier Weg ist. Man kann es ja wenn gewollt verlinken.

[zoglo]

Für mich war das nicht "natürlich".

Für mich auch nicht, bin jeden Tag bei komplexen Themen teils mit Trial & Error unterwegs und habe viele Dokumentationen offen
Wenn man einen Thread wie diesen findet, ist man jedoch dankbar, daher wäre es schön, wenn man es doch dokumentiert.

Würde hier schon sagen, dass du einfach weiter machst.
Bei Tests schaue ich es mir (wie von Spooky erwähnt) aus dem Core ab, wenn es um Contao-Plugins geht.

Hab heute auch wieder einiges dazu gelernt, danke dafür

Ich frage echt kaum im Forum, bin da eher in Slack unterwegs aber zu 99% war es dann immer ein Bug

[zonky]

Mein Engagement wird weder gewürdigt noch gibt es konkrete Unterstützung. Schade um die eingesetzte Zeit und Energie und den Ärger den ich mit überheblichen Äusserungen hatte

Damit habt ihr mich erfolgreich vertrieben.

Mit Deinen Fragen hier und auf Slack haben etliche Leute viel Geduld mit Dir obwohl öfter im Nachsatz was überheblichen Äußerungen, schleichende Kommunikation usw. zu hören ist - darüber lässt Du offensichtlich auch auf anderen Kanälen aus...

Dir wurde schon mehrfach erklärt, dass die Leute, die Dir Hilfe anbieten das in ihrer Freizeit und unentgeldlich machen. Forum, Sack usw. sind "Hilfe zur Selbsthilfe" - wenn Dir das nicht reicht, beauftrage jemanden, der Dir Deine Sachen umsetzt. Der kann sich dann gern mit Deinen Anschuldigungen rumschlagen und die - so hoffe ich - in Rechnung stellen.

Die Aussage

Damit habt ihr mich erfolgreich vertrieben.

hab ich nun schon so oft von Dir gehört - im Leben gehört es ab und an dazu, seine Konsequenzen auch umzusetzen.

~~


PS: Meine Anerkennung für die Geduldigen hatte ich heute auf Slack mal finalisiert...

[Ernestopheles]

Danke für deine Meinung, Ingolf.

Aber weißt du, es macht gerade wieder richtig Spaß, da musst du mich wohl noch weiter ertragen

P.S. Gelobt seien die, die Kritik vertragen

[zonky]

P.S. Gelobt seien die, die Kritik vertragen

Gelobt seien die, die Kritik vertragen und was daraus lernen! Ich bin gespannt...