Modify

Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#18150 closed defect (worksforme)

option to ignore validator warnings is missing to me

Reported by: mkoniecz Owned by: mkoniecz
Priority: normal Milestone:
Component: Core validator Version:
Keywords: template_report ignore Cc:

Description

What steps will reproduce the problem?

  1. Run validator
  2. Encounter "Incomplete pedestrian crossing tagging. Required tags are highway=crossing|traffic_signals and crossing=*. (4)"
  3. Realize that I will fix once I walk there with StreetComplete and it is better to fix errors easier to fix with remote editing
  4. Become confusing by missing ignore option that I am sure that was somewhere in validator list in earlier versions

What is the expected result?

I can ignore groups and specific errors easier to fix during survey or that I reported as bug already.

What happens instead?

No ability to ignore.

Please provide any additional information below. Attach a screenshot if possible.

URL:https://josm.openstreetmap.de/svn/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2019-09-12 23:23:47 +0200 (Thu, 12 Sep 2019)
Build-Date:2019-09-12 21:28:35
Revision:15348
Relative:URL: ^/trunk

Identification: JOSM/1.5 (15348 en) Linux Ubuntu 16.04.6 LTS
Memory Usage: 389 MB / 869 MB (103 MB allocated, but free)
Java version: 1.8.0_201-b09, Oracle Corporation, Java HotSpot(TM) 64-Bit Server VM
Screen: :0.0 1920x1080
Maximum Screen Size: 1920x1080
libcommons-logging-java: libcommons-logging-java:all-1.2-1+build1
fonts-noto: fonts-noto:-
Dataset consistency test: No problems found

Plugins:
+ OpeningHoursEditor (34977)
+ PicLayer (35104)
+ buildings_tools (34982)
+ continuosDownload (82)
+ ejml (35049)
+ geotools (34908)
+ imagery_offset_db (34908)
+ jts (35064)
+ log4j (34908)
+ measurement (35051)
+ reverter (35084)
+ todo (30306)

Validator rules:
+ ${HOME}/Documents/install_moje/OSM software/josm/data/validator/deprecated.mapcss
+ ${HOME}/Documents/install_moje/OSM software/josm/data/validator/unnecessary.mapcss
+ ${HOME}/Documents/install_moje/OSM software/josm/data/validator/combinations.mapcss

