Login   Register  
PHP Classes
elePHPant
Icontem

This one looking good but not covered all aspects of the cont...

Recommend this page to a friend!
Stumble It! Stumble It! Bookmark in del.icio.us Bookmark in del.icio.us

      Auto CMS  >  All threads  >  This one looking good but not covered all aspects of the cont...  >  (Un) Subscribe thread alerts  
Subject:This one looking good but not covered all aspects of the cont...
Summary:Package rating comment
Messages:4
Author:Suresh Pandian
Date:2009-09-04 15:37:51
Update:2009-09-06 14:57:27
 

Suresh Pandian rated this package as follows:

Utility: Not sure
Consistency: Not sure
Documentation: Sufficient

  1. This one looking good but not covered all aspects of the cont...   Reply  
Picture of Suresh Pandian
Suresh Pandian
2009-09-04 15:37:51
This one looking good but not covered all aspects of the content. I appreciate that you are changing the permissions to 600 after writing the content.

Due to the entered content stored in a file as defined constants, you should make sure that the quotes (' & ") in the content are properly escaped.

Otherwise, it will show you the below error whenever the admin entering the content with some quotes.

Parse error: syntax error, unexpected T_CONSTANT_ENCAPSED_STRING in /home/testistore/public_html/test/autocms/vars.php on line 1

Please make necessary changes and update it.

If this is fixed, then the non-programmer can use it without any issues.
Thanks!

  2. Re: This one looking good but not covered all aspects of the cont...   Reply  
Picture of Suresh Pandian
Suresh Pandian
2009-09-04 17:37:50 - In reply to message 1 from Suresh Pandian
This one looking good but not covered all aspects of the content. I appreciate that you are changing the permissions to 600 after writing the content.

Due to the entered content stored in a file as defined constants, you should make sure that the quotes (' & ") in the content are properly escaped.

Otherwise, it will show you the below error whenever the admin entering the content with some quotes.

Parse error: syntax error, unexpected T_CONSTANT_ENCAPSED_STRING in /home/testistore/public_html/test/autocms/vars.php on line 1

Please make necessary changes and update it.

If this is fixed, then the non-programmer can use it without any issues.
Thanks!

  3. Re: This one looking good but not covered all aspects of the con   Reply  
Picture of Roberto Aleman
Roberto Aleman
2009-09-05 14:12:38 - In reply to message 2 from Suresh Pandian
Thanks, pleases have many comments as the yours, since that helps to that the new versions improve the code, since they indicate the important points to improve. In future versions this by applying your comments, thanks

  4. Re: This one looking good but not covered all aspects of the con   Reply  
Picture of Roberto Aleman
Roberto Aleman
2009-09-06 14:57:27 - In reply to message 1 from Suresh Pandian
How much PHP clases you have published? You the tight curls by criticizing my work but see that it collaborate with the community of phpclasses.org... must contribute somewhat but that dirty comments on the work of other, Indian dolt