DiscoveredReflection: join přes dvě tabulky v select

Ahoj. Díky za odpověď!

Tak to funguje s ConventionalReflection, ale ne s DiscoveredReflection. Tedy kupodivu mi to v tomto modelovém příkladu zafungovalo i s Discovered, ale problém je v tom, že ten sloupec s cizím klíčem se ve skutečnosti nejmenuje country_id, ale nějak libovolně, např fk_cntr_id. A pak už to tímto zápisem nejde.

Díky!

Převzatou databázi si nevybereš. ;-) Mě to taky netěší.

Jak to teda funguje? Když ten cizí klíč končí na _id, tak to napíšu bez _id; a když nekončí na _id, tak napíšu jeho celý název? Proč má to _id výjimku?

Perfektní. Díky moc za vysvětlení. Proč mi ale tedy nefunguje to, co mám v původní otázce?

Totiž, teď si s tím hraju a toto funguje:

$row->ref('fk_id_athr')->ref('fk_id_cntr')->name;

Zatímco toto nefunguje:

$row->fk_id_athr->fk_id_cntr->name;

Pre lepšie zorientovanie môžeš sem napísať sql dump? Minimálne všetky FK, PK a názvy tabuliek v ktorých sa nachádzajú.

A co toto?

    ->select('book.id_athr.id_cntr.name');

Mozna tam bude chyba, vim ze sem neco opravoval a jeste to nebylo mergnuto, pac to bylo ve vetsim rfc.

->select('athr.cntr.name');

?

edit: :D kto preboha vymýšľal tie názvy FK?

edit2: a ak takto upravíme to od hracha?

    ->select('id_athr.id_cntr.name');

Editoval vvoody (21. 12. 2012 1:47)

Myslím že som našiel bug v SqlBuilder->buildJoins().

Testoval som na kode:

$table = $c->table('book')->select('book.*, athr.name AS aname, athr.cntr.name AS cname');
foreach ($table as $row) {
	dump($row);
}

a dátach:

DROP TABLE IF EXISTS `author`;
CREATE TABLE `author` (
  `id` int(11) NOT NULL AUTO_INCREMENT,
  `name` varchar(255) NOT NULL,
  `fk_id_cntr` int(11) NOT NULL,
  PRIMARY KEY (`id`),
  KEY `fk_id_cntr` (`fk_id_cntr`),
  CONSTRAINT `author_ibfk_1` FOREIGN KEY (`fk_id_cntr`) REFERENCES `country` (`id`)
) ENGINE=InnoDB AUTO_INCREMENT=2 DEFAULT CHARSET=latin1;

INSERT INTO `author` (`id`, `name`, `fk_id_cntr`) VALUES
(1,	'Pišta',	1);

DROP TABLE IF EXISTS `book`;
CREATE TABLE `book` (
  `id` int(11) NOT NULL AUTO_INCREMENT,
  `name` varchar(255) NOT NULL,
  `fk_id_athr` int(11) NOT NULL,
  PRIMARY KEY (`id`),
  KEY `fk_id_athr` (`fk_id_athr`),
  CONSTRAINT `book_ibfk_1` FOREIGN KEY (`fk_id_athr`) REFERENCES `author` (`id`)
) ENGINE=InnoDB AUTO_INCREMENT=2 DEFAULT CHARSET=latin1;

INSERT INTO `book` (`id`, `name`, `fk_id_athr`) VALUES
(1,	'Islo vajce na vandrovku',	1);

DROP TABLE IF EXISTS `country`;
CREATE TABLE `country` (
  `id` int(11) NOT NULL AUTO_INCREMENT,
  `name` varchar(255) NOT NULL,
  PRIMARY KEY (`id`)
) ENGINE=InnoDB AUTO_INCREMENT=2 DEFAULT CHARSET=latin1;

INSERT INTO `country` (`id`, `name`) VALUES
(1,	'Slovensko');

Problém bol celkom jasný: No reference found for $athr->cntr
Prečo metóda $reflection->getBelongsToReference($parent, $name) dostala ako parameter $paren = ‚athr‘ namiesto správneho ‚author‘. Tieto úpravy mi to fixli, ale netrufam si povedať či to niečo ine nerozoserie.

protected function buildJoins($val, $inner = FALSE)
{
	$reflection = $this->selection->getConnection()->getDatabaseReflection();
	$joins = array();
	preg_match_all('~\\b([a-z][\\w.:]*[.:])([a-z]\\w*|\*)(\\s+IS\\b|\\s*<=>)?~i', $val, $matches);
	foreach ($matches[1] as $names) {
		// v prvej iteracii bude parent a alias v ON podmienke totozny
		$alias = $parent = $this->selection->getName();

		if ($names !== "$parent.") { // case-sensitive
			preg_match_all('~\\b([a-z][\\w]*|\*)([.:])~i', $names, $matches, PREG_SET_ORDER);
			foreach ($matches as $match) {
				list(, $name, $delimiter) = $match;

				if ($delimiter === ':') {
					list($table, $primary) = $reflection->getHasManyReference($parent, $name);
					$column = $reflection->getPrimary($parent);
				} else {
					list($table, $column) = $reflection->getBelongsToReference($parent, $name);
					$primary = $reflection->getPrimary($table);
				}

				$joins[$name] = ' '
					. (!isset($joins[$name]) && $inner && !isset($match[3]) ? 'INNER' : 'LEFT')
					. ' JOIN ' . $this->driver->delimite($table) . ($table !== $name ? ' AS ' . $this->driver->delimite($name) : '')
					// before:
					// . ' ON ' . $this->driver->delimite($parent) . '.' . $this->driver->delimite($column)
					// after
					. ' ON ' . $this->driver->delimite($alias) . '.' . $this->driver->delimite($column)
					. ' = ' . $this->driver->delimite($name) . '.' . $this->driver->delimite($primary);
				// treba rozlisovat realny nazov tabulky potreby pre reflection od aliasu v ON podmienke
				$alias = $name;
				$parent = $table;
			}
		}
	}
	return $joins;
}

Aha. Díky moc, pánové. Myslíte, že se to někdy v dohledné době dostane do Nette, nebo si to tam mám přidat ručně?

Upraviť Nette určite neupravuj. Úpravu db zrejme nechceš na rozbehnutom projekte. Posledná možnosť (než bude fix v Nette) je si napísať vlastný reflection ktorý bude presne vyhovovať tvojej db a aj tomu bugu aby sa neprejavil :D

Je to tam. Tak děkuju moc ještě jednou.