"Division by zero" in content_manager.php

[guest1552]

Hallo,
wenn man im Admin Bereich im Content Manager für die Sprachgruppe "0" eingibt, oder das Feld leer lässt, entsteht ein "Division by zero" in der Datei content_manager.php und das Layout wird zerstückelt.

Eine einfache Lösung ist, man ändert Zeile 383 in admin/content_manager.php folgendermassen ab:

Code: PHP  [Auswählen]

<td bgcolor="<?php echo substr((6543216554/($content[$ii]['CONTENT_GROUP']+1)),0,6); ?>" class="dataTableContent" align="left"> </td>

Dies fängt den Fehler ab und dürfte keine sonstigen Auswirkungen haben.

Grüße
Harvey

Linkback: https://www.modified-shop.org/forum/index.php?topic=10714.0

[DokuMan]

Ich weiß nicht ob der Fix wirklich die Ursache bekämpft.
Das Thema hatten wir auch mal unter: Installation Willkommenseite geht nicht weiter

Da hat es aber leider nicht funktioniert.

[franky_n]

Hallo Zusammen,

also ich hätte jetzt die Version genommen:

Code: SQL  [Auswählen]

ALTER TABLE content_manager CHANGE content_group content_group INT( 11 ) NOT NULL DEFAULT '1'

EDIT:
Und zusätzlich in der admin/content_manager.php in Z.69 aus dem:

Code: PHP  [Auswählen]

          $group_id=xtc_db_prepare_input($_POST['content_group']);

das gemacht:

Code: PHP  [Auswählen]

        if ($_POST['content_group'] == '0' || $_POST['content_group'] == '') {
          $group_id='1';
        } else {
          $group_id=xtc_db_prepare_input($_POST['content_group']);
        }
 

Viele Grüße

Franky

[DokuMan]

Hi Franky,
es immer auf 1 zu setzen, wird früher oder später zu Problemen führen, da die Versandkosteninfos schon die eins haben. Wird neuer Content wieder mit 1 angelegt, erscheint zwar der Link im Shop, er verweist dann allerdings auch auf die Versandkosten

Das Non-Plus-Ultra wäre vielleicht, das Feld "Sprachgruppe" mit der nächsten freien Nummer (der jeweiligen Sprache vorzubelegen)

[franky_n]

Hi Simon,

jo das stimmt wohl aber besser als ein Fehler oder?

Viele Grüße

Franky

[Tomcraft]

[...]
Das Non-Plus-Ultra wäre vielleicht, das Feld "Sprachgruppe" mit der nächsten freien Nummer (der jeweiligen Sprache vorzubelegen)

Eine gute Idee, da hier eh die meisten Fehler dadurch entstehen, dass Sprachgruppen-IDs doppelt vergeben werden.

Grüße

Torsten

[franky_n]

Hallo Zusammen,

bin ich wieder der der es machen soll?

Also gut, dann oben das vergessen!

In der admin/content_manager.php in Z. 69 aus dem:

Code: PHP  [Auswählen]

        $group_id=xtc_db_prepare_input($_POST['content_group']);

das machen:

Code: PHP  [Auswählen]

        $content_query = xtc_db_query("SELECT MAX(content_group) AS content_group FROM ".TABLE_CONTENT_MANAGER."");
        $content_data = mysql_fetch_row($content_query);
        if ($_POST['content_group'] == '0' || $_POST['content_group'] == '') {
          $group_id = $content_data[0] + 1;
        } else {
          $group_id = xtc_db_prepare_input($_POST['content_group']);
        }
 

Das wars schon!

Viele Grüße

Franky

[Tomcraft]

Danke Franky!

Blöde Frage, aber reicht der untere Teil nicht aus?

Das Thema hier setze ich damit mal auf "behoben".

Nachtrag: Franky hat die Änderung in r1640 in den Trunk übernommen.

Grüße

Torsten

[cYbercOsmOnauT]

Ich hätte da eine andere, noch simplere Lösung anzubieten.

Aus

Code: PHP  [Auswählen]

 <td bgcolor="<?php echo substr((6543216554/$content[$ii]['CONTENT_GROUP']),0,6); ?>" class="dataTableContent" align="left"> </td>

wird

Code: PHP  [Auswählen]

 <td bgcolor="<?php echo ($content[$ii]['CONTENT_GROUP'] ? substr((6543216554/$content[$ii]['CONTENT_GROUP']),0,6) : 0); ?>" class="dataTableContent" align="left"> </td>

Grüße,
Tekin

[Tomcraft]

Simpel trifft es. Du hättest auch sagen können "quick & dirty".

Ich mag die Lösung von Franky lieber.

Grüße

Torsten

[cYbercOsmOnauT]

Wobei sich mir die Frage stellt wieso die Hintergrundfarbe einer Tabellenzelle überhaupt aus der Division zweier variablen Werte bildet. Da kommt es bestimmt auch immer wieder zu Fällen die dank fehlendem Kontrast sehr unleserlich werden, oder?