Modify

Opened 15 years ago

Closed 14 years ago

#5270 closed defect (fixed)

OSB

Reported by: RafalR Owned by: hampelratte
Priority: normal Milestone:
Component: Plugin openstreetbugs Version:
Keywords: Cc:

Description

doubleclick on the bugs list (nothing has been loaded)

Repository Root: http://josm.openstreetmap.de/svn
Build-Date: 2010-07-24 01:32:01
Last Changed Author: jttt
Revision: 3382
Repository UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
URL: http://josm.openstreetmap.de/svn/trunk
Last Changed Date: 2010-07-23 21:27:18 +0200 (Fri, 23 Jul 2010)
Last Changed Rev: 3382

Identification: JOSM/1.5 (3382 pl)
Memory Usage: 989 MB / 989 MB (936 MB allocated, but free)
Java version: 1.6.0_20, Sun Microsystems Inc., Java HotSpot(TM) Client VM
Operating system: Windows Vista
Dataset consistency test: No problems found

Plugin: DirectUpload (22017)
Plugin: PicLayer (21706)
Plugin: RoadSigns (22423)
Plugin: openstreetbugs (21706)
Plugin: remotecontrol (21706)
Plugin: turnrestrictions (21706)
Plugin: validator (22204)
Plugin: wmsplugin (22286)

java.lang.NullPointerException
	at org.openstreetmap.josm.plugins.osb.gui.OsbDialog.zoomToNode(OsbDialog.java:305)
	at org.openstreetmap.josm.plugins.osb.gui.OsbDialog.mouseClicked(OsbDialog.java:310)
	at java.awt.AWTEventMulticaster.mouseClicked(Unknown Source)
	at java.awt.Component.processMouseEvent(Unknown Source)
	at javax.swing.JComponent.processMouseEvent(Unknown Source)
	at java.awt.Component.processEvent(Unknown Source)
	at java.awt.Container.processEvent(Unknown Source)
	at java.awt.Component.dispatchEventImpl(Unknown Source)
	at java.awt.Container.dispatchEventImpl(Unknown Source)
	at java.awt.Component.dispatchEvent(Unknown Source)
	at java.awt.LightweightDispatcher.retargetMouseEvent(Unknown Source)
	at java.awt.LightweightDispatcher.processMouseEvent(Unknown Source)
	at java.awt.LightweightDispatcher.dispatchEvent(Unknown Source)
	at java.awt.Container.dispatchEventImpl(Unknown Source)
	at java.awt.Window.dispatchEventImpl(Unknown Source)
	at java.awt.Component.dispatchEvent(Unknown Source)
	at java.awt.EventQueue.dispatchEvent(Unknown Source)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
	at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
	at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
	at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
	at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
	at java.awt.EventDispatchThread.run(Unknown Source)

Attachments (0)

Change History (11)

comment:1 by hampelratte, 15 years ago

Owner: changed from team to anonymous
Status: newassigned

comment:2 by hampelratte, 15 years ago

Owner: changed from anonymous to hampelratte
Status: assignednew

comment:3 by hampelratte, 15 years ago

I can reproduce the error and I have fixed it already. The SVN server does not respond, though. So it might take some time till the changes apply.

comment:4 by hampelratte, 15 years ago

Status: code checked in, plugin updated. New version should be available soon. Please close this ticket, if it works for you, too.

in reply to:  4 ; comment:5 by RafalR, 15 years ago

OsbDialog.java

308	    public void mouseClicked(MouseEvent e) {
309	        if (e.getButton() == MouseEvent.BUTTON1 && e.getClickCount() == 2) {
310	            Node selectedNode = getSelectedNode();
311	            if(selectedNode != null) { 
312	                zoomToNode(selectedNode);
313	            }
314	        }
315	    }

this still dont work correctly (22434)

in reply to:  5 comment:6 by hampelratte, 15 years ago

Replying to RafalR <rrudzik@…>:

this still dont work correctly (22434)

You could please describe what does not work correctly? Any exception? Wrong behaviour?

comment:7 by RafalR, 15 years ago

function zoomToNode is called even if nothing is selected (before the data is loaded from OSB)

comment:8 by hampelratte, 15 years ago

After spending some time playing around with different settings, I cannot trigger a method call of zoomToNode without having a node selected in the list. Furthermore zoomToNode is only called from mouseClicked, where I added the check for null. So in my opinion, it should be ok.

Could you please describe the steps you take, to produce the error? Is osb initially online / offline? Is data loaded before activating osb etc.

comment:9 by RafalR, 15 years ago

OK, dubleclick problem fixed, now is problem with right mouse click.

Repository Root: http://josm.openstreetmap.de/svn
Build-Date: 2010-07-17 01:31:31
Last Changed Author: bastiK
Revision: 3376
Repository UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
URL: http://josm.openstreetmap.de/svn/trunk
Last Changed Date: 2010-07-16 23:44:00 +0200 (Fri, 16 Jul 2010)
Last Changed Rev: 3376

Identification: JOSM/1.5 (3376 pl)
Memory Usage: 46 MB / 494 MB (25 MB allocated, but free)
Java version: 1.6.0_20, Sun Microsystems Inc., Java HotSpot(TM) Client VM
Operating system: Windows Vista
Dataset consistency test: No problems found

Plugin: openstreetbugs (22434)
Plugin: remotecontrol (21706)
Plugin: validator (22445)
Plugin: wmsplugin (22286)

java.lang.NullPointerException
	at org.openstreetmap.josm.plugins.osb.gui.action.PopupFactory.createPopup(PopupFactory.java:45)
	at org.openstreetmap.josm.plugins.osb.gui.OsbDialog.mayTriggerPopup(OsbDialog.java:329)
	at org.openstreetmap.josm.plugins.osb.gui.OsbDialog.mouseReleased(OsbDialog.java:322)
	at java.awt.AWTEventMulticaster.mouseReleased(Unknown Source)
	at java.awt.Component.processMouseEvent(Unknown Source)
	at javax.swing.JComponent.processMouseEvent(Unknown Source)
	at java.awt.Component.processEvent(Unknown Source)
	at java.awt.Container.processEvent(Unknown Source)
	at java.awt.Component.dispatchEventImpl(Unknown Source)
	at java.awt.Container.dispatchEventImpl(Unknown Source)
	at java.awt.Component.dispatchEvent(Unknown Source)
	at java.awt.LightweightDispatcher.retargetMouseEvent(Unknown Source)
	at java.awt.LightweightDispatcher.processMouseEvent(Unknown Source)
	at java.awt.LightweightDispatcher.dispatchEvent(Unknown Source)
	at java.awt.Container.dispatchEventImpl(Unknown Source)
	at java.awt.Window.dispatchEventImpl(Unknown Source)
	at java.awt.Component.dispatchEvent(Unknown Source)
	at java.awt.EventQueue.dispatchEvent(Unknown Source)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
	at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
	at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
	at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
	at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
	at java.awt.EventDispatchThread.run(Unknown Source)

comment:10 by hampelratte, 15 years ago

Ok, fixed that NPE, too. New Version should be available soon.

comment:11 by stoecker, 14 years ago

Resolution: fixed
Status: newclosed

Modify Ticket

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