Best way to allow users to reset a password. - Page 6

tangoforce · 10-09-2012, 10:53 PM

Quote:

Originally Posted by LearningCoder

Edit:Just ran the same print_r($stmt) code through my localhost and it is also returning affected_rows -1 but it lets me login....

Quote:

Originally Posted by tangoforce

As it's a select statement you should be using num_rows() not affected_rows() which is for insert, update, delete etc.

Remember what I said ? - You are still looking at affected_rows() and talking about it! You even got me following that idea and scratching my head again.

Lets stick with num_rows() and have no more mention of affected_rows() since we're not affecting anything.

Quote:

Originally Posted by LearningCoder

Well it seems nowadays you're the only one who replies to me so I might have to seek a bit of advice from another forum or something because my last 2-3 threads are replied to only by yourself, no one else helps me lol.

Don't take that personally. It's true your requests for help do turn into multi-page topics but that isn't unusual, it just shows your determination to get to the bottom of it.

What it does do however is scare off those who are less experienced because they know that they're not able to support your request for help. Those are the sort of people who will purely guess at something without really understanding debugging. Generally around here once you've got a reasonably respected helper assisting you, others will leave you to it unless you miss something and its glaring them in the face. It's kind of a silent understanding between those who donate time here. There are many here who would help you - Fou-Lu, firepages, Sppokster, Inigoesdr, Lamped and ShaneC to name a few (those were taken from the mod list - but there are others who are not mods). If you've ever heard the proverb "Too many cooks spoil the broth" then you'll understand it.

Don't be put off codingforums, I'm also on sitepoint but I find it a site that is 'PR heavy'. In other words its highly advertised as the expert site but the quality of support can be a bit low sometimes. I've found that codingforums is actually a lot better.

Quote:

Originally Posted by LearningCoder

This is what is being returned:

Code:

mysqli_stmt Object
(
    [affected_rows] => -1
    [insert_id] => 0
    [num_rows] => 0
    [param_count] => 2
    [field_count] => 7
    [errno] => 2014
    [error] => Commands out of sync; you can't run this command now
    [sqlstate] => HY000
    [id] => 1
)

Regards,

Lc.

Which line of code output that? - The error does have a message so I'd like you to repost your code and point out which line has the error by putting a comment in please.

LearningCoder · 10-09-2012, 11:05 PM

Sorry I did mean num_rows...lol had affected_rows in my head..

I must say I haven't found any other forum really which has supported my questions as well as CF has which is why I frequent here. Help is always appreciated taking into account you guys don't really need to be sat there going through someone elses work, especially when it turns into a topic like mine ;p

I've just taken a look and I was checking the $stmt with print_r BEFORE executing. I printed afterwards and it seems to just be printing 0 now and not -1....I can't seem to get the error back, sorry.

Just about to put it live again and re-test it.

Will report back in 20.

Regards,

LC.

LearningCoder · 10-10-2012, 08:20 PM

Ok it's logging in on localhost returning num_rows and affected_rows with 1 for both.

On my live host, it is returning 0.

Does this mean a difference in versions? The select query in the login_select.php file is what it should be. I've echo'd out the password which I use to query the database just before executing the statement and it matches the correct password.

Really at a hurdle which I can't get over. I've looked over and over the code now as you know for the last 3-4 days maybe even longer. Are there any good free debugging tools that you know of?

Kind regards,

LC.

tangoforce · 10-10-2012, 08:58 PM

Quote:

Originally Posted by LearningCoder

Ok it's logging in on localhost returning num_rows and affected_rows with 1 for both.

Ok, thats good..

Quote:

Originally Posted by LearningCoder

On my live host, it is returning 0.

Bad..

Quote:

Originally Posted by LearningCoder

Does this mean a difference in versions?

Possibly but why it would make a difference I've no idea.

Quote:

Originally Posted by LearningCoder

Really at a hurdle which I can't get over. I've looked over and over the code now as you know for the last 3-4 days maybe even longer. Are there any good free debugging tools that you know of?

print(), var_dump(), print_r(), x_debug (may not be installed on your host), your brain or even me via a remote connection

(remember though charges apply - £20 per hour, minimum of one hour with no fix no fee).

Also you could comment out the current code (reinstate it later) and run the query through mysql instead of mysqli and see what happens. This is your best next step from my POV as it allows you to determine if the data is being matched correctly or not.

LearningCoder · 10-10-2012, 10:21 PM

OMG i've done it...............sleeeps for a week.....

I ran a var_dump() on the password variable just after using the sha1() function, ready for the query to match the user and pass, it was returning:

Code:

string(40) "examplestring40characterslater";

I looked at the table and it was 8 or 9 characters short....Deleted user, re-registered and correct amount of characters were there and it let me login.....

About 100 hours, 50 coffees, 50 cigs, sleep deprivation and the biggest pain in my * which php has thrown my way so far....a simple var_dump() could have resolved this. I should have counted every character when comparing them before, I just went off the first 5-6 and then thought it must match.....

I feel ill after this, I thought i'd never sort it. I'm surpised I have hair I really am, I was so close to just giving up!

Thanks so much for the help and all your tips and ideas.

Kind regards,

LC.

tangoforce · 10-10-2012, 11:23 PM

You see, thats why you should check the first 3-4 and last 3-4 characters of a string when you're manually comparing them! You quite clearly told me that they matched so I ruled that out as being the problem and was looking at other things.

Now you've learned how to debug a bit better dumping stuff and examining it in detail I think you deserve a pat on the back

Well done

The only thing that does worry me.. how do you know the rest of the accounts haven't suffered the same fate? - You need to check them and make sure they're all 40 characters too.

As for giving up, I did that a few years ago on one project I was working on in delphi. It was a string termination error in a socket server I was writing. I couldn't nail the fault and gave up after many desk thumping moments. 2 years later I revisited it trying to decide wether to delete the project dir or not. Opened it up and the fault was glaring me in the face. Just goes to show that giving up isn't always necessary but taking a break can be a good thing

Still, you saved yourself some money - had you of taken me up on my offer that minimum 1hr of £20 would of been wasted - I'd have identified that fault within 5 - 10 mins!

LearningCoder · 10-10-2012, 11:32 PM

Quote:

Originally Posted by tangoforce

You see, thats why you should check the first 3-4 and last 3-4 characters of a string when you're manually comparing them! You quite clearly told me that they matched so I ruled that out as being the problem and was looking at other things.

Now you've learned how to debug a bit better dumping stuff and examining it in detail I think you deserve a pat on the back

Well done

I will bear that in mind for the future, I can't believe it took me so long to spot it...

Quote:

The only thing that does worry me.. how do you know the rest of the accounts haven't suffered the same fate? - You need to check them and make sure they're all 40 characters too.

I have only created that one dummy account. I will create 10 or so more and check everything works accordingly.

Quote:

As for giving up, I did that a few years ago on one project I was working on in delphi. It was a string termination error in a socket server I was writing. I couldn't nail the fault and gave up after many desk thumping moments. 2 years later I revisited it trying to decide wether to delete the project dir or not. Opened it up and the fault was glaring me in the face. Just goes to show that giving up isn't always necessary but taking a break can be a good thing

Still, you saved yourself some money - had you of taken me up on my offer that minimum 1hr of £20 would of been wasted - I'd have identified that fault within 5 - 10 mins!

I will take all your help on board and store it! Lol oh dear that would have REALLY done my head in!

Again thank you. I'm sure you'll hear from me again but I'll really research my issue next time before asking for help.

Kind regards,

LC.