0004215: Default values for radio buttons in attributes not honored

Notes
(0011780) lukejackson (reporter) 25-03-06 10:34	One way to fix it is to make the following changes in admin/subscribelib2.php, search for all three instances of: if (isset($_POST[$fieldname])) and replace them with: if ($_POST[$fieldname] != 0) search for all three instances of : else if (isset($data[$attr["id"]])) and replace them with: else if ($data[$attr["id"]] != "") it can be seen that only a few lines before this code, there is a check to see whether the two variables are not set, and if so to set them. this then breaks the code immediately below! the other way to fix it therefore would be to remove that code and then make any necessary changes to the following code that is affected. the above solution is much simpler.

(0011781) lukejackson (reporter) 25-03-06 10:35	this is also a problem for check box groups and drop down lists. the above solution fixes these too.

(0030384) h2b2 (reporter) 12-08-07 01:16	This issue may need reviewing as it has been raised again for ver. 2.10.4. See this forum thread: http://forums.phplist.com/viewtopic.php?t=11137 [^]

(0030385) michiel (administrator) 12-08-07 02:20	instead of "isset" it should use "!empty" which will evaluate to false, if the var is 0 or "", but it will not throw warnings on non-initialised variables. in phplist a lot of code is in the form if ($var) { something } which for PHP is not a nice way to do it, as it has security implications, so in those cases it's better to use if (!empty($var)) { something } Hernan, can you check this ?

(0030455) hernan (developer) 13-08-07 17:48	Yes, changed two instances of if ($var) { something } to if (!empty($var)) { something } For select and radio butttons.