markspec87 Posted November 26, 2006 Share Posted November 26, 2006 you guys have been so helpful with my previous problems so i might aswell ask this here :)ive finally got a semi working email activation, the email is sent however when they click the link, the database isnt updated.For simplicty im simply using the timestamp to validate.My signup looks like:[code]$t= time();$url='http://mywebsite.com/activate.php?hash='.$t;[/code]The activation link, when sent, looks like:[quote]mysite.com/activate.php?hash=1164562953[/quote]Which is how it looks in the database too.Activate.php:[code]<? include("config.php");$hash = $_GET[’hash’];@mysql_select_db($dbname) or die( "Unable to select database");$query = "UPDATE users SET status = 1 WHERE hash = $hash";mysql_query($query);echo "Your account has been activated<BR>"; ?>[/code]Any ideas why it isnt updating? Link to comment https://forums.phpfreaks.com/topic/28540-more-email-activation-sorry/ Share on other sites More sharing options...
taith Posted November 26, 2006 Share Posted November 26, 2006 try this... see if it gives you any errors...[code]mysql_query("UPDATE users SET `status`='1' WHERE `hash`='$hash'") or die(mysql_error);[/code] Link to comment https://forums.phpfreaks.com/topic/28540-more-email-activation-sorry/#findComment-130592 Share on other sites More sharing options...
markspec87 Posted November 26, 2006 Author Share Posted November 26, 2006 no errors but it doesnt work still.Ive tried echoing $hash however it doesnt show, so obviously the problem is around that variable.Ive checked the code and i dont know why its not able to get the hash value from the url. Link to comment https://forums.phpfreaks.com/topic/28540-more-email-activation-sorry/#findComment-130594 Share on other sites More sharing options...
kenrbnsn Posted November 26, 2006 Share Posted November 26, 2006 The single quotes look weird in this statement:[code]<?php$hash = $_GET[’hash’];?>[/code]Try:[code]<?php$hash = $_GET['hash'];?>[/code]Ken Link to comment https://forums.phpfreaks.com/topic/28540-more-email-activation-sorry/#findComment-130600 Share on other sites More sharing options...
Recommended Posts
Archived
This topic is now archived and is closed to further replies.