Gastkonto löschen

[speedy]

Hallo,

ich habe den Eindruck, dass Gastkonten nicht gelöscht werden, obwohl die entsprechende Einstellung gesetzt wurde.
Bedeutet dann immer manuell unter Kunden löschen.

Habe dann als Ursache herausgefunden, dass Gastkonten erst gelöscht werden, wenn sich der Gast abmeldet und das werden wohl die wenigsten machen.
Eigentlich sollte es sowas wie abmelden bei Gästen gar nicht geben, der Kunde will ja schließlich kein Kundenkonto haben. Doch durch sowas könnte genau der Eindruck entstehen, er hätte doch eines, obwohl er nicht wollte.

Auf folgender Seite gibt es eine Lösung, die wohl von Hetfield noch optimiert / korrigiert wurde. [gelöst] Gastkonten zuverlässig löschen

Zum Thema "Gastkonten löschen" empfehle ich noch diesen Beitrag, der sinnvoll beschreibt, warum man Gastkonten unbedingt löschen sollte. Löschen von Gastkonten: Ja oder nein?



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

[Tomcraft]

Mir war so, als hätten wir das schon gefixt gehabt, aber ich frage nochmal die anderen, denn der Code ist im Trunk noch nicht enthalten.

Grüße

Torsten

[speedy]

Man müsste evtl. auch noch abklären, was passiert, wenn das Kundenkonto durch den Code aus dem Link beim Checkout gelöscht wird und der Kunde dann doch ausnahmsweise auf "Abmelden" klickt.
Der Kunde wäre ja dann schon gelöscht, erscheint dann eine Fehlermeldung oder wird was anderes gelöscht ?

[Eichenhorn]

Sehr nützlicher Hinweis. Vielen Dank, speedy.

[Tomcraft]

Wir hatten das gefixt, allerdings an falscher Stelle!

Wir hatten den Fix in der logoff.php drin.

Wird demnächst gefixt, vielen Dank für den Hinweis!

Grüße

Torsten

[speedy]

Bitte, kein Problem.

Allerdings ist Eure logoff.php noch zu 100% xt-commerce v3.04 SP2.1 - habe es mir gerade mit dem Dateivergleicher angesehen.

Der Code vom Fix taucht in der logoff.php fast identisch auf, etwas anders "formatiert", da die anderen Kundengruppen an dieser Stelle noch betroffen sind. Man beachte deshalb die Klammern.

Allerdings ist im Original noch das hier, was im Fix fehlt.

Code: PHP  [Auswählen]

$_SESSION['cart']->reset();
 

Ich würde vorschlagen das hier aus der logoff.php zu entfernen, da man den Rest noch für die angemeldeten Kunden braucht:

Code: PHP  [Auswählen]

//delete Guests from Database  

if (($_SESSION['account_type'] == 1) && (DELETE_GUEST_ACCOUNT == 'true')) {
        xtc_db_query("delete from ".TABLE_CUSTOMERS." where customers_id = '".$_SESSION['customer_id']."'");
        xtc_db_query("delete from ".TABLE_ADDRESS_BOOK." where customers_id = '".$_SESSION['customer_id']."'");
        xtc_db_query("delete from ".TABLE_CUSTOMERS_INFO." where customers_info_id = '".$_SESSION['customer_id']."'");
}
 

und das hier ans Ende der checkout_success.php einzufügen (Fix um eine Zeile kurz vor Ende erweitert):

Code: PHP  [Auswählen]

// BOF - Delete Guests from Database
if (($_SESSION['account_type'] == 1) && (DELETE_GUEST_ACCOUNT == 'true')) {
   xtc_db_query("DELETE FROM ".TABLE_CUSTOMERS." WHERE customers_id = '".$_SESSION['customer_id']."'");
   xtc_db_query("DELETE FROM ".TABLE_ADDRESS_BOOK." WHERE customers_id = '".$_SESSION['customer_id']."'");
   xtc_db_query("DELETE FROM ".TABLE_CUSTOMERS_INFO." WHERE customers_info_id = '".$_SESSION['customer_id']."'");
   xtc_session_destroy();
   unset ($_SESSION['customer_id']);
   unset ($_SESSION['customer_default_address_id']);
   unset ($_SESSION['customer_first_name']);
   unset ($_SESSION['customer_country_id']);
   unset ($_SESSION['customer_zone_id']);
   unset ($_SESSION['comments']);
   unset ($_SESSION['user_info']);
   unset ($_SESSION['customers_status']);
   unset ($_SESSION['selected_box']);
   unset ($_SESSION['navigation']);
   unset ($_SESSION['shipping']);
   unset ($_SESSION['payment']);
   unset ($_SESSION['ccard']);
   unset ($_SESSION['gv_id']);
   unset ($_SESSION['cc_id']);
   $_SESSION['cart']->reset();
   require (DIR_WS_INCLUDES.'write_customers_status.php');
}
// EOF - Delete Guests from Database
 

