Gutscheinaddon

[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.

[jo18nny]

Danke nochmal und hier die Lösung:

db_befehl:

CREATE TABLE IF NOT EXISTS `vms_benutzte_gutscheine` (
`Uid` INT NOT NULL ,
`Gutschein` TEXT NOT NULL 
) ENGINE=MyISAM DEFAULT CHARSET=latin1;

dann nach der Zeile:

$Gutschein_check=db_query("Select Gutschein from ".$db_prefix."_gutschein where
Gutschein='". $einloesen."'"); 

das

$eingeloest=db_query("Select Uid from ".$db_prefix."_benutzte_gutscheine where
Gutschein='". $einloesen."'"); 

dann nach der Zeile:

if ($test2['Uid'] == $_SESSION['uid']) $error .= 'Du kannst nicht deine eigenen Gutscheine
einlosen!<br />'; 

dies:

if (mysql_num_rows($eingeloest)) $error .="Du hast diesen Gutschein schon eingelöst
<br />"; 

einfügen
unten unter der Zeile:

db_query("Update ".$db_prefix."_gutschein set Menge=Menge -1 where Gutschein='".
$einloesen."'"); 

dies einfügen:

db_query("INSERT INTO ".$db_prefix."_benutzte_gutscheine (Uid,Gutschein) VALUES ('".$_SESSION['uid']."','".$einloesen."')");
$info.="Gutschein eingelöst!"; 

so das war das der User ein Gutschein nur einmal einlösen kann:
Hier für die Info :
Nach der Zeile:

if (!isset($error)) $error                 = ""; 

if (!isset($info)) $info                 = ""; 

einfügen
dann weiter unten unter

db_query("INSERT INTO ".$db_prefix."_benutzte_gutscheine (Uid,Gutschein) VALUES ('".$_SESSION['uid']."','".$einloesen."')"); 

das

$info.="Gutschein eingelöst!"; 

und ganz unten vor dem ?>
das

if ($info){
head("Information");
echo '
<font color="#cc0000">'.$info.'</font>';
foot();
} 

einfügen