Modify ↓

Opened 11 years ago

Closed 11 years ago

Last modified 5 years ago

#8517 closed enhancement (fixed)

Validation results are unsorted

Reported by: windu.2b Owned by: team
Priority: minor Milestone: 13.12
Component: Core validator Version: latest
Keywords: Cc:

Description (last modified by Don-vip)

The validation windows displays unsorted elements, which it's not always easy to read:

unsorted elements ('empty role found')

Attachments (2)

unsorted_validation_results.png​ (9.2 KB ) - added by windu.2b 11 years ago.
unsorted elements ('empty role found')
sortValidatorDialog.patch​ (10.3 KB ) - added by windu.2b 11 years ago.
sort validator dialog

Download all attachments as: .zip

Change History (11)

by windu.2b, 11 years ago

Attachment: unsorted_validation_results.png​ added

unsorted elements ('empty role found')

by windu.2b, 11 years ago

Attachment: sortValidatorDialog.patch​ added

sort validator dialog

comment:1 by windu.2b, 11 years ago

see the patch

comment:2 by Don-vip, 11 years ago

Description: modified (diff)
Summary: Validation results are unsorted → [Patch] Validation results are unsorted

comment:3 by Don-vip, 11 years ago

Resolution: → fixed
Status: new → closed

In 5783/josm:

fix #8517 - Sort validator results (based on patch by windu.2b)

comment:4 by Don-vip, 11 years ago

Resolution: fixed
Status: closed → reopened
Summary: [Patch] Validation results are unsorted → Validation results are unsorted
Type: defect → enhancement

The patch does not work. TreeSet cannot be used in MultiMap because it does not respect the general contract of Set (see Javadoc). I have reverted this change in r5822 as it caused a major bug (see #8546).

Last edited 11 years ago by Don-vip (previous) (diff)

comment:5 by naoliv, 11 years ago

Ticket #9415 has been marked as a duplicate of this ticket.

comment:6 by simon04, 11 years ago

Resolution: → fixed
Status: reopened → closed

In 6468/josm:

fix #8517 - sort validation results

This is accomplished by sorting the error list before constructing the tree.

comment:7 by Don-vip, 11 years ago

Milestone: → 13.12 (6502)

comment:8 by stoecker, 7 years ago

Milestone: 13.12 (6502) → 13.12

Milestone renamed

comment:9 by GerdP, 5 years ago

In 14973/josm:

simplify code, improve performance: Remove obsolete method sortErrors()

The sorting was introduced to fix #8517 but is no longer needed since errors are reordered in OsmValidator.getErrorsBySeverityMessageDescription() when the tree is created. The obsolete sort takes much longer and doesn't improve anything.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
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 .
Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.