Modify ↓
Opened 13 years ago
Closed 13 years ago
#7745 closed defect (fixed)
[PATCH] TagCheck reports an identical warning multiple times after viewing validator preferences
Reported by: | mrwojo | Owned by: | team |
---|---|---|---|
Priority: | minor | Milestone: | |
Component: | Core validator | Version: | |
Keywords: | Cc: |
Description
With r5210
Repro:
- Create a simple way tagged
highway=service
andbarrier=wall
. - Validate. You should see a warning: "barrier used on a way - Suspicious tag/value combinations (1)".
- Open JOSM's Preferences dialog. Open the Validator settings page. Just press OK.
- Validate. The number of warnings has incremented: "barrier used on a way - Suspicious tag/value combinations (2)". The single problematic way is being reported twice with an identical warning.
TagCheck.initializeData()
is called once on startup and when the user presses "OK" on the Validator Preferences dialog. checkerData
(and a few other static lists) is appended to every time initializeData()
is called, but it's never reset anywhere, so checks are added redundantly without bound.
Attachments (1)
Note:
See TracTickets
for help on using tickets.
initializeData: clear the static lists first