Re: [Phpslash-devel] config.php vs. config-dist.php

[Ajay S. <ss...@od...>]

> Now for you guys input on this:
> 
> 1.  delete config.php from cvs entirely
> 
> 2.  move config-dist.php to config.php
> 
> 3.  change config.php to only contain text that phpslsash is not 
> configured and to read the install documentation.
> 
> All three seem to have pros and cons to me.

just for consistancy reasons, I would choose number 1 because we don't 
distribute a config.ini.php file, so right now there are three config 
files:

config.php
config-dist.php
config-dist.ini.php

config.php is just out of place and should be removed.  Then in the 
install docs, we should specify that the first thing you should do when 
installing phpslash is:

mv config-dist.php config.php
mv config-dist.ini.php config.ini.php

>  Another small issue - the poll list cache is not deleted when a new
> poll is added. If someone would tell me a quick regex for "string
> contains poll and not question_id" this would be fixed too.

does it have to be a regex:

if (strstr('poll',$string) and !strstr('question_id',$string)) {

> Ajay - Luis found a problem with the poll total votes if you manually
> put values into the form.  Maybe you could give him a hint.  I
> remember the old days with the bad counts and you fighting with this.

I'll have to dig into the code since I've been pretty much AWOL for the 
last year.  :)  I do have a working 7.0 installation at home so I'll try 
to recreate this bug.

later,
ajay

--------------------------------------------------------------------
Satyajot (Ajay) Sharma 
ss...@od...
--------------------------------------------------------------------