_registration and possibly other configuration storage tables

[plaukas]

Hello,

Just look at _registration table. Each registration setting name is in comlumn name of this table, and value - in coresponding single line field. And most values are multiple, separated with "|" ...

Storing settings like this is ugly as hell and totally not appropriate for commercial product .

So please, can we have a proper settings table with (name,value) and simple api like isEnabled("cityRequired") or getValue("smtpHost"), setValue("smtHost", $server). Hope you understand the idea. More elegant, simple and clean code. And ONLY ONE TABLE for all global settings (site, registration settings and whatever else, didn't review all settings storage tables).

Realy simple to implement, a bit more complex to migrate everything, but a must do thing.

[Eric Barnes]

Yes this has been thought about but as you say "more complex to migrate everything".

That is the tricky part and the reason it is still the way it is. At the end of the day it is ugly but it works. So it hasn't been a high priority.

[Lhotch]

Originally Posted by plaukas

Realy simple to implement, a bit more complex to migrate everything, but a must do thing.

Why is this a "must do"?

[plaukas]

For ease if further development, simpler and more clear database structure (less redundant tables).

Yes, it works now. But sometimes.. For example, I needed to check if captchas are enabled for user registration.

I have choices:
1. write inline query
2. write function for example captchaEnabled, put it somewhere and use.
3. or, like someone DID, just decide proper captcha validation is just not important, because it's just too much to code:

//the form has been submitted so lets validate it and update the info
//is captcha enabled?
if(isset($_POST['captcha']) && $_POST['captcha']=="Y")
{
.......
}

in file userjoin.php

What actualy makes captchas easily bypassable. So now you see where ugly data structures leads?

PS: I hope it gets noticed (I mean captcha check) and will be fixed in next release