WFSFilterBuilder - Add fields support for titles

[belug]

The idea is to add the field support to this widget. So the fields could be renamed after something more readable than feature attributes names.

As an example :

Database field : "opt003u" Field configuration title : "Number of Plants"

Currently the widget writes "opt003u" in the search combobox and in the Grid columns name. It could be nicer to write "Number of Plants".

[belug]

Here I added the support for the fields. I'm not sure it's the best way but it's working.

But it gets the fields from the config, and add the title to the attribute store defined from the WFS DescribeFeatureType. If there is no title defined it set the title to the attribute name by default.

The title is used by the field combobox to create the filter, the combobox of the quick filter and in the grid columns name.

[adube]

Here's a review. The patch looks good to me. It would be nice to leave a notice in the widget documentation that it supports reading the resource field titles for that purpose.

What do you think ?

[belug]

Here's the file to modify the documentation so it now talks about setting up fields to the ressource.

[adube]

belub, could you please also provide an update of the sample showing that new feature ?

(sorry for using "feature" again)

[adube]

"belug", sorry.

[belug]

While adding it to the sample I've noticed two bugs, the first was in my patch, the fields titles could be erased by other ressources before the atributes would be loaded so the titles would not change, for I made the object containing the titles golbal and that it would not be erased.

The thing with this solution is that if a field is declared in one ressource it could change the title in an other one where it is not declared and that if it is declared in two ressources with defferent titles it is possible that only one title will be used for both.

The second bug was in te field object in PHP if the field was declared with no options, the code would break. So I added a validation if there is no options the foreach will have an empty array to loop in.

[adube]

Fixed in r1258, with minor modifications.

Thanks a lot for the patch.