Bug in Contenido_FrontendNavigation::isActiveChild

[Oldperl]

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

[_wiewo_]

ist nur das return true der aussteig aus der schleife oder auch nen return false?

[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

[Oldperl]

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

[_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

[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;
	}

[rbi]

Dank an Rocco für den Hinweis, der Bug ist im SVN bereits gefixt.

[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.

[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.