Reihenfolge Newsletter im Modul Persönliche Daten

[Acta]

Ich habe 7 Newsletter die die nutzer via dem Modul Persönliche Daten bestellen und abbestellen können.

Leider wird die Reihenfolge im Modul nicht nach dem Alphabet oder nach der ID vorgenommen. Nun Frage ich mich... welche Sortierung erfolgt überhaupt?

[Acta]

Ich habe mal ein Bild angehangen... und auch die IDs dahinter geschrieben... Ich hätte gerne eine Alphabetische Sortierung... Sprich "AKTUELL" zu erst....Könnt ihr mir helfen?

[Acta]

In der ONline Demo von Contao ist es soweit ich testen konnte zumindestens nach der ID

[Acta]

// HOOK: add newsletter fields
        if (in_array('newsletter', $this->Config->getActiveModules()))
        {
            $this->Template->newsletter = $arrFields['newsletter'];
            $this->Template->newsletterDetails = $GLOBALS['TL_LANG']['tl_member']['newsletterDetails'];
        } 

Ich denke hier müsste ich doch dann ansetzen oder?

[the_scrat]

Hi,

es hat nichts mit der ID oder sonstiger Sortierung zu tun.
Ein Blick in den Quellcode verrät, dass die Newsletter(-liste) so geladen wird, wie sie aus der DB kommt.

Schau mal in der Datenbank welche Reihenfolge tl_newsletter_channel hat. Müsste 1:1 übereinstimmen.


Um die Reihenfolge zu ändern müsstest du ggf. ein Ticket auf Github öffnen oder die Newsletter Klasse (=> Methode getNewsletters) überschreiben

[Acta]

Hallo danke für die Info... in der Tat wird es nach der DB sortiert.

Nur verstehe ich die Sortierung in der Datenbank nicht.

Es wird ja eine ID und ein Tstamp vergeben... doch wo nach sortiert die Datenbank dann die Einträge????

[Acta]

Ich habe mal ein Ticket aufgemacht:

https://github.com/contao/core/issues/7864

Hat jemand von euch eine entsprechende Anpassung im Einsatz oder könnte mir ein passendes Beispiel bennen?

Bis jetzt habe ich noch keine Idee wo und wie ich eingreifen müsste, um eine Alphabetische Sortierung zu generieren.

[the_scrat]

Hi,

ich weiß zwar nicht wen du mit "euch" meinst..... ich glaub wir sind hier alleine ;-)

Also, hier eine quick&dirty Lösung die NICHT Updatesicher ist, aber vielleicht wird das Ticket ja relativ schnell geschlossen und der Fehler damit behoben.

Also, ändere folgendes ab, damit du deine Sortierung entsprechend bekommst

Öffne die Datei /system/modules/newsletter/classes/Newsletter.php

Zeile 830

public function getNewsletters($objModule)
{
       $objNewsletter = \NewsletterChannelModel::findAll();

       if ($objNewsletter === null)
       {
              return array();
       }

       $arrNewsletters = array();

       // Return all channels if $objModule is null (see #5874)
       if ($objModule === null || TL_MODE == 'BE')
       {
              while ($objNewsletter->next())
              {
                     $arrNewsletters[$objNewsletter->id] = $objNewsletter->title;
              }
       }
       else
       {
              $newsletters = deserialize($objModule->newsletters, true);

              if (!is_array($newsletters) || empty($newsletters))
              {
                     return array();
              }

              while ($objNewsletter->next())
              {
                     if (in_array($objNewsletter->id, $newsletters))
                     {
                            $arrNewsletters[$objNewsletter->id] = $objNewsletter->title;
                     }
              }
       }

       return $arrNewsletters;
} 

GANZ unten das return $arrNewsletters änderst du ab in

return sort($arrNewsletters); 

Damit müsste, wenn ich mich nicht irre, die Liste alphabetisch zurückgegeben werden.

Test it! Bin gespannt!

[Acta]

Auf meiner alten Contao Version werden dann im Frotend Modul Persönliche Daten... einfach keine Newsletter mehr angezeigt.

Habe gerade keine 3.xx er da... aber der Aufbau

    public function getNewsletters($objModule)
    {
        $objNewsletter = $this->Database->execute("SELECT id, title FROM tl_newsletter_channel");

        if ($objNewsletter->numRows < 1)
        {
            return array();
        }

        $arrNewsletters = array();

        // Back end
        if (TL_MODE == 'BE')
        {
            while ($objNewsletter->next())
            {
                $arrNewsletters[$objNewsletter->id] = $objNewsletter->title;
            }

            return $arrNewsletters;
        }

        // Front end
        $newsletters = deserialize($objModule->newsletters, true);

        if (!is_array($newsletters) || count($newsletters) < 1)
        {
            return array();
        }

        while ($objNewsletter->next())
        {
            if (in_array($objNewsletter->id, $newsletters))
            {
                $arrNewsletters[$objNewsletter->id] = $objNewsletter->title;
            }
        }

        return $arrNewsletters ;
    } 

[the_scrat]

Na dann ändere doch da einfach die Datenbankabfrage.....

$objNewsletter = $this->Database->execute("SELECT id, title FROM tl_newsletter_channel ORDER BY title"); 

[Acta]

Oh man ... das war so einfach... das es schon wieder peinlich war das überhaupt nicht kapiert zu haben ...

großes DANKESCHÖN!