Seite 2 von 3
Verfasst: Sa 16. Okt 2004, 16:33
von 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!
Verfasst: Do 13. Jan 2005, 19:51
von emergence
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
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
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