dynamische Bildgrössen

[Luminanz]

Hallo

Ich habe folgenden code aus einer contao 3.2 Installation, mit dem für Redakteure die Bildgrössen dynamisch vorbelegt werden:

class dynamicDefaults extends \Backend
{
        public function onload_callback($dc)
        {
        //Default-Werte für das Inhaltselement Galerie
        $arrDefaultGallery = array
        (
        'size' => array('100','100','center_center'),
        'fullsize' => true
        );
        //Default-Werte für das Inhaltselement Bild
        $arrDefaultImage = array
        (
        'size' => array('200','','proportinal')
        );
        //Default-Werte für das Inhaltselement Text (Bild hinzufügen)
        $arrDefaultText = array
        (
        'size' => array('200','','proportinal'),
        'floating' => 'left'
        );
        if (!$dc->activeRecord) {
             $dc->activeRecord = $this->Database
                ->prepare("SELECT * FROM tl_content WHERE id=?")
                ->execute($dc->id);
        }
            if ($dc->activeRecord->type == 'gallery' && !$dc->activeRecord->multiSRC) {
                $dc->activeRecord = $this->Database
                  ->prepare('UPDATE tl_content %s WHERE id=?')
                  ->set($arrDefaultGallery)
                  ->execute($dc->id);
            }
            if ($dc->activeRecord->type == 'image' && !$dc->activeRecord->singleSRC) {
                $dc->activeRecord = $this->Database
                  ->prepare('UPDATE tl_content %s WHERE id=?')
                  ->set($arrDefaultImage)
                  ->execute($dc->id);
            }
            if ($dc->activeRecord->type == 'text' && !$dc->activeRecord->singleSRC) {
                $dc->activeRecord = $this->Database
                  ->prepare('UPDATE tl_content %s WHERE id=?')
                  ->set($arrDefaultText)
                  ->execute($dc->id);
            }
        } 
} 

Leider funktioniert dies unter contao 3.5 nicht mehr.
Was genau hat sich geändert?

Vielen Dank für eure Tipps

[Spooky]

Ich kann mir irgendwie nicht vorstellen dass dieser Code jemals funktioniert hat. Du versuchst den activeRecord zu laden, wenn dieser nicht existiert - aber wenn dieser nicht existiert, kannst du ihn auch nicht laden.

Solltest du nicht einfach nur die default Werte des DCAs ändern?

[Luminanz]

Solltest du nicht einfach nur die default Werte des DCAs ändern?

Ja, grundsätzlich schon. Es sollen aber unterschiedliche Bildgrössen definiert werden können (Inhaltselement Bild und Galerie).

[Spooky]

Ja natürlich. Ich meine damit - solltest du in deinem onload_callback nicht einfach nur den default Wert im DCA ändern? Eben abhängig vom Typ.

[Luminanz]

Ja natürlich. Ich meine damit - solltest du in deinem onload_callback nicht einfach nur den default Wert im DCA ändern? Eben abhängig vom Typ.

Und wie meinst du das konkret [emoji15]?

[Spooky]

Sieh's dir einfach beim core ab - dort wird auch je nach Inhaltselement Typ das DCA verändert:

• https://github.com/contao/core-bundl...nt.php#L25-L27
• https://github.com/contao/core-bundl...hp#L1031-L1060

[Luminanz]

Ok, sehe was du meinst (adjustDcaByType).
Kapiere nicht alles, werde es heute Abend aber mal versuchen. Danke!

[Luminanz]

Hab es damit versucht. Funktioniert leider nicht.

initconfig.php

<?php

// Put your custom configuration here
// callback für dynamische Default-Werte aufrufen:
$GLOBALS['TL_DCA']['tl_content']['config']['onload_callback'][] = array('dynamicDefaults','set_defaults');

dynamicDefaults.php

<?php
/**
 * Namespace
 */
namespace dynamicDefaults;
/**
 * Class dynamicDefaults
 *
 */