Last errors/warnings:
- W: Failed to add ${HOME}/Documents/install_moje/OSM software/josm/data/validator/unnecessary.mapcss to tag checker
- W: java.nio.file.NoSuchFileException: ${HOME}/Documents/install_moje/OSM software/josm/data/validator/unnecessary.mapcss
- W: Failed to add ${HOME}/Documents/install_moje/OSM software/josm/data/validator/combinations.mapcss to tag checker
- W: java.nio.file.NoSuchFileException: ${HOME}/Documents/install_moje/OSM software/josm/data/validator/combinations.mapcss
- W: javax.script.ScriptException: An error occurred during evaluation of the value "Mo-Fr 11:00-16:00;Sa-Su 11:30-14:30;SH off". Please submit a pull request: https://github.com/ypid/opening_hours.js. no SH definition in <eval> at line number 46 at column number 4733. Cause: jdk.nashorn.internal.runtime.ECMAException: An error occurred during evaluation of the value "Mo-Fr 11:00-16:00;Sa-Su 11:30-14:30;SH off". Please submit a pull request: https://github.com/ypid/opening_hours.js. no SH definition
- E: javax.script.ScriptException: An error occurred during evaluation of the value "Mo-Fr 11:00-16:00;Sa-Su 11:30-14:30;SH off". Please submit a pull request: https://github.com/ypid/opening_hours.js. no SH definition in <eval> at line number 46 at column number 4733. Cause: jdk.nashorn.internal.runtime.ECMAException: An error occurred during evaluation of the value "Mo-Fr 11:00-16:00;Sa-Su 11:30-14:30;SH off". Please submit a pull request: https://github.com/ypid/opening_hours.js. no SH definition
- W: javax.script.ScriptException: An error occurred during evaluation of the value "Mo-Fr 11:00-16:00;Sa-Su 11:30-14:30;SH off". Please submit a pull request: https://github.com/ypid/opening_hours.js. no SH definition in <eval> at line number 46 at column number 4733. Cause: jdk.nashorn.internal.runtime.ECMAException: An error occurred during evaluation of the value "Mo-Fr 11:00-16:00;Sa-Su 11:30-14:30;SH off". Please submit a pull request: https://github.com/ypid/opening_hours.js. no SH definition
- E: javax.script.ScriptException: An error occurred during evaluation of the value "Mo-Fr 11:00-16:00;Sa-Su 11:30-14:30;SH off". Please submit a pull request: https://github.com/ypid/opening_hours.js. no SH definition in <eval> at line number 46 at column number 4733. Cause: jdk.nashorn.internal.runtime.ECMAException: An error occurred during evaluation of the value "Mo-Fr 11:00-16:00;Sa-Su 11:30-14:30;SH off". Please submit a pull request: https://github.com/ypid/opening_hours.js. no SH definition
- W: javax.script.ScriptException: An error occurred during evaluation of the value "Mo-Fr 11:00-16:00;Sa-Su 11:30-14:30;SH off". Please submit a pull request: https://github.com/ypid/opening_hours.js. no SH definition in <eval> at line number 46 at column number 4733. Cause: jdk.nashorn.internal.runtime.ECMAException: An error occurred during evaluation of the value "Mo-Fr 11:00-16:00;Sa-Su 11:30-14:30;SH off". Please submit a pull request: https://github.com/ypid/opening_hours.js. no SH definition
- E: javax.script.ScriptException: An error occurred during evaluation of the value "Mo-Fr 11:00-16:00;Sa-Su 11:30-14:30;SH off". Please submit a pull request: https://github.com/ypid/opening_hours.js. no SH definition in <eval> at line number 46 at column number 4733. Cause: jdk.nashorn.internal.runtime.ECMAException: An error occurred during evaluation of the value "Mo-Fr 11:00-16:00;Sa-Su 11:30-14:30;SH off". Please submit a pull request: https://github.com/ypid/opening_hours.js. no SH definition

Attachments (3)

ignore.png (14.4 KB ) - added by Don-vip 6 years ago.
Selection_006.png (30.4 KB ) - added by mkoniecz 6 years ago.
18150.PNG (16.4 KB ) - added by Klumbumbus 6 years ago.

Download all attachments as: .zip

Change History (11)

comment:1 by Don-vip, 6 years ago

Component: CoreCore validator
Keywords: ignore added

by Don-vip, 6 years ago

Attachment: ignore.png added

comment:2 by Don-vip, 6 years ago

Owner: changed from team to mkoniecz
Status: newneedinfo

What's wrong with ignore button?

comment:3 by mkoniecz, 6 years ago

It is not present in my JOSM.

Also, right-click used to be possible on groups (folders) like "Crossing buildings" - now it is available only for individual items.

I retested with

URL:https://josm.openstreetmap.de/svn/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2019-09-19 00:20:17 +0200 (Thu, 19 Sep 2019)
Build-Date:2019-09-19 01:30:57
Revision:15355
Relative:URL: ^/trunk
Last edited 6 years ago by mkoniecz (previous) (diff)

by mkoniecz, 6 years ago

Attachment: Selection_006.png added

comment:4 by Don-vip, 6 years ago

Did you set validator.ignore to false in the advanced preferences?

comment:5 by mkoniecz, 6 years ago

I have it set in this way but I do not remember doing this and I never wanted to get rid of "ignore" dialog.

After flipping it and restart I got ignore interface back.

Thanks!

But I still have no idea how this setting flipped.

Last edited 6 years ago by mkoniecz (previous) (diff)

comment:6 by Don-vip, 6 years ago

Resolution: worksforme
Status: needinfoclosed

Strange. Reopen the issue if some JOSM feature or plugin accidentally disables this property.

by Klumbumbus, 6 years ago

Attachment: 18150.PNG added

in reply to:  5 comment:7 by Klumbumbus, 6 years ago

Replying to mkoniecz:

But I still have no idea how this setting flipped.

It's the first check box in the preference settings. Maybe you accidently hit it when you wanted to switch the tab:

comment:8 by Klumbumbus, 6 years ago

(BTW, this setting needs a JOSM restart even if JOSM doesn't ask for it.)

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain mkoniecz.
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.