Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

StateSetter.set "ignore" argument does not work #302

Open
johannesloibl opened this issue Nov 12, 2021 · 0 comments
Open

StateSetter.set "ignore" argument does not work #302

johannesloibl opened this issue Nov 12, 2021 · 0 comments

Comments

@johannesloibl
Copy link

johannesloibl commented Nov 12, 2021 •
edited
Loading

image

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
johannesloibl added a commit to johannesloibl/apptools that referenced this issue Jun 29, 2022
@johannesloibl
Fix for enthought#302
c688806
johannesloibl added a commit to johannesloibl/apptools that referenced this issue Jun 29, 2022
@johannesloibl
Fix for enthought#302 part 2
c752239 
Make sure ignore, first, last are propagated to nested set() calls
@johannesloibl johannesloibl mentioned this issue Jun 29, 2022
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@johannesloibl