Modify ↓

Opened 14 years ago

Closed 14 years ago

#3336 closed defect (fixed)

[PATCH] WMS plugin does not check if user aborts save dialog

Reported by: plaicy Owned by: team
Priority: minor Milestone:
Component: Core imagery Version:
Keywords: NullPointerException Cc:

Description

  1. start JOSM
  2. create new data layer
  3. add empty wms data layer
  4. right click on new wms layer and choose "save in file"
  5. abort dialog

results in

java.lang.NullPointerException
        at java.io.FileOutputStream.<init>(FileOutputStream.java:172)
        at java.io.FileOutputStream.<init>(FileOutputStream.java:131)
        at wmsplugin.WMSLayer$SaveWmsAction.actionPerformed(WMSLayer.java:303)
        at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:1995)
        at javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2318)
        at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:387)
        at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:242)
        at javax.swing.AbstractButton.doClick(AbstractButton.java:357)
        at javax.swing.plaf.basic.BasicMenuItemUI.doClick(BasicMenuItemUI.java:1225)
        at javax.swing.plaf.basic.BasicMenuItemUI$Handler.mouseReleased(BasicMenuItemUI.java:1266)
        at java.awt.Component.processMouseEvent(Component.java:6216)
        at javax.swing.JComponent.processMouseEvent(JComponent.java:3265)
        at java.awt.Component.processEvent(Component.java:5981)
        at java.awt.Container.processEvent(Container.java:2041)
        at java.awt.Component.dispatchEventImpl(Component.java:4583)
        at java.awt.Container.dispatchEventImpl(Container.java:2099)
        at java.awt.Component.dispatchEvent(Component.java:4413)
        at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4556)
        at java.awt.LightweightDispatcher.processMouseEvent(Container.java:4220)
        at java.awt.LightweightDispatcher.dispatchEvent(Container.java:4150)
        at java.awt.Container.dispatchEventImpl(Container.java:2085)
        at java.awt.Window.dispatchEventImpl(Window.java:2475)
        at java.awt.Component.dispatchEvent(Component.java:4413)
        at java.awt.EventQueue.dispatchEvent(EventQueue.java:599)
        at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:269)
        at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:184)
        at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:174)
        at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:169)
        at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:161)
        at java.awt.EventDispatchThread.run(EventDispatchThread.java:122)

No "send bug dialog" is created, because wmsplugin catches all exceptions.

Tested with r1997 and wmsplugin version 17278.

Attachments (1)

wmssave.patch​ (472 bytes) - added by xeen 14 years ago.

Download all attachments as: .zip

Change History (3)

comment:1 Changed 14 years ago by xeen

Summary: WMS plugin does not check if user aborts save dialog → [PATCH] WMS plugin does not check if user aborts save dialog

Changed 14 years ago by xeen

Attachment: wmssave.patch​ added

comment:2 Changed 14 years ago by Gubaer

Resolution: → fixed
Status: new → closed

patch not necessary anymore. Fixed in o17397

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.

Add Comment

E-mail address and name can be saved in the Preferences.

Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.