While developing a module for version 2.18 I have come across a loading of coding that to be quite frank, is not required or is totally overboard. One such class is xoopsFormRadio and the new method of securing this class.

While the first fact is that it no longer works,

Example you have to change:

Quote:

To:

Quote:

As under the newer method it will never truly equal.

But my main problem with the changes is that you are not treating the variables in the correct manner, or I am really failing to see what real benefits are from the changes made.

Example:



$value is and should be treated like an INT. Here we are treating that variable in the same manner as we would treat a text value.

In this case it should have been:



Actually with the case in mention, it would have been better to treat these values as boolean instead.

Hello John,

I have some scripts where the value "returned" by the option is not an integer but a string.
I believe that's the reason of this no ?

Finally, I also believe that when a value is returned from a form's field and when this value is surrounded by quotes, it is considered (and converted) as a string by Php (it may depends of your Php version).

Bye,
Hervé