Nette\Database\Table\Selection metoda group HAVING s LIKE

czita

Sestavuji SQL dotaz přes Nette\Database\Table\Selection. Ukázkový kód:

$table->group("sloupec", "sloupec LIKE '%vyraz%')

Vygeneruje chybně podmínku HAVING klauzuli:

GROUP BY sloupec HAVING `sloupec` LIKE '%`vyraz`%'

ViPEr*CZ*

Tak to není chyba jen u group. Blbne to i u where.

$where = "sloupec LIKE '%vyraz%';
$table->where($where);

Vygeneruje toto:

WHERE `sloupec` LIKE '%`vyraz`%'

Takhle se to chová na poslední verzi 2.0.1.

Editoval ViPEr*CZ* (2. 3. 2012 20:39)

ViPEr*CZ*

V nightly build to je již opraveno, kdyby na to někdo taky narazil ;-)

ViPEr*CZ*

Tak bohužel to není opravené vše. Například toto mi stále špatně escapuje:

REGEXP LOWER('^`li`')

ViPEr*CZ*

Jenom pro další zajímavost… jak si s tím tak hraju, tak s velkýma písmenkama to je normal:

REGEXP LOWER('^LI'))

ViPEr*CZ*

Nejsem si zcela jist, ale viděl bych to na tuto úpravu?

Slection.php

protected function tryDelimite($s)
{
	$driver = $this->connection->getSupplementalDriver();
	return preg_replace_callback('#(?<=[^\w`\'"\[]|^)[a-z_][a-z0-9_]*(?=[^\w`\'"(\]]|$)#i', function($m) use ($driver) {
		return strtoupper($m[0]) === $m[0] ? $m[0] : $driver->delimite($m[0]);
	}, $s);
}

Editoval ViPEr*CZ* (4. 3. 2012 10:56)

David Grudl

Řekl bych, že není vhodné ve výrazech používaných u Table\Selection vůbec používat řetězce. Ty by měly být oddělené od SQL a předávány jako parametry pomocí otazníku. Tedy $table->where('sloupec LIKE ?', $vyraz).

hrach

Coz neplati pro group!

ViPEr*CZ*

To je jedna věc. A ani tak to nefunguje 100%. Teď jsem třeba narazil na todle:

SELECT * FROM `contact` WHERE (`typeID` = ?) AND (`parentID` IS NULL) AND (`dateDeleted` IS NULL) AND (`name` IN (?)) ORDER BY `name` ASC LIMIT 20 OFFSET 0

A parametr pro test posílám: 1 a „Obi,Pokus“ (9)
A najde mi to prázdný výsledek což je špatně. Pokud místo otazníku pro IN pošlu takto apostrofem upravený string IN('Obi', 'Pokus') tak to najde dva řádky což je dobře.
Čekal bych, že to bude možné posílat bez otazníku přes WHERE, pokud je to povoleno.

Editoval ViPEr*CZ* (9. 3. 2012 9:46)

hrach

@ViPEr: neposilej tam string, ale array, ne? array("Obi", "pokus")

ViPEr*CZ*

@hrach: Právě tak jsem to tam posílal:

$test = array("Pokus", "Pokus2");
...->table('tabulka')->where("name IN(?)", $test);

A toto mi neprošlo. Trochu krkolomě jsem to poslal, za což se omlouvám. Už jsem si to přepsal, aby mi tydle složené WHERy chodily přes zástupný znak ?, ale u IN jsem to bohužel zatím nechal přes řetězec, takže si to pole převádím zpět přes implode(„,“, $params);
A na to GROUP jsi už koukal?

Editoval ViPEr*CZ* (14. 3. 2012 10:45)

uestla

Stačit by mělo

$test = array("Pokus", "Pokus2");
...->table('tabulka')->where("name", $test);

Nicméně ta nefunkčnost HAVING + LIKE mě taky trápí… Zkoušel jsem vše, ale všechno selže právě na https://api.nette.org/…ion.php.html#533 (tryDelimite() mi to odelimituje nepěkně).

Bude velký oříšek přidendat podporu pro HAVING s parametry?

petr.pavel

Re HAVING: nejde třeba použít NSqlLiteral? Nezkoušel jsem.

