View Single Post
  #25  
Old November 8th, 2003, 07:25 AM
Fyron's Avatar

Fyron Fyron is offline
Shrapnel Fanatic
 
Join Date: Jul 2001
Location: Southern CA, USA
Posts: 18,394
Thanks: 0
Thanked 12 Times in 10 Posts
Fyron is an unknown quantity at this point
Default Re: PBW Emergency Site files v1.51 Finished and fixed.

You do realize that you can use linebreaks in echo commands, right? You do not need a separate one for every line of HTML...

Why is it still reading in from text files? What happened to the variables php file to include?

The admin script should NOT instantly auto-redirect to index.php! It should display a dialogue telling whether it was successful or not, and provide a link back to the index! I can not even see the errors it is trying to tell me about.

The Readme needs to mention that you need to set chmod permissions to 646 (allow public write) on all the "configuration" files to be edited by the admin scripts.

Things like "set password" would IMO be better handled as functions in the admin.php, rather than as separate files.

You should look into cookies... it is annoying to have to keep typing in the password over and over and over again...

Your complete Version 15 file has the index.php copied into the gamefile folder.

Please include a blank index.html file in each subfolder so that snooping browsers can't see their contents.

If the email for a player is blank, then the page should not generate a mailto: link for it. It should just display the (none) text, and that is it. No sense making a link that doesn't do anything.

[ November 08, 2003, 06:15: Message edited by: Imperator Fyron ]
__________________
It's not whether you win or lose that counts: it's how much pain you inflict along the way.
--- SpaceEmpires.net --- RSS --- SEnet ModWorks --- SEIV Modding 101 Tutorial
--- Join us in the #SpaceEmpires IRC channel on the Freenode IRC network.
--- Due to restrictively low sig limits, you must visit this link to view the rest of my signature.
Reply With Quote