[DokuMan]

Ist ab Version 899 im Trunk

[speedy]

Guten Morgen,

prima danke, DokuMan.

Wurde diese Zeile absichtlich ausgelassen ?

Code: PHP  [Auswählen]

$_SESSION['cart']->reset();

Im Original-Code ist diese Zeile enthalten und diese gilt für alle Kundengruppen, also auch für Gäste. Im Fix von Hetfield fehlt diese aber.

[DokuMan]

Damit wird der Warenkorb geleert, könnte ich noch mit aufnehmen...
Ich weiß nicht, warum Hetfield diese Funktion weggelassen hat.

[speedy]

Da fällt mir noch was ein.
Sollte man um diesen Teil in der logoff.php nicht noch eine If-Abfrage setzen, damit dieser nur bei Nicht-Gästen ausgeführt wird ?

Code: PHP  [Auswählen]

xtc_session_destroy();

unset ($_SESSION['customer_id']);
unset ($_SESSION['customer_default_address_id']);
unset ($_SESSION['customer_first_name']);
unset ($_SESSION['customer_country_id']);
unset ($_SESSION['customer_zone_id']);
unset ($_SESSION['comments']);
unset ($_SESSION['user_info']);
unset ($_SESSION['customers_status']);
unset ($_SESSION['selected_box']);
unset ($_SESSION['navigation']);
unset ($_SESSION['shipping']);
unset ($_SESSION['payment']);
unset ($_SESSION['ccard']);
// GV Code Start
unset ($_SESSION['gv_id']);
unset ($_SESSION['cc_id']);
// GV Code End
$_SESSION['cart']->reset();
// write customers status guest in session again
require (DIR_WS_INCLUDES.'write_customers_status.php');
 

Ansonsten würde es 2x ausgeführt werden, wenn sich doch mal ein Gast dazu entscheidet auf "Abmelden" zu klicken.
Weiß nicht was das für Folgen hat, wenn das unter Umständen 2x ausgeführt wird.

[DokuMan]

Der Gast kann sich nach dem Checkout nicht mehr abmelden, von daher wird es nicht 2x ausgeführt.

[speedy]

Doch der Gast muss sich abmelden können, wie auch hier steht:
http://www.xtc-supportforum.de/viewtopic.php?f=8&t=1104&start=0

Bislang war ja genau das das Problem. Nur, wenn sich der Gast abgemeldet hat, wurde das Gast-Konto gelöscht. Hat er sich nicht abgemeldet, blieb das Gast-Konto erhalten.

[Tomcraft]

Achso... naja... stimmt eigentlich der Code hätte in der logoff.php eigentlich in r899 nicht auskommentiert werden müssen.

Grüße

Torsten

[speedy]

Hallo Torsten,

ne das war anders gemeint, als es glaub ich verstanden wurde.
Das Problem war ja anfangs, dass nur die Gast-Kunden gelöscht wurden, die sich ordentlich abgemeldet haben, alle anderen blieben trotz Funktion "Gastkonten löschen" erhalten.

So, dafür gibt es ja den Fix von Hetfield für die checkout_success.php
ABER, damit es nicht doppelt gelöscht wird oder zumindest versucht wird, muss natürlich dann der Code in der logoff.php raus, da die Löscherei mit dem Fix ja schon in der checkout_success.php erledigt wurde.
Doch der dann in der logoff.php verbleibende Code, siehe Beitrag Nr. 57741, ist ja ohne eine IF-Abfrage und gilt deshalb für alle Kundengruppen, also auch für Gäste.
Somit würde dieser Code-Teil einmal in der checkout_success.php und noch einmal in der logoff.php ausgeführt werden, sobald sich ein Gast abmeldet.

[DokuMan]

Mit Version r899 ist es so (kann gerne im Trunk-Demoshop nachgetestet werden):

Besucher registriert sich als Gast, legt etwas in den Warenkorb und geht zur Kasse. Beim Bestellprozess wird nun das Gast-Konto beim dem Aufruf der "checkout_success.php" gelöscht und die Session refresht! Ein Ausloggen des Gastes ist nicht mehr erforderlich und auch gar nicht mehr möglich.

Der Abmelden-Button ist dann für Gäste nicht mehr erreichbar. Vor der "checkout_success" kann sich der Gast allerdings abmelden. Tut er das, werden seine Daten jetzt NICHT mehr gelöscht!

Die Frage ist vielmehr: was soll passieren, wenn ein Gast sich noch vor Bestellabschluss ausloggt? Mit dem derzeitigen Code wird er nicht mehr gelöscht, da er in der logoff.php auskommentiert wurde (DELETE TABLE...).