search

flame-org / Modules

Nette modules on the Steroids
http://flame-org.github.io/Modules/
GNU Lesser General Public License v3.0
13 stars 7 forks source link

Nelze mít více nepojmenovaných modulů #4

Closed jirinapravnik closed 10 years ago

jirinapravnik commented 10 years ago

Po commitu 375c4d1d8438ccd8306dd62217f8603a599c9843 mi přestaly fungovat nepojmenované moduly. Problém jsem vyzjistil je v souboru NamedExtension a metody: getShortName . Jelikož v podstatě nedělá co by měla, a když je více nepojmenovaných modulů, tak všechny mají ten první název díky statické property. Mě pomohlo ten if zakomentovat nebo vyodit statičnost, ale třeba s tím máš nějaké jiné úmysly a někde je to více provázano

jsifalda commented 10 years ago

Tohle je klasický bug. Naivně jsem veřil, že ušetřím nějaký malý výkon a neotestoval jsem si to. Commit https://github.com/flame-org/Modules/commit/331c55bf359d5ef3828087ee8a4e1cb1b2ac8d8c by to měl spravit. A hlavně, měl bych si najít čas na napsaní testů:X Omlouvám se.

jirinapravnik commented 10 years ago

Díky, to je hodně rychlý fix:)

Není proč se omlouvat, já jsem rád, že vlbec něco takovýho je a tak občas nějaká chybka nevadí:)