Re: A Review Request

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On May 18, 2011, at 3:22 PM, tedd wrote:

> Hi gang:
> 
> I am considering providing PHP code to the general public via my website
> 
> This is my first attempt:
> 
> http://sperling.com/php/authorization/
> 
> What do you people think?
> 
> Cheers,
> 
> tedd
> 
> -- 
> -------
> http://sperling.com/
> 

I can say I really don't like your bracing style.

I don't see a reason to use a form submit to go back to the login page, instead I normally present errors on page.

Other then that, looks good. Maybe redirect http://sperling.com/php/ to an index of examples?

Regards,

-Josh
____________________________________
Joshua Kehn | Josh.kehn@xxxxxxxxx
http://joshuakehn.com



[Index of Archives]     [PHP Home]     [Apache Users]     [PHP on Windows]     [Kernel Newbies]     [PHP Install]     [PHP Classes]     [Pear]     [Postgresql]     [Postgresql PHP]     [PHP on Windows]     [PHP Database Programming]     [PHP SOAP]

  Powered by Linux