$table->group("sloupec", new NSqlLiteral("sloupec LIKE '%vyraz%'"))

uestla

Výsledek je bohužel totožný :(

Odelimitování se totiž provádí za jakéhokoli počasí.

Editoval uestla (8. 7. 2012 11:35)

hrach

Ja bych to tryDelimite zrusil proste uplne, toto je Daviduv boj…

uestla

Jen se pro jistotu otáži – SQL injection se ošetřuje pak ještě na interní bázi PDO, který si příkaz proparsuje a rozescapuje?

ViPEr*CZ*

hrach napsal(a):

Ja bych to tryDelimite zrusil proste uplne, toto je Daviduv boj…

Tak v některých případech to ulehčuje práci. Když už tak to možná nějak vymyslet, aby to šlo vypnout či nějak obejít… třeba, tak že co si označím mezi ‚…‘ se nebude rozparsovávat a dále označovat do , aby to pak nedopadlo takto LIKE '%vyraz`%‚, ale takto LIKE '%vyraz%‘. Nebo podobně jako to je v dibi, že se mi nastaví '' tam kde je [].

peterpp

ViPErCZ napsal(a):

Nejsem si zcela jist, ale viděl bych to na tuto úpravu?

Slection.php

protected function tryDelimite($s)
{
	$driver = $this->connection->getSupplementalDriver();
	return preg_replace_callback('#(?<=[^\w`\'"\[]|^)[a-z_][a-z0-9_]*(?=[^\w`\'"(\]]|$)#i', function($m) use ($driver) {
		return strtoupper($m[0]) === $m[0] ? $m[0] : $driver->delimite($m[0]);
	}, $s);
}

Tato uprava je podla mna v poriadku a priam nutna. Pokial regurany vyraz venechava retazce escapovane pomocou uvodzoviek a spetneho apostrofu, tak potom musi brat do uvahy i klasicky apostrof.

peterpp

hrach napsal(a):

Ja bych to tryDelimite zrusil proste uplne, toto je Daviduv boj…

Suhlasim. Funkcia tryDelimite je skutocne zlo. Okrem toho, ze neberie do uvahy uz escapovane hodnoty v apostrofoch, tak to ma problemy i s pomlckami v hodnotach. Napr. z:

where `a` = "usti-nad-labem"

to vyrobi

where `a` = "usti-`nad`-labem"

Editoval peterpp (12. 9. 2012 11:46)

Prochy20

BUMP!

Ahoj,

mám ten samý problém, tentokrát s operátorem „LIKE“ viz:

public function getByCondition($condition){
	return $this->table("tabulka")->where("condition LIKE '%".$condition."%'");
}

Vrácený výsledek:

"SELECT `id` FROM `tabulka` WHERE (`condition` LIKE '%`mmm02`%')"

tato metoda mi občas vrátí korektní výsledek a občas ne. Má někdo z Vás nějaký workaround či opravu pro tuto funkci, resp. řešení?

Děkuji,

Prochy20.

EDIT: Přidán příklad výsledku.

Editoval Prochy20 (28. 8. 2013 12:43)

enumag

@Prochy20: Tohle nepomůže?

return $this->table("tabulka")->where("condition LIKE ?", "%" . $condition . "%");

Prochy20

@enumag:

Nepomohlo –

upravil jsem metodu takto:

public function getByCondition($condition){
	return $this->table("tabulka")->where("condition LIKE ?", "%" . $condition . "%");
}

potom jsem dostal následující výsledek:

"SELECT `id` FROM `tabulka` WHERE (`condition` LIKE ?)";

Nicméně jsem ověřil, že je pravda co psal ViPErCZ – je skutečně problém s case sensitivitou. Napíše-li se string pomocí UCase vše funguje; LCase nikoliv.

Prochy20

Editoval Prochy20 (28. 8. 2013 13:00)

enumag

@Prochy20: Ten otazník je tam správně, to si nahradí až PDO. Podstatné je zda to vrátí správné řádky.

Editoval enumag (28. 8. 2013 13:10)

Prochy20

@enumag: ahh… tak na toto jsem zapomněl. Ano, už to funguje.

Děkuji,

Prochy20

MartinitCZ

Není tento LIKE %% zápis trochu větší díra do systému?