Changes between Initial Version and Version 1 of Ticket #21825, comment 38


Ignore:
Timestamp:
2022-03-18T13:43:53+01:00 (4 years ago)
Author:
taylor.smock

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #21825, comment 38

    initial v1  
    44The tests aren't actually run when you change the selection in the member list (or at least, it didn't hit my breakpoint). They are run when you change the selection in the TagEditorModel (as in, the TagEditorModel fires a table changed event regardless of whether or not the ''table changed''). This does seem like a bug -- I'll open a ticket for this. It will probably take quite a bit of investigation (the code hasn't been touched since 2010, so who knows what is depending upon the current behavior).
    55> I'd prefer to minimize the patch to really check for empty relations and maybe open new tickets for the rest.
    6 I'll go ahead and do this.
     6I'll go ahead and do this. See #21950, #21951.
    77> A general problem with the displaying of the `TestError`:
    88> I see e.g. "Multipolygon rings share node with 2 more problems", a rather confusing message. I tried to click on that message to get more info. Same problem with a text like "Intersection between multipolygon ways with 1 more problem"