Kleiner Bug in /inc/xtc_get_customers_statuses.inc.php

[noRiddle (revilonetz)]

Hallo Community.
In der im Betreff genannten Datei gibt es einen kleinen Bug.

Code: PHP  [Auswählen]

$customers_statuses_array = array(array());

muß lauten

Code: PHP  [Auswählen]

$customers_statuses_array = array();

Zwar wird die Funktion soweit ich gesehen habe nirgends verwendet, ich benötigte sie jedoch letztens zufällig für eine kleine Erweiterung.
Sollte man also, wenn es die Funktion denn schon gibt, fixen.

Gruß,
noRiddle

*NACHTRAG*
Ticket dazu wurde erstellt.

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

[Angeal]

Es freut mich immer wenn hier selbst noch so kleine Fehler aufgedeckt werden

Vielen Dank dafür

[web28]

Was ist falsch?

Das kann man durchaus so schreiben (mehrdimensionales Array):

Code: PHP  [Auswählen]

$arr = array(array());

[noRiddle (revilonetz)]

Jau, das wäre die Definition einer Varaiblen mit einem mehrdimensionalen Array,
aaaber, machen wir einen Test nach dem Muster der Funktion um die es hier geht :

Code: PHP  [Auswählen]

<?php
$test = array(array());
$test[] = array('eins' => '1',
                'zwei' => '2',
                'drei' => '3'
               );

echo '<h1>Array $test</h1><pre>'.print_r($test, true).'</pre>';

$test2 = array();
$test2[] = array('eins' => '1',
                'zwei' => '2',
                'drei' => '3'
               );

echo '<h1>Array $test2</h1><pre>'.print_r($test2, true).'</pre>';

?>

Das gibt aus:

Code: [Auswählen]

Array $test

Array
(
    [0] => Array
        (
        )

    [1] => Array
        (
            [eins] => 1
            [zwei] => 2
            [drei] => 3
        )

)

Array $test2

Array
(
    [0] => Array
        (
            [eins] => 1
            [zwei] => 2
            [drei] => 3
        )

)


Das zweite ist das was man haben möchte in der Funktion.
$customers_statuses_array[0]['text'] z.B. würde im ersten Beispiel was die Funktion betrifft nicht existieren, bzw. leer sein.

Gruß,
noRiddle

[web28]

Deshalb habe ich nach dem Fehler gefragt.

Wird gefixt.

Gruss Web28

[peterdd]

Zwar wird die Funktion soweit ich gesehen habe nirgends verwendet ...

grep ist dein Freund. Nutze ich sehr oft um toten oder doppelten Code im Shop aufzuspüren.

Code: [Auswählen]

