Resolve "TQTaggable list overwriting"
Merge request reports
Activity
added bug - label
added 539 commits
-
63a894aa...62e6cfb3 - 538 commits from branch
master
- 48b8e260 - Merge branch 'master' into '179-tqtaggable-list-overwriting'
-
63a894aa...62e6cfb3 - 538 commits from branch
- Resolved by Carsten Burgard
@kolehman: As this is only adding a test, I suppose this could be merged?
mentioned in commit 62729372
mentioned in merge request !687 (merged)
Ok, merging this was not a good idea (I also notice now that the CI didn't even pass in the MR). This is a test that specifies what would be reasonable behaviour. Unfortunately, this is not how TQTaggable behaves. That's why master fails right now.
We should revert this merge in my opinion.
In terms of a long-term fix: I remember now that I tried to fix the behaviour at the time, but I didn't manage easily, because of the way that lists of TQTaggables are stored. I left this MR open, because it defines the desired behaviour in my opinion.
Ignore previous comment. Instead, a fix is in !687 (merged).
mentioned in commit 15c73168