Skip to content

Commit

Permalink
Fix propagation of negations
Browse files Browse the repository at this point in the history
While _negations in the Applications type was being properly
initialised, it wasn't copied on merges. This simple fix merely extends
the list, which should work just fine i.e. I don't think it makes sense
to ensure uniqueness here.

Closes: madduck#44
Signed-off-by: martin f. krafft <[email protected]>
  • Loading branch information
madduck authored and simu committed Jul 26, 2022
1 parent 5246e6a commit 80d6d15
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions reclass/datatypes/applications.py
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,7 @@ def merge_unique(self, iterable):
self._items.remove(negation)
except ValueError:
pass
self._negations.extend(iterable._negations)
iterable = iterable.as_list()
for i in iterable:
self.append_if_new(i)
Expand Down

0 comments on commit 80d6d15

Please sign in to comment.