Eigenes DCA für tl_newsletter_recipients wird ignoriert

[Samson1964]

Mein DCA für die tl_newsletter_recipients wird ignoriert. Die Debugausgabe im Code ist leer. Offensichtlich existiert $GLOBALS['TL_DCA']['tl_newsletter_recipients']['palettes']['default'] zu dem Zeitpunkt noch nicht.

dca/tl_newsletter_recipients.php:

<?php

// Palette manipulieren
$debug = print_r($GLOBALS['TL_DCA']['tl_newsletter_recipients']['palettes']['default'], true);

log_message($debug,'spielerregister.log');
$GLOBALS['TL_DCA']['tl_newsletter_recipients']['palettes']['default'] = str_replace('active', 'active,spielerregister_mailTime', $GLOBALS['TL_DCA']['tl_newsletter_recipients']['palettes']['default']);

/**
 * Neue Felder in tl_newsletter_recipients
 */

// Zeitstempel des letzten Versands des Spielerregisters
$GLOBALS['TL_DCA']['tl_newsletter_recipients']['fields']['spielerregister_mailTime'] = array
(
    'label'                   => &$GLOBALS['TL_LANG']['tl_newsletter_recipients']['spielerregister_mailTime'],
    'exclude'                 => true,
    'flag'                    => 5,
    'inputType'               => 'text',
    'eval'                    => array
    (
        'tl_class'            => 'w50'
    ),
    'sql'                     => "int(10) unsigned NOT NULL default '0'"
);

In meiner composer.json habe ich sicherheitshalber eingetragen:

"require": {
    "contao/newsletter-bundle": ">=0"
}, 

Ist aber scheinbar ohne Wirkung.
Auch meine Felddefinition von spielerregister_mailTime wird ignoriert. Trage ich das Feld manuell in die Datenbank ein, wird mir das Feld zum Löschen vorgeschlagen.

Ich finde den Fehler in meinem Code nicht. Die Datei wird aber geparst, denn wenn ich einen PHP-Fehler provoziere, wird der ausgegeben. Ich habe die Vermutung, das mein DCA später vom Newsletter-Bundle überschrieben wird. Falls das so ist, wie muß ich dieses Bundle als Abhängigkeit einfügen?

[Spooky]

Ist das nun eine Applikationsanpassung oder implementierst du das in einer eigenen Extension?

[Samson1964]

In einer eigenen Extension.

[markocupic]

Hallo
In der src/ContaoManager/Plugin.php musst du definieren, dass dein Bundle nach ContaoNewsletterBundle geladen werden muss.

setLoadAfter....

Lg Marko

[Spooky]

Siehe auch https://docs.contao.org/dev/framewor...lugininterface

[markocupic]

setLoadAfter....

class Plugin implements BundlePluginInterface
{
    public function getBundles(ParserInterface $parser): array
    {
        return [
            BundleConfig::create(ContaoExampleBundle::class)
                ->setLoadAfter([
                    ContaoCoreBundle::class,
                    ContaoNewsletterBundle::class,
                ]),
        ];
    }
}

und danach ein composer install

Lg Marko

[Samson1964]

Ich kann noch keinen Erfolg melden. Bei composer install kommt:

In tl_settings.php line 480:                                                 
syntax error, unexpected '$GLOBALS' (T_VARIABLE) 

Jetzt beginnt das Rätselraten welche der vielen tl_settings.php gemeint ist. Vor allen Dingen auch noch eine mit vielen Zeilen.

[markocupic]

Vielleicht die tl_settings.php im cache var/cache/prod/contao/dca/tl_settings.php oder var/cache/dev/contao/dca/tl_settings.php

Ich kann noch keinen Erfolg melden. Bei composer install kommt:

In tl_settings.php line 480:                                                 
syntax error, unexpected '$GLOBALS' (T_VARIABLE) 

Jetzt beginnt das Rätselraten welche der vielen tl_settings.php gemeint ist. Vor allen Dingen auch noch eine mit vielen Zeilen.

[Spooky]

Ich kann noch keinen Erfolg melden. Bei composer install kommt:

In tl_settings.php line 480:                                                 
syntax error, unexpected '$GLOBALS' (T_VARIABLE) 

Jetzt beginnt das Rätselraten welche der vielen tl_settings.php gemeint ist. Vor allen Dingen auch noch eine mit vielen Zeilen.

