Bug v Nette/Database/Selection::createJoins

Upozornění: Tohle vlákno je hodně staré a informace nemusí být platné pro současné Nette.
martincohen
Člen | 14
+
0
-

Cau kuci,

zrovna som objavil chybku v Nette/Database/Selection (2.0-beta ze dne 28. 05. 2011) vo funkcii createJoins:

$joins[$name] = ' ' . (!isset($joins[$name]) && $inner && !isset($match[3]) ? 'INNER' : 'LEFT')
	. ' JOIN ' . $supplementalDriver->delimite($table)
	. ' AS '. $supplementalDriver->delimite($table)
	. ($table !== $name ? ' AS ' . $supplementalDriver->delimite($name) : '')
	. " ON $this->delimitedName." . $supplementalDriver->delimite($column)
	. ' = ' . $supplementalDriver->delimite($name) . '.' . $supplementalDriver->delimite($primary);

Riadok 3 sposobi chybu v SQL dotaze, za predpokladu ze $table !== $name (r. 4) a vygeneruje nieco ako JOIN users AS users AS author.

Je to chyba alebo som nieco nepochopil?

hrach
Člen | 1838
+
0
-

Cau, je to chyba ktera vznikla asi spatnym mergem. :( Je to vec, kterou jsem pulloval a ktera neni jeste dokoncena. Srr!

martincohen
Člen | 14
+
0
-

@hrach Aha, no tak ok. Mne to ale zatial asi funguje (po odstraneni toho riadku). Mas nejake varovanie co presne nefunguje (okrem tohto bugu)?

hrach
Člen | 1838
+
0
-

Myslim, ze to by mel byt jediny problem.
Tedy, krom tri bugu, ktery opravuji posledni 3 commity. Zbyly commity byly prijmuty, nebo prave neprijaty. https://github.com/…tte/pull/266

martincohen
Člen | 14
+
0
-

@hrach Ok, dik za odpoved. <3

David Grudl
Nette Core | 8228
+
0
-

Máte někdo patch?

hrach
Člen | 1838
+
0
-

Davide, mergni tam prosim ty posledni tri commity, pak ten pull request zavru a udelam novy na jednotlivy veci. Diky

David Grudl
Nette Core | 8228
+
0
-

Je to tam.