Rev 91 0.6.4-pre2

[Getrix]

Rev 91 0.6.4-pre2

This code there was added "Merged rev 144..145 from trunk (nounderscore bug fix)"
The problem here is that it seems it was added code to "conf/MySQL.conf"
Following code:

 /*
 If you have $table_prefix Defined your tables will be prefixed as above with a _ added on
 if you dont want the _ added uncomment below
 */
 $nounderscore = FALSE;

This will make bots running with "botname_TABLE" create new tables with "botnameTABLE" because they miss "$nounderscore = TRUE;" in there config.
Atleast this happen to me.

[Thomas Juberg (thomas-juberg)]

Already have this one on my list, thank you for the report :)

[DJKRose (dj-krose)]

I just saw how it got fixed and would like to comment:

if ($nounderscore)
if ($nounderscore == TRUE)

Both lines do exactly 100% the same in 100% of all possible values and types!

If a cosmetic code change (refactoring) was intended, thats ok, even though I think the first one is better: An if-statement implies that the expression is always compared to true. The second statement says "if it's true that $nounderscore true is", which is somehow redundant ;) If something else was meant with the change, it' wasn't achieved. If it's working anyway, the change wasn't necessary.

btw... I didn't look into the rest of the code...

[Thomas Juberg (thomas-juberg)]

The change in question was not the fix, and should as you point out not have been commited.