Das dürfte aber nun ein komplett anderes Problem sind und hat nichts mit deinem ursprünglichen Thread zu tun.

[Samson1964]

Stimmt. Das ist ein anderes Problem. So lange ich aber den Fehler in:

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

/**
 * Contao Open Source CMS
 *
 * Copyright (C) 2005-2013 Leo Feyer
 *
 * @package   fen
 * @author    Frank Hoppe
 * @license   GNU/LGPL
 * @copyright Frank Hoppe 2013
 */

/**
 * palettes
 */
$GLOBALS['TL_DCA']['tl_settings']['palettes']['default'] .= ';{spielerregister_legend:hide},spielerregister_imageSize';

/**
 * fields
 */

$GLOBALS['TL_DCA']['tl_settings']['fields']['spielerregister_imageSize'] = array
(
    'label'                   => &$GLOBALS['TL_LANG']['tl_settings']['spielerregister_imageSize'],
    'exclude'                 => true,
    'inputType'               => 'imageSize',
    'reference'               => &$GLOBALS['TL_LANG']['MSC'],
    'eval'                    => array('rgxp'=>'natural', 'includeBlankOption'=>true, 'nospace'=>true, 'helpwizard'=>true, 'tl_class'=>'w50'),
    'options_callback' => static function ()
    {
        return System::getContainer()->get('contao.image.image_sizes')->getOptionsForUser(BackendUser::getInstance());
    },
    'sql'                     => "varchar(255) NOT NULL default ''"
);

(Zeile 480 ist die mit /** und nachfolgend fields)

Hat bis vor Kurzem funktioniert. Erst seitdem ich versuche das Newsletter-Bundle reinzubekommen, kracht es bei composer install an der Stelle.

Ich werde mal das setLoadAfter-Zeug wieder rauswerfen und ein neues composer install versuchen.

[Spooky]

Dieses

if (!defined('TL_ROOT')) die('You cannot access this file directly!'); 

ist noch ein Überbleibsel aus Typolight/Contao 2. Das kannst du entfernen.

[Samson1964]

Das "composer install"-Problem ist gelöst. UltraEdit hat ein BOM in die Datei eingefügt, obwohl diese Option in den Programmeinstellungen nicht aktiviert ist. Ich habe die Datei auf ANSI umgestellt, danach funktionierte es.

Nicht funktioniert nach wie vor die Erweiterung des DCA von tl_newsletter_recipients.

Auszug Plugin.php des Bundles:

public function getBundles(ParserInterface $parser)
{
    return [
        BundleConfig::create(ContaoSpielerregisterBundle::class)
            ->setLoadAfter([ContaoCoreBundle::class, ContaoNewsletterBundle::class]),
    ];
} 

Auszug composer.json des Bundles:

"require": {
    "php": "^5.6 || ^7.0",
    "contao/core-bundle": "^4",
    "contao/newsletter-bundle": ">=0",
    "schachbulle/contao-helper-bundle": ">=0"
}, 

Irgendwas fehlt da bestimmt noch. Ich frage mich sowieso, warum ich composer install extra ausführen muß. Ich entwickle im vendor-Ordner und habe immer den Debug-Modus an. Bisher funktionierten alle Änderungen im vendor-Ordner. Gelegentlich mußte ich den Composer-Cache löschen, mehr noch nie. Und composer install hat einen unschönen Nebeneffekt: Schlägt die Installation fehl, wird der vendor-Ordner mit alten Dateien überschrieben.

[Spooky]

Nicht funktioniert nach wie vor die Erweiterung des DCA von tl_newsletter_recipients.

Auszug Plugin.php des Bundles:

public function getBundles(ParserInterface $parser)
{
    return [
        BundleConfig::create(ContaoSpielerregisterBundle::class)
            ->setLoadAfter([ContaoCoreBundle::class, ContaoNewsletterBundle::class]),
    ];
} 

Poste die gesamte Plugin.php

Auszug composer.json des Bundles:

"require": {
    "php": "^5.6 || ^7.0",
    "contao/core-bundle": "^4",
    "contao/newsletter-bundle": ">=0",
    "schachbulle/contao-helper-bundle": ">=0"
}, 

Statt >=0 kannst du auch einfach * schreiben. Aber es wäre empfehlenswerter gewisse Mindestversionen anzugeben. Davon abgesehen musst du dir überlegen, ob dein Bundle tatsächlich eine Hard Dependency auf das contao/newsletter-bundle hat - oder ob deine Änderungen einfach nur Optional sind.

[Samson1964]

vendor/schachbulle/contao-spielerregister-bundle/src/ContaoManager/Plugin.php:

<?php

namespace Schachbulle\ContaoSpielerregisterBundle\ContaoManager;

use Contao\CoreBundle\ContaoCoreBundle;
use Contao\ManagerPlugin\Bundle\BundlePluginInterface;
use Contao\ManagerPlugin\Bundle\Config\BundleConfig;
use Contao\ManagerPlugin\Bundle\Parser\ParserInterface;
use Schachbulle\ContaoSpielerregisterBundle\ContaoSpielerregisterBundle;

class Plugin implements BundlePluginInterface
{
    /**
     * {@inheritdoc}
     */
    public function getBundles(ParserInterface $parser)
    {
        return [
            BundleConfig::create(ContaoSpielerregisterBundle::class)
                ->setLoadAfter([ContaoCoreBundle::class, ContaoNewsletterBundle::class]),
        ];
    }
}

