Modify

Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#17354 closed defect (fixed)

relation panel dont shows all relations in revert sometimes

Reported by: MKnight Owned by: Upliner
Priority: normal Milestone: 19.03
Component: Core Version: tested
Keywords: template_report Cc:

Description

What steps will reproduce the problem?

  1. close relation-panel
  2. revert a CS with deleted relations for example CS67295231
  3. open relation panel, there are 4 relations
  1. open relation panel
  2. revert a CS with deleted relations for example CS67295231
  3. look at relation window, there are 2 relations

Attachments (1)

17354.patch (695 bytes ) - added by GerdP 6 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 by Don-vip, 6 years ago

Component: CorePlugin reverter
Owner: changed from team to Upliner

by GerdP, 6 years ago

Attachment: 17354.patch added

comment:2 by GerdP, 6 years ago

@Vincent: I can reproduce this but I see that the dataset always contains 5 relations, so maybe the problem is that the relation list window is not in sync with the dataset. In my case only one relation is shown when the panel was open (2nd list of steps)
Closing and opening the relation list window shows 4 relations, searching for type:relation with checkbox "all objects" enabled shows finds 5 relations.
The attached patch seems to fix this so that always 4 relations are shown. One of the relations is very incomplete, so I guess it is intended that it doesn't appear in this window: Relation 7589682 exists only because it is a member of Relation 8329249.
What do you think?

comment:3 by GerdP, 6 years ago

Resolution: fixed
Status: newclosed

In 14886/josm:

fix #17354: Relation list is not always updated

The problem was that RelationListDialog.otherDatasetChange(AbstractDatasetChangedEvent event) did nothing.
The dialog therefore misses events of type PRIMITIVE_FLAGS_CHANGED and if the flags of a relation are changed this should change the list.
When the dialog is build from scratch it looks at the relations with status complete and shows them.

comment:4 by GerdP, 6 years ago

Component: Plugin reverterCore

Problem was not related to plugin reverter.

comment:5 by Don-vip, 6 years ago

Milestone: 19.03

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Upliner.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.