Skip to content

Fix SetParameterString for listview parameters

Cédric Traizet requested to merge 2246_setparameterstring_listview into develop

Summary

Closes #2246 (closed)

Fix SetParameterString for listview parameters, it now has the same behavior as SetParameterStringList if the input list has one element, i.e.

app.SetParameterString("list_view_param", "value")
app.SetParameterStringList("list_view_param", ["value"])

now produce the same results

Implementation Details

ListViewParameter::FromString has been modified to use SetSelectedNames instead of SetValue (which works on key names).

Copyright

The copyright owner is CNES and has signed the ORFEO ToolBox Contributor License Agreement.

Check before merging:

  • All discussions are resolved
  • At least 2 👍 votes from core developers, no 👎 vote.
  • The feature branch is (reasonably) up-to-date with the base branch
  • Dashboard is green
  • Copyright owner has signed the ORFEO ToolBox Contributor License Agreement
  • Optionally, run git diff develop... -U0 --no-color | clang-format-diff.py -p1 -i on latest changes and commit

Merge request reports