Problems with my article.

Ok ok ok, so im only human!

For all those who have experienced problems heres your chance. On my "building a members area with PHP" article there are a few mistakes, but i honestly do not have the time too look for them. If you have a problem with my article, if you were mislead, or i forgot something/made a typo, please tell me.

Im prepaired to fix my mistakes, but i need peoples help.

ALSO please remember that the article is at least an INTERMEADIATE LEVEL, which means, if i have made a mistake, you should at least have the skills to debug it!

BTW: dont tell me its wrong, and then dont tell me what i need to change. I need to know what to change so i can fix it.

Everyone who helps will get credit for it.

ben is human??

no, actually im not human, i just suffer human like qualities

Hello Ben
Despite highlighting a couple of problems with the code, thanks very much for the article. It's one of the best i've found on the Internet. Your work is much appreciated :-)

Kind regards,

You are greatly, Ben. You have taught us lots of things it could have been days to learn it. Your articles are so excellent. Greatly appreciative for your times.

maybe u can add to your article that when people use extra field names when adding sessions they have to create the fields first heh

thats the only problem i found...

keep up the good work

i didn't see any problems - this is the best authntication tutorial on the web. The only problem i have is waiting for the next article in the series

phantom

The next article is already done, i just have to get the time to edit it, as ive got a million things to do today.

Thanks for letting me know of these problems, once i put up the next article I will fix up the others.

Just a suggestion for you.

Maybe you could write some sort of a admin script so that you could see the registered users and the information they entered when they signed up. I know I would use it and other people would probably also.

By the way. Love your articles. The very BEST I can find on the internet.

Kudos

I don't know if this is a problem since im not a hacker like you guys...

In class.security.php

function AddLog($strLogData = '')
{
/*
Function Name: AddLog()
Desc: Adds a new record to the log table. This allows tracking of
who is logging in, and who is trying to 'hack' into certain areas
*/
$dbVars = new dbVars;


//dont you need a $ before dbVars?

//like this

*shrug*

nope.. because dbVars is a class name

Ben: I've only had a little bit of a problem with any of your code, and it was easily fixable.. and to be honest with you I can't even recall where the problems were. I think if anything there may have been a mispelling in one place, and an extra space which may have caused errors, but those are mistakes that anyone could make.. I think that you did a good job writing the articles. Keep em coming strong, no one is perfect.. that's why we have the discussion for the articles, right?

Code is a living creature that isn't usually perfect the first time it is born, sometimes a little "surgery" is needed..

Ben, This the best Authentication tutorial on the net so far. Keep it up.

Though the errors, the tutorial stands out from the crowd.

When is part 6 going to be out?

ive just finished the updates for MailWorks Professional (out soon), so ive been quite busy developing and testing the new version. Im hoping to finish up the members area articles, (as well as fix up the code (sorry to all those who had problems))

im also going to be starting a article series on php classes.

a lot of your code is not xhtml 1.0 transitional so in that sense your code is "bad code." I suggest making your code xhtml transitionable. That is all.

i get an error

Parse error: parse error, expecting `']'' in /home/virtual/site88/fst/var/www/html/login.php on line 121


if($dtlsSecurity->StoreSession($result[‘userId‘], $result[‘username’], $result[‘level’], "{$result['firstname']},{$result['lastname']}" ));


line 121 --^

any help?

Firstly because my code isnt xhtml 1.0 transitional, doesnt mean that I have "bad code". The article wasn't on xhtml compliency, it was to illistrate how to create a security class, which allows access to files. Also I only code in HTML 4 not XHTML. The end result is still the same, infact using XHTML makes your html files bigger.

if($dtlsSecurity->StoreSession($result[‘userId‘], $result[‘username’], $result[‘level’], "{$result['firstname']},{$result['lastname']}" ));


it's an if statement.. try taking out the semicolon at the end

To be honest with you, h00kup, I don't know many people who write in xhtml... If it were me, and I were talking to my IT director, they would tell me to convert/port the code myself...