Modify

Opened 4 years ago

Last modified 4 years ago

#21234 new defect

During "OK" click in preference window. Should catch errors (Tag Checker Data Sources) without closing the preference window.

Reported by: reichg Owned by: team
Priority: normal Milestone:
Component: Core Version:
Keywords: Cc:

Description

Reproduce:

1) Open JOSM and go to edit -> preferences -> Data Validator
2) Input invalid text into the "Data Sources" field within the Tag Checker Test
3) Click "OK".
4) Notice error pop up but the whole preferences window closes.

Expected Behavior:

1) Input invalid text into the "Data Sources" field within the Tag Checker Test
2) Click "OK".
3) Notice error pop up
4) exit error popup and keep preference window open.

In general while running through validation during the "OK" button click we should keep the preference window open so the user doesn't have to come back to preferences to fix the issue.

Attachments (0)

Change History (6)

comment:1 by reichg, 4 years ago

Cc: Don-vip added

comment:2 by Don-vip, 4 years ago

Cc: Don-vip removed
Milestone: 21.09

Please don't CC us and don't set the milestone by yourself. This is up to us.

in reply to:  2 ; comment:3 by reichg, 4 years ago

Replying to Don-vip:

Please don't CC us and don't set the milestone by yourself. This is up to us.

I received some contradictory information saying I could cc maintainers and set milestones. I will not do this in the future.

in reply to:  3 ; comment:4 by skyper, 4 years ago

Replying to reichg:

Replying to Don-vip:

Please don't CC us and don't set the milestone by yourself. This is up to us.

I received some contradictory information saying I could cc maintainers and set milestones. I will not do this in the future.

Think you mixed up some information. Depending on the user's settings cc might be useful if you ask a direct question to a person.
Dirk Stoecker wrote that it is ok to set a milestone for tickets with a patch ready for commit. In any other situation, please, leave it untouched (empty).

comment:5 by skyper, 4 years ago

I had the same problems some hours ago:

Relative:URL: ^/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2021-08-24 02:43:50 +0200 (Tue, 24 Aug 2021)
Revision:18173
Build-Date:2021-08-24 01:31:24
URL:https://josm.openstreetmap.de/svn/trunk

Identification: JOSM/1.5 (18173 en) Linux Debian GNU/Linux 11 (bullseye)
Java version: 17-ea+19-Debian-1, Debian, OpenJDK 64-Bit Server VM
Look and Feel: com.formdev.flatlaf.FlatDarculaLaf
Desktop environment: GNOME

Plugins:
+ flatlaf (35799)

in reply to:  4 comment:6 by reichg, 4 years ago

Replying to skyper:

Replying to reichg:

Replying to Don-vip:

Please don't CC us and don't set the milestone by yourself. This is up to us.

I received some contradictory information saying I could cc maintainers and set milestones. I will not do this in the future.

Think you mixed up some information. Depending on the user's settings cc might be useful if you ask a direct question to a person.
Dirk Stoecker wrote that it is ok to set a milestone for tickets with a patch ready for commit. In any other situation, please, leave it untouched (empty).

yeah, sure not a problem. How do you become a maintainer?

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The owner will remain team.
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from team to the specified user.
Next status will be 'needinfo'. The owner will be changed from team to reichg.
as duplicate The resolution will be set to duplicate. Next status will be 'closed'. The specified ticket will be cross-referenced with this ticket.
The owner will be changed from team to anonymous. Next status will be 'assigned'.

Add Comment


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