changing setting values to null creates crash

[groundup]

If you change some variables to null in the admin CP - settings, it will cause for many errors:

$TopAmmount - creates parse errors among other things
$MembersPerPage - creates a timeout for memberlist.php
$maxdisplay - 2: Division by zero: MessageIndex.php ln 61
$maxmessagedisplay - 2: Division by zero: MessageIndex.php ln 228
$ClickLogTime - creates parse errors and many warnings
$max_log_days_old - "                           "
$fadertime - "                              "
$timeout - "          "

those are the ones that I have tried for now.  I have not tried placing 0's in.

[Jeff Lewis]

Ok it MIGHT be called a bug but why are you setting them to null?  They hve default values and when changed through the admin panel they are given proper values not null...

[David]

I went to the admin panel and changed "Number of Top Posters for Members List" to be empty and it wrote the Settings.php file with no value for that variable.  Then it crashed the board.

[Jeff Lewis]

Why would you change it to empty?

[David]

Why would you change it to empty?

The point isn't to ask why I would do such a thing but rather this is something that needs to be fixed.  The software should be smarter than a dumb user.  In our application, BoardNation, we may have really dumb users who do something like this.

[Jeff Lewis]

No, my point is to ask why.  I acknowledge the issue but why did you guys do that in the first place?

[David]

I did it because I saw this thread an decided to see what it did.  I can't speak for Josh though.

[seth4chaos]

I agree with Jeff. There are a lot of other stuff that has to be fix of whatever and it's not something a user can do so keep your admins informed and let Jeff and the other fix some real problems

[Jeff Lewis]

See, that is exactly my thinking.  I mean, you're right that it shouldn't do that but what NORMAL admin inputs blank values?

[groundup]

i did it to test what some dumbass users will do on Boardnation.  I think that i will come up with a JS fix though

[Champ]

Jeff, if you want a serious answer, I changed the time offset by leaving this field blank because I assumed it would be the same like 0. Fortunately for me the old settings.php was still on the server renamed to settings_bak.php. People should look for it, if its still there, they can fix it by deleting the settings.php and rename the settings_bak.php to settings.php and of course don't make the same mistake again

[Jeff Lewis]

Hehe thanks - serious answers I like  Naturally it will be fixed.