Filtern nach GET-Liste

[pout]

Ich übergebe per GET in order_no eine Liste von Order-Nummern und möchte danach mit eigenem SQL filtern:

SELECT * FROM {{table}} WHERE order_no IN ({{param::get?name=order_no}}) AND published=1 ORDER BY FIELD(order_no,{{param::get?name=order_no}}); 

Leider werden die Parameter in einfache Anführungszeichen eingehüllt, sodass das nicht geht. Und in der Orderanweisung braüchte ich alles in einzeln in Anführungszeichen:

SELECT * FROM mm_products WHERE order_no IN ('3,4,5,6') AND published=1 ORDER BY FIELD(order_no,'3,4,5,6'); 

bräuchte:

SELECT * FROM mm_products WHERE order_no IN (3,4,5,6) AND published=1 ORDER BY FIELD(order_no,'3','4','5','6'); 

Wie kann man das hinbekommen, eventuell auch mit MySQL Funktionen?

[07alex07]

hast du schon mit der PHP Funktion

explode

und

implode

versucht?

[pout]

Es geht um das Feld "Eigene SQL-Abfrage" in den MM Filtereinstellungen. Kann ich da php-Funktionen verwenden? Wenn wie?

[Andreas]

Gehe ich recht in der Annahme, dass du dort kein PHP ausführen kannst? Wenn ja, dann kannst du es so probieren.

SELECT * FROM {{table}}
WHERE order_no IN ({{file::input-get.html5?key=order_no}})
AND published=1
ORDER BY FIELD(order_no,{{param::get?name=order_no}});

system/templates/input-get.html5

<?php
if(\Input::get('key') == 'order_no' && \Input::get('order_no'))
{
  echo \Input::get('order_no');
}

Bist du dir sicher, dass du für ORDER BY FIELD die Zahlen als String benötigst? Ich meine nicht.

[pout]

super das hat funktioniert. Vielen Dank!

SELECT * FROM {{table}}
WHERE order_no IN ({{file::input-get.html5?key=order_no}})
AND published=1
ORDER BY FIELD(order_no,{{file::input-get.html5?key=order_no}}); 

und input-get.html5 in /templates.

<?php
if(\Input::get('key') == 'order_no' && \Input::get('order_no'))
{
  echo \Input::get('order_no');
} 
else echo 0;

oder unverseller:

 <?php
$key = \Input::get('key');
if(\Input::get($key))
{
  echo \Input::get($key);
} 
else echo 0;

[Andreas]

Super.

?key=order_no kannst du natürlich weglassen. So hast du aber die Möglichkeit die gleiche Datei nochmal für andere GETs zu benutzen.

[pout]

Ja, oder ich übergebe den Key und formuliere die template-Datei allgemeiner, wie in meinem obigen bearbeiteten Post. Dann kann ich mir im Filter die Get-Variable aussuchen/festlegen.

[Andreas]

Ja, genau. Es ging ja nur um die Technik. Du kannst die Datei übrigens auch .php anstatt .html5 nennen. .html5 hat nur den Vorteil (oder Nachteil) dass du sie im BE unter den Templates siehst.

[webstar]

Du baust dir damit aber eine gravierende Sicherheitslücke ein, da du jegliches escapen umgehst.

Stell dir mal einen Key in der Form "1); UPDATE tl_user SET password='newpasswordhash'; SELECT IRGENDWAS DAMIT DIE DB ABFRAGE VALIDE BLEIBT".

Wenn dann solltest du die Parameter schützen. Wenn es IDs sind, z.B. folgendermaßen.

 <?php 
$key    = \Input::get('key'); 
$value =\Input::get($key);

if($value) { 
    $value = trimsplit(',', $value);
    $value = array_map('intval', $value);

    echo implode(',', $value);
}  
else echo 0;

[Andreas]

Aber das macht die Methode doch schon. Deswegen benutze ich doch die Input-Klasse. Oder?
https://github.com/contao/core/blob/...t.php#L73-L117

[webstar]

Aber das macht die Methode doch schon. Deswegen benutze ich doch die Input-Klasse. Oder?
https://github.com/contao/core/blob/...t.php#L73-L117

Nein. Geht ja auch garnicht. Es ist ja garnicht klar, dass die Daten aus der Input-Klasse in die DB kommen. Auch weiß man dort garnicht, welche DB verwendet wird. Zwar führt die Input-Klasse eine Filterung durch, aber erfodert dennoch, dass man nicht an anderer Stelle leichtfertig arbeitet.

Normalerweise übernimmt die Statement-Klasse der Datenbank das Escapen, weshalb man halt §db->prepare('SELECT id FROM tl_table WHERE foo=?')->execute('bar') macht.

[zonky]

Du kannst die Datei übrigens auch .php anstatt .html5 nennen. .html5 hat nur den Vorteil (oder Nachteil) dass du sie im BE unter den Templates siehst.

.php kann man auch für Ansicht/Bearbeitung "freischalten" indem man die Extension in der "Systemsteuerung" von Contao einträgt

[Andreas]

Nein. Geht ja auch garnicht.

Ok, danke für den Hinweis. Also kann man sagen, man soll die GETs daraufhin überprüfen, ob da auch das kommt, was man erwartet. Richtig?

[webstar]

Ok, danke für den Hinweis. Also kann man sagen, man soll die GETs daraufhin überprüfen, ob da auch das kommt, was man erwartet. Richtig?

Ja, man sollte die ersten drei Grundsätze der (Web-)programmierung beachten:

1. Never trust user data.
2. Never trust user data.
3. Never trust user data.

http://i.stack.imgur.com/XTknC.png