class dynamicDefaults extends \Backend
{
        public function set_defaults($dc)
        {
          if ($_POST || Input::get('act') != 'edit')
          {
            return;
          }
          $objCte = ContentModel::findByPk($dc->id);
          if ($objCte === null)
          {
            return;
          }
          switch ($objCte->type)
          {
            case 'gallery':
              // Default image size - width, height, mode|size-ID
              $GLOBALS['TL_DCA']['tl_content']['fields']['size']['default']  = array('', '', 1);
              break;
            case 'image':
              // Default image size - width, height, mode|size-ID
              $GLOBALS['TL_DCA']['tl_content']['fields']['size']['default']  = array('', '', 2);
              break;
          }
        }

}

[Luminanz]

Habe noch diesen Beitrag gefunden, hier besteht ein ähnliches Anliegen:

DCA Einschränkung der Optionen für "Bildbreite und Bildhöhe" am CE "Image"

https://community.contao.org/de/show...l=1#post427898

[Luminanz]

Wenn ich den onload_callback in der dcaconfig.php aufrufe,
erhalte ich im backend eine weisse Seite, sobald ich einen Artikel bearbeiten will.

Hat jemand eine Erklärung hierfür?

[Spooky]

Was genau machst du in der dcaconfig.php?

[Luminanz]

 <?php

// Put your custom configuration here
// callback für dynamische Default-Werte aufrufen:
$GLOBALS['TL_DCA']['tl_content']['config']['onload_callback'][] = array('dynamicDefaults','set_defaults');

[Spooky]

Das solltest du lieber in einer Extension machen, nicht in der dcaconfig.php (und auch nicht in der initconfig.php). Die dcaconfig.php wird potentiell mehrmals aufgerufen - mit deinem Code wird also dieser Callback merhmals registriert. Du könntest natürlich alternativ dort das Array durchsuchen und überprüfen, ob der Callback schon registriert ist und nur wenn er noch nicht registriert ist den Callback einfügen.

Davon abgesehen sollten Klassennamen mit einem Großbuchstaben beginnen, aber das hat nichts mit dem Fehler zu tun, sondern wäre einfach nur best-practice

[Luminanz]

Habe im config-Ordner eine config.php erstellt und den Aufruf des callbacks dort eingefügt. Die Artikel können nun wieder normal bearbeitet werden.

Aber leider werden die defaults noch immer nicht gesetzt.
Ist es möglich, dass meine Modul (dynamicDefaults) zu spät geladen wird. Müsste es eventuell vor dem core geladen werden?

[Spooky]

Den config Ordner deiner eigenen Extension meinst du, oder?

[Luminanz]

Ja, genau.

[Spooky]

Füge mal ein

var_dump('foo'); exit; 

ganz oben in deinem Callback ein, um zu überprüfen, dass dieser Callback tatsächlich aufgerufen wird. Erzeuge dann ein neues Inhaltselement - du solltest eine weiße Seite mit "foo" sehen.

[Luminanz]

Habe den Code eingefügt, eine weisse Seite erhalte ich aber nicht.
Anbei mein Zwischenstand.

[Spooky]

1. Die Extension würde ich von dynamicDefaults in dynamic_defaults umbenennen, das wäre der Contao Standard. Vergiss nicht nach dieser Änderung überall alle Pfade anzupassen.
2. Den Namespace würde ich auch entweder in DynamicDefaults umbenennen oder ganz entfernen. Ebenso auch hier nicht vergessen nach dieser Änderung überall alles anzupassen.
3. Der eigentliche Fehler liegt in der autoload.php - deine Klasse wird nämlich gar nicht geladen, da du erstens den falschen Klassennamen verwendest und zweitens den falschen Pfad zur Klasse.
4. Die Methode set_defaults würde ich in setDefaults umbenennen.

[Luminanz]

Danke für die vielen Hinweise!
Habe (hoffentlich) alles berücksichtigt.
Leider klappt es aber noch nicht

[Spooky]

In diesem zip ist alles unverändert

[Luminanz]

Sorry, anbei nun die aktuelle Version

[Spooky]

