Fehler in "categories.php"

[Michael_Warmuth]

Liebe Programmiererinnen und Programmierer!

In „categories.php“ gibt es ein kleines Problem. Es wird in Zeile 35 eine Schleife gestartet mit:

…
foreach($categories as $cat)
{
if($cat[‘data’][‘color’])
{
$content .= “.cat_{$cat[‘id’]} { background-color: {$cat[‘data’][‘color’]};} /{$cat[‘name’]}/\n”;
}
if($cat[‘data’][‘icon’])
{
$content .= “.cat_{$cat[‘id’]} .cat_icon { background-image: url(’”. admin_categories::icon_url($cat[‘data’][‘icon’]) ."’);} /{$cat[‘name’]}/$
}
}
…

Das kleine Problem ist, dass $categories sichtlich leer sein kann (ist es bei mir), womit die „foreach“ Anweisung einen Fehler bringt:

„… PHP Warning: Invalid argument supplied for foreach() in /usr/share/egroupware/phpgwapi/categories.php on line 35, …"

Man sollte daher vorher zumindest überprüfen, ob $categories überhaupt ein Array ist:

…
// MW: Get all categories - if there are any
if (is_array($categories))
{
foreach($categories as $cat)
{
if($cat[‘data’][‘color’])
{
$content .= “.cat_{$cat[‘id’]} { background-color: {$cat[‘data’][‘color’]};} /{$cat[‘name’]}/\n”;
}
if($cat[‘data’][‘icon’])
{
$content .= “.cat_{$cat[‘id’]} .cat_icon { background-image: url(’”. admin_categories::icon_url($cat[‘data’][‘icon’]) ."’);} /{$cat[‘name’]}/$
}
}
}
…

Systeminformation:
System: Linux … 3.16.0-4-amd64 #1 SMP Debian 3.16.7-ckt20-1+deb8u4 (2016-02-29) x86_64
egroupware-epl: 14.3.20160304

Liebe Grüße
Michael

Transform Data into Opportunity.
Accelerate data analysis in your applications with
Intel Data Analytics Acceleration Library.
Click to learn more.
http://pubads.g.doubleclick.net/gampad/clk?id=278785351&iu=/4140

---

egroupware-german mailing list
egroupware-german@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/egroupware-german

[Ralf_Becker_Stylite]

Hallo Michael,

ich hab’ mir das mal genau angesehen (nachdem ich meine egw_categories
komplett geleert und den Instanzcache unter Admin >> Cache löschen und
Hooks registrieren gelöscht habe):

Die verwendete return_array Methode liefert immer ein - evtl. leeres -
Array, dementsprechend bekomme ich kein PHP Warning.

Kann es sein, dass Du einen eigenen Patch in der
phpgwapi/inc/class.categories.inc.php hast?

Ralf

Liebe Programmiererinnen und Programmierer!

In „categories.php“ gibt es ein kleines Problem. Es wird in Zeile 35 eine Schleife gestartet mit:

…
foreach($categories as $cat)
{
if($cat[‘data’][‘color’])
{
$content .= “.cat_{$cat[‘id’]} { background-color: {$cat[‘data’][‘color’]};} /{$cat[‘name’]}/\n”;
}
if($cat[‘data’][‘icon’])
{
$content .= “.cat_{$cat[‘id’]} .cat_icon { background-image: url(’”. admin_categories::icon_url($cat[‘data’][‘icon’]) ."’);} /{$cat[‘name’]}/$
}
}
…

Das kleine Problem ist, dass $categories sichtlich leer sein kann (ist es bei mir), womit die „foreach“ Anweisung einen Fehler bringt:

„… PHP Warning: Invalid argument supplied for foreach() in /usr/share/egroupware/phpgwapi/categories.php on line 35, …"

Man sollte daher vorher zumindest überprüfen, ob $categories überhaupt ein Array ist:

…
// MW: Get all categories - if there are any
if (is_array($categories))
{
foreach($categories as $cat)
{
if($cat[‘data’][‘color’])
{
$content .= “.cat_{$cat[‘id’]} { background-color: {$cat[‘data’][‘color’]};} /{$cat[‘name’]}/\n”;
}
if($cat[‘data’][‘icon’])
{
$content .= “.cat_{$cat[‘id’]} .cat_icon { background-image: url(’”. admin_categories::icon_url($cat[‘data’][‘icon’]) ."’);} /{$cat[‘name’]}/$
}
}
}
…

Systeminformation:
System: Linux … 3.16.0-4-amd64 #1 SMP Debian 3.16.7-ckt20-1+deb8u4 (2016-02-29) x86_64
egroupware-epl: 14.3.20160304

Liebe Grüße
Michael

Transform Data into Opportunity.
Accelerate data analysis in your applications with
Intel Data Analytics Acceleration Library.
Click to learn more.
http://pubads.g.doubleclick.net/gampad/clk?id=278785351&iu=/4140

---

egroupware-german mailing list
egroupware-german@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/egroupware-german

–
Ralf Becker
Director Software Development

Stylite AG

Isaac-Fulda-Allee 9 | Tel. +49 6131 32702-0
D-55124 Mainz | Fax. +49 6131 32702-70

Email: rb@stylite.de

www.stylite.de | www.egroupware.org

Managing Directors: Andre Keller | Ralf Becker | Gudrun Mueller
Chairman of the supervisory board: Prof. Dr. Birger Leon Kropshofer

VAT DE214280951 | Registered HRB 46224 Mainz Germany

[Michael_Warmuth]

Lieber Ralf!

ich hab’ mir das mal genau angesehen (nachdem ich meine egw_categories
komplett geleert und den Instanzcache unter Admin >> Cache löschen und
Hooks registrieren gelöscht habe):

der Standard-Handgriff des EGw Admins seit 14.x. Mache ich auch regelmäßig – meist aber aus …/setup nach einem EGw-Update.

Die verwendete return_array Methode liefert immer ein - evtl. leeres -
Array, dementsprechend bekomme ich kein PHP Warning.

Bei mir ist das aber anders (siehe meine Orignal-E-Mail) - sonst hätte ich mir das ja gar nicht angesehen. Ich bin nämlich derzeit dabei alle Fehlermeldungen nach und nach zu analysieren, allerdings komme ich nur mangels Zeit in größeren Abständen dazu.

Kann es sein, dass Du einen eigenen Patch in der
phpgwapi/inc/class.categories.inc.php hast?

Nein:

Hier sind die einzigen Dateien mit einem Patch derzeit:
/usr/share/egroupware/emailadmin/inc/class.emailadmin_notifications.inc.php.mw
/usr/share/egroupware/emailadmin/inc/class.emailadmin_imap_cyrus.inc.php.mw
/usr/share/egroupware/phpgwapi/categories.php.mw

Und in „categories.php“ gibt es nur meinen Patch zum gegenständlichen Thema:

xxx:~# diff /usr/share/egroupware/phpgwapi/categories.php.mw /usr/share/egroupware/phpgwapi/categories.php.ori
35,36c35
< // MW: Get all categories - if there are any
< if (is_array($categories))

foreach($categories as $cat)
38c37
< foreach($categories as $cat)

---

if($cat[‘data’][‘color’])
40,47c39,43
< if($cat[‘data’][‘color’])
< {
< $content .= “.cat_{$cat[‘id’]} { background-color: {$cat[‘data’][‘color’]};} /{$cat[‘name’]}/\n”;
< }
< if($cat[‘data’][‘icon’])
< {
< $content .= “.cat_{$cat[‘id’]} .cat_icon { background-image: url(’”. admin_categories::icon_url($cat[‘data’][‘icon’]) ."’);} /{$cat[‘name’]}/\n";
< }

---

  $content .= ".cat_{$cat['id']} { background-color: {$cat['data']['color']};} /*{$cat['name']}*/\n";

}
if($cat[‘data’][‘icon’])
{
$content .= “.cat_{$cat[‘id’]} .cat_icon { background-image: url(’”. admin_categories::icon_url($cat[‘data’][‘icon’]) ."’);} /{$cat[‘name’]}/\n";

Ich habe sogar, bevor ich begonnen habe mich der Dinge anzunehmen „/usr/share/egroupware“ gelöscht und mittels „apt-get --reinstall …" alles neu installiert.

Allerdings ist der Datenbestand (mysqli) – meine Echtdaten – schon sehr alt: Stammt aus 2008 (mit allen eingespielten EGw-DB-Updates im Laufe der Zeit).

Liebe Grüße
Michael

[Ralf_Becker_Stylite]

Hi Michael,

Problem gefunden und behoben:

https://svn.stylite.de/viewvc/egroupware/branches/14.2/phpgwapi/inc/class.categories.inc.php?r1=55557&r2=55556&pathrev=55557

War im Trunk/16.1 schon behoben, deswegen konnte ich es nicht
reproduzieren bei mir.

Danke für die Hinweise

Ralf

Lieber Ralf!

ich hab’ mir das mal genau angesehen (nachdem ich meine egw_categories
komplett geleert und den Instanzcache unter Admin >> Cache löschen und
Hooks registrieren gelöscht habe):
der Standard-Handgriff des EGw Admins seit 14.x. Mache ich auch regelmäßig – meist aber aus …/setup nach einem EGw-Update.

Die verwendete return_array Methode liefert immer ein - evtl. leeres -
Array, dementsprechend bekomme ich kein PHP Warning.
Bei mir ist das aber anders (siehe meine Orignal-E-Mail) - sonst hätte ich mir das ja gar nicht angesehen. Ich bin nämlich derzeit dabei alle Fehlermeldungen nach und nach zu analysieren, allerdings komme ich nur mangels Zeit in größeren Abständen dazu.

Kann es sein, dass Du einen eigenen Patch in der
phpgwapi/inc/class.categories.inc.php hast?
Nein:

Hier sind die einzigen Dateien mit einem Patch derzeit:
/usr/share/egroupware/emailadmin/inc/class.emailadmin_notifications.inc.php.mw
/usr/share/egroupware/emailadmin/inc/class.emailadmin_imap_cyrus.inc.php.mw
/usr/share/egroupware/phpgwapi/categories.php.mw

Und in „categories.php“ gibt es nur meinen Patch zum gegenständlichen Thema:

xxx:~# diff /usr/share/egroupware/phpgwapi/categories.php.mw /usr/share/egroupware/phpgwapi/categories.php.ori
35,36c35
< // MW: Get all categories - if there are any
< if (is_array($categories))

foreach($categories as $cat)
38c37
< foreach($categories as $cat)

---

if($cat[‘data’][‘color’])
40,47c39,43
< if($cat[‘data’][‘color’])
< {
< $content .= “.cat_{$cat[‘id’]} { background-color: {$cat[‘data’][‘color’]};} /{$cat[‘name’]}/\n”;
< }
< if($cat[‘data’][‘icon’])
< {
< $content .= “.cat_{$cat[‘id’]} .cat_icon { background-image: url(’”. admin_categories::icon_url($cat[‘data’][‘icon’]) ."’);} /{$cat[‘name’]}/\n";
< }

---

  $content .= ".cat_{$cat['id']} { background-color: {$cat['data']['color']};} /*{$cat['name']}*/\n";

}
if($cat[‘data’][‘icon’])
{
$content .= “.cat_{$cat[‘id’]} .cat_icon { background-image: url(’”. admin_categories::icon_url($cat[‘data’][‘icon’]) ."’);} /{$cat[‘name’]}/\n";

Ich habe sogar, bevor ich begonnen habe mich der Dinge anzunehmen „/usr/share/egroupware“ gelöscht und mittels „apt-get --reinstall …" alles neu installiert.

Allerdings ist der Datenbestand (mysqli) – meine Echtdaten – schon sehr alt: Stammt aus 2008 (mit allen eingespielten EGw-DB-Updates im Laufe der Zeit).

Liebe Grüße
Michael

---

Transform Data into Opportunity.
Accelerate data analysis in your applications with
Intel Data Analytics Acceleration Library.
Click to learn more.
http://pubads.g.doubleclick.net/gampad/clk?id=278785471&iu=/4140

---

egroupware-german mailing list
egroupware-german@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/egroupware-german

–
Ralf Becker
Director Software Development

Stylite AG

Isaac-Fulda-Allee 9 | Tel. +49 6131 32702-0
D-55124 Mainz | Fax. +49 6131 32702-70

Email: rb@stylite.de

www.stylite.de | www.egroupware.org

Managing Directors: Andre Keller | Ralf Becker | Gudrun Mueller
Chairman of the supervisory board: Prof. Dr. Birger Leon Kropshofer

VAT DE214280951 | Registered HRB 46224 Mainz Germany