[markocupic]

Das use statement fehlt für ContaoNewsletterBundle.

Richtig wäre:

<?php

namespace Schachbulle\ContaoSpielerregisterBundle\ContaoManager;

use Contao\CoreBundle\ContaoCoreBundle;
use Contao\NewsletterBundle\ContaoNewsletterBundle;
use Contao\ManagerPlugin\Bundle\BundlePluginInterface;
use Contao\ManagerPlugin\Bundle\Config\BundleConfig;
use Contao\ManagerPlugin\Bundle\Parser\ParserInterface;
use Schachbulle\ContaoSpielerregisterBundle\ContaoSpielerregisterBundle;

class Plugin implements BundlePluginInterface
{
    /**
     * {@inheritdoc}
     */
    public function getBundles(ParserInterface $parser)
    {
        return [
            BundleConfig::create(ContaoSpielerregisterBundle::class)
                ->setLoadAfter([ContaoCoreBundle::class, ContaoNewsletterBundle::class]),
        ];
    }
}

[Spooky]

Du referenzierst ContaoNewsletterBundle::class ohne use statement. Das wäre in deinem Fall dann der folgende FQCN: Schachbulle\ContaoSpielerregisterBundle\ContaoManager\ContaoNewsletterBundle, eine solche Klasse gibt es aber höchstwahrscheinlich nicht. Wenn dann \Contao\NewsletterBundle\ContaoNewsletterBundle::class bzw.

use Contao\NewsletterBundle\ContaoNewsletterBundle;

…

ContaoNewsletterBundle::class 

[Samson1964]

Danke das wars! Ich habe selbst ein bißchen nach meinem Posting mit use experimentiert, aber nicht den richtigen Aufruf erraten. Mit

->setLoadAfter([ContaoCoreBundle::class, \Contao\NewsletterBundle\ContaoNewsletterBundle::class]), 

wurde jetzt mein DCA erkannt und es kann weitergehen.

Vielen Dank an euch beide!

[Spooky]

Nutzt du keine IDE? Die würde das für dich machen bzw. dich auch gleich auf ein fehlendes use statement hinweisen.

[markocupic]

Wenn du das use statement

use Contao\NewsletterBundle\ContaoNewsletterBundle;

gesetzt hast, wäre dann aber das hier gscheiter ;-)

->setLoadAfter([ContaoCoreBundle::class, ContaoNewsletterBundle::class]), 

Sonst macht das use sttement keinen Sinn.

[Samson1964]

Nutzt du keine IDE? Die würde das für dich machen bzw. dich auch gleich auf ein fehlendes use statement hinweisen.

Die Version von UltraEdit die ich habe, hat keine Unterstützung dafür.

Wenn du das use statement

use Contao\NewsletterBundle\ContaoNewsletterBundle;

gesetzt hast, wäre dann aber das hier gscheiter ;-)
Sonst macht das use sttement keinen Sinn.

Habe use nicht eingesetzt.

[Spooky]

Die Version von UltraEdit die ich habe, hat keine Unterstützung dafür.

UltraEdit ist ja auch nur ein einfacher Editor. Ich würde dir PHPStorm oder VSCode + Intelliphense empfehlen.