RobotLoader::addClass – nespravna vynimka pri kolizii nazvov tried

V RobotLoader (Nette 1.0-dev 64b5b90 z 2010–09–17 pre PHP 5.2 bez prefixov) je jeden preklep a dve vaznejsie chyby.

1. preklep RobotLoader::addClass: $class → $lClass na konci vyvolania InvalidStateException:

<?php
...
$lClass = strtolower($class);
if (!empty($this->list[$lClass]) && $this->list[$lClass][0] !== $file && is_file($this->list[$lClass][0])) {
	spl_autoload_call($class); // hack: enables exceptions
	//throw new InvalidStateException("Ambiguous class '$class' resolution; defined in $file and in " . $this->list[$class][0] . ".");
	throw new InvalidStateException("Ambiguous class '$class' resolution; defined in $file and in " . $this->list[$lClass][0] . ".");
}
...
?>

2. Vacsi problem je vsak ak sa vyskytne kolizia nazvov tried.

Konkretny priklad: subor nSMTPMailer/exceptions.php z nSMTPMailera definuje InvalidStateException a IOException, ktore su definuje aj Nette v Utils/exceptions.php.

Spravne by mal addClass vyhodit InvalidStateException s informaciou o duplicite tychto tried v oboch suboroch. Bohuzial (nezmyselnu) vynimku vyhodi uz spl_autoload_call:

Fatal error: Cannot redeclare class ArgumentOutOfRangeException in …\Nette\Utils\exceptions.php on line 39

Trieda ArgumentOutOfRangeException vsak nema koliziu (zobrazuje sa pravdepodobne preto, lebo je prva v Nette\Utils\exceptions.php). Potom da dost roboty najst pricinu problemu.

Je nutne volat v addClass pri zisteni kolizii nazvov tried pred vyhodenim vynimky spl_autoload_call?

Poznamky:
Ked som zakomentoval deklaraciu InvalidStateException v nSMTPMailer/exceptions.php (zostala duplicitna IOException), vyhodila sa (podla ocakavania) rovnaka nezmyselna vynimka.

Ked som vsak zakomentoval aj IOException a doplnil do oboch suborov definiciu TestException, vyhodila sa (nechapem preco) spravna vynimka InvalidStateException.

3. Nefunguju vynimky pre RobotLoader v subore netterobots.txt. Problem je niekde v RobotLoader::scanDirectory, pravdepodobne pri definovani filtra pre Finder.

<?php
...
$iterator->append(Finder::find(String::split($this->acceptFiles, '#[,\s]+#'))
    ->from($dir)
    ->exclude(String::split($this->ignoreDirs, '#[,\s]+#'))
    ->filter(create_function('$file', 'list($disallow) = NClosureFix::$vars['.NClosureFix::uses(array(&$disallow)).'];
        return !isset($disallow[$file->getPathname()]);
    '))->getIterator()
);
...
?>

Pole $disallow sa naplni spravne, ale Finder porovnava aktualnu polozku (subor, ci adresar) s obsahom pola $disallow len pre adresare.

Chyby c. 1 a 3 su nove v Nette 1.0 (v 0.9.5 neboli). Chyba c. 2 je stara, ale tam sa vdaka fungujucemu netterobots.txt pri nSMTPMailer-i neprejavi.
Sorry, ze som sa tak rozkecal ;-)