Archief - PHP: query wilt niet replacen

P

PoMpElSiEn

Legacy Member

• 1 okt 2004

• #1

llo
ik be nbezeg met een loginscriptje (tkan zijn da ge et raar gecoded vind ma ik vind het veilig)

http://213.224.88.211:8005/WOUDLOPERS/

hier vind je de bestanden
het probleem zit hem bij functions.phps bij de functie wdl_check
daar wilt de query niet replacen


dankje

S

Smoerf

Legacy Member

• 2 okt 2004

• #2

Het is niet REPLACE maar UPDATE

k

killgore

Legacy Member

• 2 okt 2004

• #3

Smoerf zei:

Het is niet REPLACE maar UPDATE

Klik om te vergroten...

REPLACE =/= UPDATE

(REPLACE is dus eigenlijkj een andere vorm voor INSERT)

Ma wa is uw unique key eigenlijk in die tabel logins?

S

Smoerf

Legacy Member

• 2 okt 2004

• #4

Ahzo,

Moet dit:

Code:

$wdl_result1 = mysql_query($wdl_query1) or die(mysql_error());

niet veranderen in dit

Code:

$wdl_result1 = mysql_query($wdl_query1, [b]$wdl_conn[/b]) or die(mysql_error());

P

PoMpElSiEn

Legacy Member

• 2 okt 2004

• #5

CREATE TABLE wdl_logins (
wdl_userid tinyint(8) NOT NULL default '0',
wdl_validate varchar(32) NOT NULL default '',
wdl_last varchar(50) NOT NULL default '',
PRIMARY KEY (wdl_userid),
UNIQUE KEY wdl_userid (wdl_userid)
) TYPE=MyISAM;

CREATE TABLE wdl_users (
uid int(10) unsigned NOT NULL auto_increment,
wdl_username varchar(50) NOT NULL default '',
wdl_password varchar(50) NOT NULL default '',
PRIMARY KEY (uid)
) TYPE=MyISAM;

dat zijn de 2 tables

k

killgore

Legacy Member

• 2 okt 2004

• #6

Smoerf zei:

Ahzo,

Moet dit:

Code:

$wdl_result1 = mysql_query($wdl_query1) or die(mysql_error());

niet veranderen in dit

Code:

$wdl_result1 = mysql_query($wdl_query1, [b]$wdl_conn[/b]) or die(mysql_error());

Klik om te vergroten...

als je maar 1 connectie hebt doet dat er niet echt toe hoor

ik heb wel nog een fout gevonden: bij mysql moet je enkel ' (niet ", is vervelend in combo met php ) gebruiken voor text-velden, jij gebruikt " ook voor int-velden, het is dus goed mogelijk dat het daaraan ligt .

edit, ik zou het dus zo doen:
$wdl_query1 = "REPLACE INTO wdl_logins (wdl_userid, wdl_validate, wdl_last) VALUES ($wdl_uid, '$wdl_validate', '$wdl_last')";

edit2: ik kan niet echt oordelen of het slecht gecode is, maar ik vind het alleszins slordig gecode. Als je later nog iets van je code wilt snappen --> Comments en deftige layout voorzien

P

PoMpElSiEn

Legacy Member

• 2 okt 2004

• #7

het werkt nog steeds niet :\

en hoe bedoelde slordig, inspringenen enz kloppe ntoch?