Add a checked_array() to compare arrays like checked()

[sudar]

The checked() function checks whether two parameters passed to it are equal and then outputs checked='checked', which can be used in forms having checkboxes or radio buttons.

Sometimes (particularly for checkboxes), we might need to check for multiple values. So I am proposing a change to the checked() function.

It can first check whether the default value is an array and if yes, it can return checked='checked' if the current value is present in the default array.

I have attached a patch with this change.

[sudar]

patch for checked function

[westi]

Could you give a more detailed example of what you are trying to do that the checked function doesn't support.

An example simple plugin which has an page with a check box on would be fine.

I would like to get a clear understanding of the issue that presents.

[sudar]

Here is a scenario which explains what I am try to do.

Let's say I am writing a Plugin, which is displaying all top level pages (as checkboxes) and the user will be selecting some of them. I will have the name of the checkboxes as an array (eg: selected_pages[]), so that I can easily handle the post variables when the form is submitted.

Now the next time, when the page is displayed, I have to keep the checkboxes that were already selected as checked. I will have the selected pages as an array, and instead of looping through every time to find out whether the current page is there in the array or not, I can pass the current id and the array to the checked function which can print checked='checked', if the current page id is present in the array.

I have attached a small Plugin to demonstrate this.

(sorry for the delay in reply, I didn't noticed the status change)

[sudar]

Sample Plugin to demonstrate checked array function

[hakre]

Updated the template patch according to the principles in the plugin. Original code had repeated parts I was able to remove. There is no need for an additional function, checked() and selected() accepted already arrays based on their definition (they converted them to strings which does not make sense).

[Denis-de-Bernardy]

I like neither patches, personally. On the one hand side, we're checking using the === operator, whereas on the other we're using in_array(), which amounts to using the == operator.

the === was introduced in order to distinguish an empty string from a numerical zero.

[shidouhikari]

So, we can´t use in_array() because of the operstor?

That's sad, I liked the 2nd patch.

hakre, I believe the (string) is meant for ints, and not for arrays.

[nacin]

in_array() has a strict parameter.

[westi]

Until we need this in core this is WONTFIX IMHO - to much of an edge case.

[nacin]

Agreed.