Gutscheinaddon

[WaechterMedia]

Bischn spät, was mir direkt auffällt du sicherst die eingabe des User in die db überhaupt nicht ab ich könnte da reinschreiben was ich will und das ist nicht gut :/

Ansonsten nett das du es kostenfrei zu verfügung stellst

[jo18nny]

Bischn spät, was mir direkt auffällt du sicherst die eingabe des User in die db überhaupt nicht ab ich könnte da reinschreiben was ich will und das ist nicht gut :/

Ansonsten nett das du es kostenfrei zu verfügung stellst

Stimmt da hast du recht.Hab es jetzt mal bisschen bearbeitet.
Hoffe das es so besser ist.

[CeleronD]

Man kann Gutscheine mit gleichen Namen erstellen. Das ist doof, denn lösche ich einen sind alle mit gleichen Namen weg. Und dem User wird nur einer gutgeschrieben wieder.

Und ist es normal zweimal die selbe ($menge) Variable zu nutzen. (ich weiss nicht)

db_query("Insert Into vms_gutschein
 (Uid,Gutschein,aMenge,Menge,Wert) values ('".$_SESSION['uid']."','".$schein."','".$menge."','".$menge."','".$wert."'); 

Im SQL ist aber nix mit aMenge wohin soll er das denn packen.

[jo18nny]

Man kann Gutscheine mit gleichen Namen erstellen. Das ist doof, denn lösche ich einen sind alle mit gleichen Namen weg. Und dem User wird nur einer gutgeschrieben wieder.

Und ist es normal zweimal die selbe ($menge) Variable zu nutzen. (ich weiss nicht)

db_query("Insert Into vms_gutschein
 (Uid,Gutschein,aMenge,Menge,Wert) values ('".$_SESSION['uid']."','".$schein."','".$menge."','".$menge."','".$wert."'); 

Im SQL ist aber nix mit aMenge wohin soll er das denn packen.

Danke für den Hinweis und sorry für die Fehler hab es jetzt oben verbessert .
Und aMenge muss man noch in der Datenbank hinzufügen.
Hier der Befehl:

ALTER TABLE `vms_gutschein` ADD `aMenge` INT( 10 ) NOT NULL

und die Änderungen in der Gutschein.php sind:
unter die Zeile :

$kontodaten   = mysql_fetch_array(db_query("SELECT `kontostand` FROM ".$db_prefix."_kontodaten WHERE uid=".$_SESSION['uid']." LIMIT 1")); 

das einfügen:

$check=db_query("SELECT Gutschein FROM ".$db_prefix."_gutschein WHERE Gutschein='".$name."'"); 

dann unter der Zeile :

if ($_POST['erstellen']){ 

das einfügen :

if (mysql_num_rows($check)) {
$error.= "Es existiert bereits ein Gutschein mit dem Namen überlege dir bitte einen anderen Namen!<br />";    
} 

[CeleronD]

db_query("Insert Into vms_gutschein 
 (Uid,Gutschein,aMenge,Menge,Wert) values ('".$_SESSION['uid']."','".$schein."','".$menge."','".$menge."','".$wert."'); 

Wieso soll er beim eintragen in die Db zweimal das gleiche Eintragen ($menge) ???

[jo18nny]

Also in die Spalte aMenge um zu wissen wie viele Gutscheine es mal waren und Menge eben wie viele es noch gibt .

[CeleronD]

Zwei Sachen hab ich nun noch gefunden. Man kann als User soviel Gutscheine einlösen wie da sind.
Und es wird keine Meldung ausgeben das der Gutschein eingelöst wurde.