Wird der Callback nun aufgerufen?

[Luminanz]

Leider nein, zumindest erhalte ich nicht eine weisse Seite mit foo beim Erstellen eines Inhaltselements.

[Luminanz]

Ich habe den Aufruf des callbacks testweise wieder in die dcaconfig.php verschoben.
Dann erhalten ich folgende Meldung, sobald ich einen Artikel öffne:
Missing data container for table "tl_content"

Wo liegt hier genau der Fehler?

[Spooky]

Momentan kann ich keinen Fehler in der Extension feststellen. Über die dcaconfig.php würde ich es ohnehin nicht machen.

[Luminanz]

Ich teste es mit einer Contao 3.5.28. Beim Contao-Check ist alles ok.
Könnte eventuell jemand die Extension bei sich testen?

[Samson1964]

Kann ich spätestens morgen Vormittag machen.

Gesendet von meinem L52 mit Tapatalk

[Luminanz]

Ok, vielen Dank schon mal.

[Samson1964]

Ist mir schleierhaft. Wenn config.php/autoload.php in meiner lokalen 3.5.27 abgearbeitet wird, ist das $GLOBALS-Array noch recht übersichtlich und $GLOBALS['TL_DCA'] fehlt noch völlig. Da wundert es mich nicht, daß Deine Callback-Zeile in der config.php später von core/dca/tl_content.php überschrieben wird. Und das trotz requires[] = "core" in der autoload.ini.
In der dcaconfig.php ist $GLOBALS['TL_DCA'] da.

Das gleiche Verhalten gibt es in einer Online-3.5.28.

<Spekulation>TL_DCA gibt es erst, wenn das dca-Verzeichnis abgearbeitet wird, was zu diesem Zeitpunkt wohl noch nicht der Fall ist. Ich würde den Callback in eine eigene dca/tl_content.php verlagern.</Spekulation>

[Spooky]

Ah, jetzt sehe ich erst den Fehler. Du hast den Callback in der config.php eingetragen - den musst du aber in der dca/tl_content.php eintragen

[Samson1964]

Ah, jetzt sehe ich erst den Fehler. Du hast den Callback in der config.php eingetragen - den musst du aber in der dca/tl_content.php eintragen

Habe ich den Meister darauf gebracht oder hast Du meine Spekulation gar nicht gesehen?

[Luminanz]

Ok, vielen Dank euch beiden. Werde ich testen!

[Spooky]

Habe ich den Meister darauf gebracht oder hast Du meine Spekulation gar nicht gesehen?

Du hast mich darauf gebracht

[Luminanz]

Der callback wird nun aufgerufen. Ich erhalte aber den Fehler <Missing data container for table "tl_content">.
Muss ich in der tl_content.php sonst noch etwas einfügen? Habe nur den Aufruf des callbacks darin.

[Spooky]

Poste deine Extension.

[Samson1964]

Laut der Core tl_content ist das nicht so klar. Bei checkPermission gibt es keinen Parameter, bei showJsLibraryHint wird $dc als Parameter angegeben. Und beide sind onload_callbacks.

[Luminanz]

Poste deine Extension.

Hier der aktuelle Stand

[Samson1964]

Kurzer Blick in Deine tl_content: Warum überschreibst Du das onload_callback-Array? Deine erste Variante ist doch ganz in Ordnung gewesen, oder?
Und warum steht jetzt ein Prefix "Contao" in der autoload.php. Hat das andere nicht funktioniert?

EDIT: Ach Du überschreibst ja ganz tl_content...

[Spooky]

• Wie Samson schon angemerkt hat, ist deine dca/tl_content.php falsch. Du löschst dort fast das gesamte DCA. Richtig wäre die Variante die du auskommentiert hast.
• Du hast in verschiedenen PHP Dateien nun ein ?> am Ende. Das brauchst du nicht. Lösche es wieder.
• Du verwendest nun den Namespace "Contao". Den brauchst du nicht. Und wenn dann solltest du lieber einen eigenen Namespace verwendne.