$ grep -rn xtc_get_customers_statuses
admin/content_manager.php:46:    $customers_statuses_array=xtc_get_customers_statuses();
admin/content_manager.php:146:    $customers_statuses_array=xtc_get_customers_statuses();
admin/content_manager.php:207:      $customers_statuses_array=xtc_get_customers_statuses();
admin/content_manager.php:598:                                      $customers_statuses_array = xtc_get_customers_statuses();
admin/content_manager.php:782:                                      $customers_statuses_array = xtc_get_customers_statuses();
admin/create_account.php:34:  $customers_statuses_array = xtc_get_customers_statuses();
admin/customers.php:40:  $customers_statuses_array = xtc_get_customers_statuses();
admin/customers_status.php:397:                          $contents[] = array('text' => '<br />' . TEXT_INFO_CUSTOMERS_STATUS_BASE . '<br />' . ENTRY_CUSTOMERS_STATUS_BASE . '<br />' . xtc_draw_pull_down_menu('customers_base_status', xtc_get_customers_statuses()));
admin/includes/application_top.php:388:xtc_get_customers_statuses();
admin/includes/classes/categories.php:127:    $customers_statuses_array = xtc_get_customers_statuses();
admin/includes/classes/categories.php:399:    $customers_statuses_array = xtc_get_customers_statuses();
admin/includes/classes/categories.php:461:    $customers_statuses_array = xtc_get_customers_statuses();
admin/includes/classes/categories.php:1127:    $customers_statuses_array = xtc_get_customers_statuses();
admin/includes/classes/import.php:55:           $this->Groups = xtc_get_customers_statuses();
admin/includes/classes/import.php:658:          $this->Groups = xtc_get_customers_statuses();
admin/includes/functions/general.php:1989:   * xtc_get_customers_statuses()
admin/includes/functions/general.php:1993:  function xtc_get_customers_statuses() {
admin/includes/functions/general.php:2075:    return xtc_draw_pull_down_menu($name, xtc_get_customers_statuses(), $customers_status_id);
admin/includes/modules/export/billiger.php:329:        $customers_statuses_array = xtc_get_customers_statuses();
admin/includes/modules/export/froogle.php:243:    $customers_statuses_array = xtc_get_customers_statuses();
admin/includes/modules/export/geizhals.php:288:      $customers_statuses_array = xtc_get_customers_statuses();
admin/includes/modules/export/golem.php:183:    $customers_statuses_array = xtc_get_customers_statuses();
admin/includes/modules/export/googlebase.php:409:    $customers_statuses_array = xtc_get_customers_statuses();
admin/includes/modules/export/idealo.php:772:       $customers_statuses_array = xtc_get_customers_statuses();
admin/includes/modules/export/idealo_real.php:891:          $customers_statuses_array = xtc_get_customers_statuses();
admin/includes/modules/export/kelkoo.php:193:    $customers_statuses_array = xtc_get_customers_statuses();
admin/includes/modules/export/metashopper.php:148:    $customers_statuses_array = xtc_get_customers_statuses();
admin/includes/modules/export/milando.php:187:    $customers_statuses_array = xtc_get_customers_statuses();
admin/includes/modules/export/preisauskunft.php:150:    $customers_statuses_array = xtc_get_customers_statuses();
admin/includes/modules/export/preisroboter.php:167:$customers_statuses_array = xtc_get_customers_statuses();
admin/includes/modules/export/preistrend.php:147:    $customers_statuses_array = xtc_get_customers_statuses();
admin/includes/modules/export/shopboy.php:159:    $customers_statuses_array = xtc_get_customers_statuses();
admin/includes/modules/export/testeo.php:243:    $customers_statuses_array = xtc_get_customers_statuses();
admin/includes/modules/export/wein.php:174:    $customers_statuses_array = xtc_get_customers_statuses();
admin/includes/modules/export/wein2.php:216:      $customers_statuses_array = xtc_get_customers_statuses();
admin/mail.php:213://    $customers_statuses_array = xtc_get_customers_statuses();
admin/module_newsletter.php:34:      $customers_status=xtc_get_customers_statuses();
admin/module_newsletter.php:394:                      $customers_status=xtc_get_customers_statuses();
admin/module_newsletter.php:470:                $customers_status=xtc_get_customers_statuses();
admin/orders_edit_address.php:204:<?php echo xtc_draw_pull_down_menu('customers_status', xtc_get_customers_statuses(), $order->info['status']). TEXT_CUSTOMER_GROUP_INFO;?>
admin/start.php:41:$customers_statuses_array = xtc_get_customers_statuses();
export/cao_xtc_functions.php:597:    require_once(DIR_FS_INC .'xtc_get_customers_statuses.inc.php');
export/cao_xtc_functions.php:599:    $customers_status=xtc_get_customers_statuses();
inc/xtc_get_customers_statuses.inc.php:3:   $Id: xtc_get_customers_statuses.inc.php 808 2005-02-27 09:04:50Z mz $
inc/xtc_get_customers_statuses.inc.php:18:   (c) 2003    nextcommerce (xtc_get_customers_statuses.inc.php,v 1.4 2003/08/13); www.nextcommerce.org
inc/xtc_get_customers_statuses.inc.php:25:  function xtc_get_customers_statuses() {

Bzw. wenn ich Suche wo eine Funktion definiert ist:

Code: [Auswählen]

$ grep -rn 'n xtc_get_customers_statuses'
admin/includes/functions/general.php:1993:  function xtc_get_customers_statuses() {
inc/xtc_get_customers_statuses.inc.php:25:  function xtc_get_customers_statuses() {


[noRiddle (revilonetz)]

Deshalb habe ich nach dem Fehler gefragt.

Wird gefixt.

Gruss Web28

Schon klar, danke.

@rö
Was du nicht sagst, dann suche nochmal richtig .
Eine Funktion des gesuchten Namens gibt es zweimal und die von der ich rede wird nirgends verwendet.
Es ist aber trotzdem gut, daß es sie gibt. Mußte letzt jemandem in der create_account ein Dropdown bauen mit Auswahl des Kunden-Status, da kam mir die Funktion gelegen.

Die Verwendungen die du gefunden hast sind nicht die Verwendung der Funktion aus dem Ordner /inc sondern aus /admin/includes/functions/general.php und da ist die Variable mit dem Array korrekt definiert.

Gruß,
noRiddle

[web28]

Die Funktion gibt es zweimal.

Da die Funktion in der general.php bereits vor langer Zeit geändert wurde, und die Funktion im inc Verzeichnis noch nie benutzt wurde ist der Fehler nicht aufgefallen.