StateSetter.set "ignore" argument does not work
Nobody has claimed this yet.
Assessment
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Newbie friendliness
- 35/100
Research direction
Locate StateSetter.set and the nested _do_instance calls, then reproduce the list-removal case where a missing item raises ValueError. Trace how the ignore, first, and last arguments are passed into nested set calls; done means missing list items follow the intended ignore behavior and all three arguments are preserved in nested calls.
Written by the indexing model from the issue text.
Description
state_keys is a list. Trying to remove an item from a list that is not contained, raises a ValueError, not a KeyError.
Also StateSetter.set arguments "ignore,first,last" are not properly propagated to nested calls to set, e.g. in _do_instance
- Dominant language
- Python
- Stars
- 38
- Forks
- 21
- PR merge metrics
- No merged PRs in 30d
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from enthought/apptools
-
type: bug
Difficulty 3/5 1-2 days Newbie friendliness 35/100
-
Difficulty 4/5 3-5 days Newbie friendliness 42/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 45/100
-
Difficulty 3/5 1-2 days Newbie friendliness 45/100
-
Difficulty 3/5 1-2 days Newbie friendliness 48/100
All issues in enthought/apptools
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
-
enhancement
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100