Seite 2 von 3

Verfasst: Sa 16. Okt 2004, 16:33
von Halchteranerin
Fragen Sie jemanden, der sich damit auskennt. :lol:
Es ist natuerlich praktisch, wenn man weiss, wo man suchen soll, und nicht im Nebel herumstochern muss. :wink: Mit den Aenderungen hat's bei mir auch geklappt, danke!

Verfasst: Do 13. Jan 2005, 19:51
von emergence
bin ja schon wieder auf den bug genagelt...
-> http://www.contenido.org/forum/viewtopi ... 2125#32125

Verfasst: Di 18. Jan 2005, 13:44
von 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.

Verfasst: Di 18. Jan 2005, 13:46
von emergence

Verfasst: Di 18. Jan 2005, 14:09
von timo
ja

Verfasst: Di 18. Jan 2005, 16:20
von 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

Verfasst: Di 18. Jan 2005, 16:50
von timo
also die Meldung bekomme ich nicht...deshalb hab ich das auch nicht eingebaut?

Verfasst: Di 18. Jan 2005, 16:53
von 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)

Verfasst: Di 18. Jan 2005, 17:16
von timo
ok ist eingebaut

Verfasst: Di 18. Jan 2005, 19:24
von 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

Verfasst: Di 18. Jan 2005, 19:43
von 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...

Verfasst: Di 18. Jan 2005, 19:51
von 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();

Verfasst: Di 18. Jan 2005, 19:54
von HerrB
:oops:

Gruß
HerrB

Verfasst: Di 18. Jan 2005, 20:20
von emergence
ach ja ne aussagekräftige beschreibung sollte man noch dafür wählen...
ich hab das kommentar ding einfach kopiert...

Verfasst: Mi 19. Jan 2005, 00:13
von 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