TalkPHP - View Single Post

zxt3st · 11-17-2008, 07:52 AM

Code:

mysql_query("update tbl_userpackage set status = 'completed' where hash = '$hash'");

change to

Code:

mysql_query("update tbl_userpackage set status = 'completed' where some_id = '$passed_id'");

Im not quite sure about your question, but as far as i understand in your post (sorry im bad in english really) i will try to give comments about some of your code.

well, i think you better use a unique id of each row such as (id) in the where clause rather than using the hash value. Because based on what i noticed, some of the hash are the same and for quite some time it can't be avoided that there will be same hashes in different accounts.

Other way, if your update code is separated in a different page, try to make sure that you have pass the right variable to use in updating the rows. I have tried to make this mistakes before when im making a project and just notice i have pass no-value variable/parameter to the page (or either way check the variables you are passing if ever).

regards,
t3st

11-17-2008, 07:52 AM	#2 (permalink)
zxt3st The Addict Join Date: Apr 2008 Posts: 200 Thanks: 18	Code: mysql_query("update tbl_userpackage set status = 'completed' where hash = '$hash'"); change to Code: mysql_query("update tbl_userpackage set status = 'completed' where some_id = '$passed_id'"); Im not quite sure about your question, but as far as i understand in your post (sorry im bad in english really) i will try to give comments about some of your code. well, i think you better use a unique id of each row such as (id) in the where clause rather than using the hash value. Because based on what i noticed, some of the hash are the same and for quite some time it can't be avoided that there will be same hashes in different accounts. Other way, if your update code is separated in a different page, try to make sure that you have pass the right variable to use in updating the rows. I have tried to make this mistakes before when im making a project and just notice i have pass no-value variable/parameter to the page (or either way check the variables you are passing if ever). regards, t3st