Re: [Phpslash-devel] back-end install wizard

[Luis M <le...@ho...>]

>Just a few things to keep in mind (I have to run; I will continue later 
>with this):
>
>1. when there is a database already setup but there is no table in it, the 
>script config_setup.php fails
>2. if the server cannot write to the config.ini.php file, the 
>config_setup.php fails (no error printed)
>3. config_setup.php seems to be very fragile... in other words, lots of 
>things can go wrong using $var+"string" to call a function. Especially 
>since $var is assigned from something submitted from a browser... 
>suggesting that it might fail if a browser mangles the strings or if users 
>inject something... etc... i'm sure we can find a million things why this 
>would not be wise. However, it works for now and we might change things 
>later to improve it.
>4. users have to be reminded to remove the config_setup.php file from the 
>public site inmediately after setting the site up... if the script could 
>remove itself it would be even better
>5. if writing to the server's config.ini.php is not an option, then i will 
>incorporate the code to push the file to the user's browser (as you 
>suggested Joe).
>6. ... i have a lot more things, but I'll try to keep them in mind...


A few things to bake your noodles..:
a. apache segfaults (php 4.3.x and apache 1.3.x) with latest version (cvs) 
of phpslas-dev. The error is:
[ error ]
[Thu Nov 13 21:17:40 2003] [error] PHP Fatal error:  Class jpcachedb:  
Cannot inherit from undefined class db_sql in 
/home/Shared/php/phpslash-dev/include/class/jpcache/type/phplib.php on line 
13
[Thu Nov 13 21:18:27 2003] [notice] child pid 29835 exit signal Segmentation 
fault (11)
[ end error ]
b. if the database is already created, but the tables don't exist, the 
config_setup.php file doesn't work correctly. I modified it so that it picks 
the username/password from the already modified config.ini.php file, but 
this is not enough. Will look into that later. If anybody want the latest 
version of config_setup.php please let me know. I have not committed it to 
the CVS -- too experimental at this point.
c. if the database is not already present, then it asks for another user 
that has rights to create/alter a database for mysql, which is nice, but 
after creating the database and populating it with the 
tables/0.8/slash-all.sql file, going to index.php fails because phplib is 
not set correctly :-D I guess Joe already knows this and I know the releases 
already have a working phplib directory inside "include", however, I think 
it should be a nice thing to allow the user to specify the path to their 
phplib directory when using config_setup.php. In a way that if the user 
leaves this blank, the default is used -- the way things work right not if 
one doesn't setup a phplib in config.ini.php.

That's it for now... I see the M8 release is getting more and more 
sophisticated (htmlarea3 and so on...). Keep up the good work!



----)(-----
Luis Mondesi
System Administrator
LatinoMixed.com

le...@ho...

"...The Mac does this so smoothly, it feels like an extension of your mind." 
- Paula Speer, MacWorld Magazine 2003-04

Public signature: http://www.latinomixed.com/lems1/public-a.asc

_________________________________________________________________
MSN Search, le moteur de recherche qui pense comme vous ! 
http://search.msn.fr/worldwide.asp