anmerkung: snapshot contenido-cvs-2004-10-08.tar

[Halchteranerin]

Fragen Sie jemanden, der sich damit auskennt.
Es ist natuerlich praktisch, wenn man weiss, wo man suchen soll, und nicht im Nebel herumstochern muss. Mit den Aenderungen hat's bei mir auch geklappt, danke!

[emergence]

bin ja schon wieder auf den bug genagelt...
-> http://www.contenido.org/forum/viewtopi ... 2125#32125

[timo]

habe beide offenen Punkte soeben gelöst

das mit dem strtolower(get_class($this)) muß sein, da PHP (warum auch immer) in PHP4 alle Klassennamen als lowercase zurückliefert, PHP5 aber nicht:

Note: A class defined in a PHP extension is returned in its original notation. In PHP 4 get_class() returns a user defined class name in lowercase, but in PHP 5 it will return the class name in it's original notation too, just like class names from PHP extensions.

[emergence]

ähm das auch korrigiert ?
-> http://www.contenido.org/forum/viewtopi ... 2575#32575

[timo]

ja

[emergence]

muss ich leider entsperren ist nicht im cvs enthalten

Warning: array_merge() [function.array-merge]: Argument #2 is not an array in e:\contenido\contenido-cvs-2005-01-18.tar\contenido_sources\contenido\classes\class.genericdb.php on line 594

Warning: implode() [function.implode]: Bad arguments. in e:\contenido\contenido-cvs-2005-01-18.tar\contenido_sources\contenido\classes\class.genericdb.php on line 594


-> include.frontend.user_menu.php

nach

Code: Alles auswählen

	$oFEMemberCollection = new FrontendUserCollection;
	$oFEMemberCollection->setWhere('idclient', $client);
	$oFEMemberCollection->setOrder('active DESC, username');

folgendes ergänzen

Code: Alles auswählen

	$oFEMemberCollection->addResultField('FrontendUserCollection.username');
	$oFEMemberCollection->addResultField('FrontendUserCollection.active');

dann gehts

alles andere ist drinnen

[timo]

also die Meldung bekomme ich nicht...deshalb hab ich das auch nicht eingebaut?

[emergence]

ähm... hmm ich kann nur sagen das ich bei php5.0.1 diese meldung bekomme...

da ich mir die genericdb noch nicht wirklich genau angesehen habe, kann ich da auch nichts weiters dazu sagen... (ich drück mich da noch immer)

[timo]

ok ist eingebaut

[HerrB]

Als Laie noch eine Anmerkung (ich mache auch wieder zu): Der eigentliche Fehler dürfte woanders sitzen (gefunden habe ich ihn freilich nicht, deswegen habe ich auch bisher nix angemerkt).

Welche Felder ausgewählt werden, ist im aktuellen Code egal, denn das verwendete next() in der Funktion verweist auf loadItem der FrontendUserCollection und die lädt mit $item->loadByPrimaryKey($itemID); alle Felder der Tabelle. D.h. jede Auswahl individueller Felder wird beim ersten next() wieder durch alle Felder ersetzt.

Da es bei mir (PHP 4.3.10) funktioniert: Kannst Du mal testen, ob es funktioniert, wenn Du nur eine Zeile ergänzt? Danke.

Gruß
HerrB

[emergence]

ähm sehr guter einwand...

habs gerade getestet
der fehler tritt nur auf wenn keine der beiden zeilen vorhanden ist.
ist eine der beiden(egal welche) vorhanden tritt es nicht auf...

ich kann auch nur folgendes anstelle der beiden zeilen hinschreiben

Code: Alles auswählen

$oFEMemberCollection->addResultField('foo');

dann gehts auch...

ich hab ne idee woran das liegt...

[emergence]

guter mann HerrB

simpler fix

die beiden zeilen können rausfliegen...

in classes/class.genericdb.php

am beginn der klasse reicht es zB das nach

Code: Alles auswählen

	/**
	 * @var array Inner group conditions
	 * @access private
	 */
	var $_innerGroupConditions = array();

einfach folgendes ergänzt wird

Code: Alles auswählen

	/**
	 * @var array Inner group conditions
	 * @access private
	 */
	var $_resultFields = array();

[HerrB]

Gruß
HerrB

[emergence]

ach ja ne aussagekräftige beschreibung sollte man noch dafür wählen...
ich hab das kommentar ding einfach kopiert...

[emergence]

das in der genericdb.php ist hier bereits geändert
-> http://www.contenido.org/forum/viewtopi ... 8874#38874

das file mit allen änderungen (cvs newsletter von HerrB)
-> http://www.contenido.org/forum/viewtopi ... 8913#38913