Falsche Anzahl der Kunden in "/admin/start.php"

[Modulfux]

In dem SQL-Query für die Gesamtanzahl der Kunden ist ein kleiner Logikfehler drin. Die Anzahl der Kunden wird komplett gezählt. Richtig wäre es aber zumindest bei der Standardversion mit einem Admin, die Anzahl um eins zu reduzieren.

Suche:

Code: PHP  [Auswählen]

$customers_query = xtc_db_query('select cs.customers_status_name cust_group, count(*) cust_count
                     from ' . TABLE_CUSTOMERS . ' c
                     join ' . TABLE_CUSTOMERS_STATUS . ' cs on cs.customers_status_id = c.customers_status
                     --  exclude admin
                     where c.customers_status> 0
                     -- restrict to current language setting
                     and cs.language_id = ' . $language_id . '
                     group by 1
                     union
                     select \'' . TOTAL_CUSTOMERS . '\', count(*)
                     from ' . TABLE_CUSTOMERS . '
                     order by 2 desc');
 

Ersetze mit:

Code: PHP  [Auswählen]

$customers_query = xtc_db_query('select cs.customers_status_name cust_group, count(*) cust_count
                     from ' . TABLE_CUSTOMERS . ' c
                     join ' . TABLE_CUSTOMERS_STATUS . ' cs on cs.customers_status_id = c.customers_status
                     --  exclude admin
                     where c.customers_status> 0
                     -- restrict to current language setting
                     and cs.language_id = ' . $language_id . '
                     group by 1
                     union
                     select \'' . TOTAL_CUSTOMERS . '\', (count(*) -1)
                     from ' . TABLE_CUSTOMERS . '
                     order by 2 desc');
 

Sollten mehrere Admins vorhanden sein, so müsste man das Query so umschreiben, dass die Anzahl der "Kunden" mit "customers_status = 0" abgezogen wird.

Gruß
Ronny



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

[Tomcraft]

Ich sehe hier keinen Handlungsbedarf. Gezählt werden alle Benutzer aller Kundengruppen und somit auch die Admins, was ich nicht schlimm finde.

Grüße

Torsten

[speedy]

Das "-1" wäre auch nur für die Standardversion richtig, wenn dann müsste man alle Admins abziehen. Da kann es ja durchaus mehrere mit ggf. unterschiedlichen Rechten geben.

[Tomcraft]

Aber ist das irgendein Wert, der für was auch immer eine Relevanz hat, ob der auf +-2 Kunden genau ist?

Solange man weiss, dass die Admins mitgezählt werden, denke ich, geht das in Ordnung.

Grüße

Torsten

[guensi]

Ist das notwendig? Bei 100 Kunden ist der Admin 1%, bei 1000 schon ein promill, ...
Das lässt sich doch locker in jeder Statistik durch einen Korrekturfaktor bereinigen - wenn's denn wirklich sein muss.

[Tomcraft]

Das ist ja genau die Frage, die ich mir stelle, ob und wofür ich überhaupt auf 2 Kunden genau wissen muss, wie viele ich im Shop habe?!

Grüße

Torsten

[speedy]

Ich halte es nicht für notwendig, da auch mal Admins selbst bestellen können und es auch nicht ins Gewicht fällt, wenn nicht.
Aber wenn man es machen will, dann würde ich die Anzahl der Kundengruppe "Admins" abziehen.