Seite 2 von 2
Re: Bug in Contenido_FrontendNavigation::isActiveChild
Verfasst: Mi 29. Apr 2009, 08:33
von Oldperl
idea-tec hat geschrieben:
ich würde dann allerdings auf jeden fall direkt hinter das return ein exit setzen, weil ich ja mein "ergebnis" habe und auf keinen fall will, dass das ding weiter da durchrauscht.
vor allem könnte es ja auch sein, dass ich das ergebnis nach der 4. von 120 schleifen habe und dann rauscht das noch weitere 116 schleifen da durch, ohne sinn und zweck
Diese Aussage ist unlogisch, da durch ein return true die Schleife abgebrochen wird und keine weitere Schleife aufgerufen wird. Daher ist dort kein exit notwendig, das return true ist quasi das exit aus der Funktion.
Gruß aus Franken
Ortwin
Re: Bug in Contenido_FrontendNavigation::isActiveChild
Verfasst: Mi 29. Apr 2009, 08:51
von _wiewo_
ist nur das return true der aussteig aus der schleife oder auch nen return false?
Re: Bug in Contenido_FrontendNavigation::isActiveChild
Verfasst: Mi 29. Apr 2009, 08:53
von idea-tec
da sieht man es mal wieder, da lerne ich doch glatt noch was dazu, und das nur weil ich zu faul zum nachlesen bin
Re: Bug in Contenido_FrontendNavigation::isActiveChild
Verfasst: Mi 29. Apr 2009, 08:59
von Oldperl
_wiewo_ hat geschrieben:ist nur das return true der aussteig aus der schleife oder auch nen return false?
Das "return" ist der Ausstieg, was das return zurück gibt ist in diesem Falle (für den Ausstieg) erst mal egal.
Gruß aus Franken
Ortwin
Re: Bug in Contenido_FrontendNavigation::isActiveChild
Verfasst: Mi 29. Apr 2009, 09:03
von _wiewo_
okay, dann wäre ja die vorgeschlagene lösung wirklich humbug
weil da würd er ja sofrt aussteigen
selbst wenns kein treffer war
Re: Bug in Contenido_FrontendNavigation::isActiveChild
Verfasst: Mi 29. Apr 2009, 09:16
von sier
hallo zusammen
es gibt tage an denen gehöre ich zu den "faulen programmieren". heute hingegen habe mir etwas die zeit genommen.... da ich gesehen habe dass doch der eine oder andere daran interesse hat. den cast auf int und die abfrage auf oCategory->getSubCategories()->count habe ich entfernt da das erste zeit kostet (ab 1. recursion kostet es nur noch unötig zeit wenn man sauber programmiert hat
). die count() abfrage würde nur dann etwas bringen wenn die unterkategorien nicht geladen werden. es müsste also eine methode Contenido_Category::getSubCategoriesCount() geben. also nachfolgend meine final-version. gruss rocco
Code: Alles auswählen
public function isActiveChild(Contenido_Category $oCategory, $iCurrentIdcat) {
$oChildCategories = $oCategory->getSubCategories();
foreach ($oChildCategories as $oChildCat) {
if ($oChildCat->getIdCat() == $iCurrentIdcat ||
$this->isActiveChild($oChildCat, $iCurrentIdcat)) {
return true;
}
}
return false;
}
Re: Bug in Contenido_FrontendNavigation::isActiveChild
Verfasst: Mi 29. Apr 2009, 09:55
von rbi
Dank an Rocco für den Hinweis, der Bug ist im SVN bereits gefixt.
Re: Bug in Contenido_FrontendNavigation::isActiveChild
Verfasst: Mi 29. Apr 2009, 12:36
von sier
[quote="rbi"]... der Bug ist im SVN bereits gefixt.[/quote]
cool, ich bin neu hier ... deshalb an dieser stelle gleich noch zwei Fragen.
* gibt es einen Zugang zu diesem SVN?
* gibt es ein öffentlich zugängliches Bug-Tracking für Contenido?
Meine Firma wird contenido in Zukunft strategisch einsetzen. SVN und Bug-Tracking wäre dann schon eine feine Sache.
Re: Bug in Contenido_FrontendNavigation::isActiveChild
Verfasst: Mo 4. Mai 2009, 12:56
von timo.trautmann_4fb
Es gibt ein SVN und auch ein Bugtragging System für Contenido, diese sind aber nicht öffentlich zugänglich. Wenn es Probleme gibt, können diese im Forum gemeldet werden und werden dann meist durch 4fb oder einige authorisierte Forenuser gefixxt. Diese haben sowohl auf das SVN als auch auf ein Bugracking Tool Zugriff. Natürlich suchen wir immer Verstärkung im Team, wenn du also Lust hast an Contenido aktiv mitzuwirken freuen wir uns natürlich über deine Kontaktaufnahme. Alternativ kannst du natürlich gerne auftretende Probleme hier im Forum besprechen.