Erweiterung - Sortierung

[JCreeper]

Hallo

habe eine kleine Erweiterung geschrieben, mit einem Slider. Nun soll der User im Inhaltselement (per Select-Feld) einstellen können, welche Reihenfolge die Bilder/Texte haben werden.
Randon, DESC, ASC

Leider bekomme ich es nicht hin. Was mache ich falsch?

Hier ein Auszug aus meinem "modules/ContentSlider.php"

 protected function compile()
    {
    
        $sortBy = "sorting DESC";

        $objStandorte = Database::getInstance()
                    ->prepare(
                        'SELECT *
                        FROM tl_gerichte LEFT JOIN tl_standorte ON tl_gerichte.pid = tl_standorte.id
                        WHERE tl_gerichte.pid = ?
                        AND tl_standorte.published = ?
                        ORDER BY ?'
                    )
                    ->execute($this->standorte, 1, $sortBy);

        $this->Template->ytvideo_data = $objStandorte->fetchAllAssoc();

    } 

[fiedsch]

Ich bin mir nicht sicher, ob Du Platzhalter (?) im ORDER BY Teil des SQL-Statements verwenden kannst.

Aber abgesehen davon: wie äußert sich denn "bekomme es nicht hin"?

[Samson1964]

Ich bin mir nicht sicher, ob Du Platzhalter (?) im ORDER BY Teil des SQL-Statements verwenden kannst.

Ich denke wahrscheinlich macht Contao daraus

ORDER BY 'sorting DESC'

Aber abgesehen davon: wie äußert sich denn "bekomme es nicht hin"?

Vielleicht mit einem SQL-Fehler bei ORDER BY?!

[JCreeper]

Genau...Contao macht daraus

ORDER BY 'sorting DESC' 

Aber im Frontend wird alles sauber ausgegeben...nur halt die Sortierung wird ignoriert. Und ich weiss nicht wie ich das lösen kann, das ich im Inhaltselement sage...Sortiere random, DESC oder ASC
und er übernimmt das im Template.

[folkfreund]

...nur halt die Sortierung wird ignoriert.

Klar, weil nach 'sorting DESC' nicht sortiert werden kann...


So könnte es gehen:

    protected function compile()
     {
        $sortBy = "sorting DESC"; 
        $objStandorte = Database::getInstance()
                     ->prepare(
                         'SELECT *
                         FROM tl_gerichte LEFT JOIN tl_standorte ON tl_gerichte.pid = tl_standorte.id
                         WHERE tl_gerichte.pid = ?
                         AND tl_standorte.published = ?
                         ORDER BY '.$sortBy
                     )
                     ->execute($this->standorte, 1); 
        $this->Template->ytvideo_data = $objStandorte->fetchAllAssoc(); 
    } 

[JCreeper]

Super...hat geklappt. Vielen Dank.

In meiner tl_content.php habe ich das Sortier-Feld eingefügt

$GLOBALS['TL_DCA']['tl_content']['fields']['lernapp_sortBy'] = array
(
    'label'                   => &$GLOBALS['TL_LANG']['tl_content']['lernapp_sortBy'],
    'exclude'                 => true,
    'inputType'               => 'select',
    'options'                 => array('product_name_asc', 'product_name_desc','random'),
    'reference'               => &$GLOBALS['TL_LANG']['tl_content'],
    'eval'                    => array('tl_class'=>'w50'),
    'sql'                     => "varchar(255) NOT NULL default ''"
); 

Wie kann ich das nun übergeben bzw den Wert in der Select Abfrage auslesen?
So???

if (\Input::get('lernapp_sortBy'))
        {
            $lernapp_sortBy = \Input::get('lernapp_sortBy');
        } 

[fiedsch]

Wie kann ich das nun übergeben bzw den Wert in der Select Abfrage auslesen?
So???

if (\Input::get('lernapp_sortBy'))
        {
            $lernapp_sortBy = \Input::get('lernapp_sortBy');
        } 

Im Prinzip ja, aber Du willst natürlich sicher stellen, daß der übergebene Parameterwert auch das beinhaltet, was Du erwartest. Ansonsten hast Du eine SQL-Injection Sicherheitslücke eingebaut :-o

[JCreeper]

Hab es nun so gelöst. Wäre das "sauber" ?

protected function compile()
    {
        //Find out sorting
        if($this->lernapp_sortBy) 
        {
            $t = 'tl_gerichte';
            if($this->lernapp_sortBy == 'product_name_asc') { $sortBy = "$t.bezeichnung ASC"; }
            elseif($this->lernapp_sortBy == 'product_name_desc') { $sortBy = "$t.bezeichnung DESC"; }
            else { $sortBy = "rand()"; }
        }


        $objStandorte = Database::getInstance()
                    ->prepare(
                        'SELECT *
                        FROM tl_gerichte LEFT JOIN tl_standorte ON tl_gerichte.pid = tl_standorte.id
                        WHERE tl_gerichte.pid = ?
                        AND tl_standorte.published = ?
                        ORDER BY '.$sortBy
                    )
                    ->execute($this->standorte, 1);

        $this->Template->ytvideo_data = $objStandorte->fetchAllAssoc();

    } 

[PaddySD]

Im Prinzip ja, aber Du willst natürlich sicher stellen, daß der übergebene Parameterwert auch das beinhaltet, was Du erwartest. Ansonsten hast Du eine SQL-Injection Sicherheitslücke eingebaut :-o

@fiedsch
Genau sowas meinte ich letzhin von wegen Audit-Team... Ich komme auf dem Stammtisch drauf zurück!