search

mkyral / josm-tracer

A tracer plugin for josm
GNU General Public License v2.0
12 stars 6 forks source link

java.lang.NoSuchMethodError: org.openstreetmap.josm.gui.conflict.tags.CombinePrimitiveResolverDialog.getInstance() #38

Closed mkyral closed 7 years ago

mkyral commented 7 years ago 
java.lang.NoSuchMethodError: org.openstreetmap.josm.gui.conflict.tags.CombinePrimitiveResolverDialog.getInstance()Lorg/openstreetmap/josm/gui/conflict/tags/CombinePrimitiveResolverDialog;
    at org.openstreetmap.josm.plugins.tracer.CombineTagsResolver.launchIfNecessary(CombineTagsResolver.java:93)
    at org.openstreetmap.josm.plugins.tracer.modules.lpis.LpisModule$LpisTracerTask.tagTracedObject(LpisModule.java:369)
    at org.openstreetmap.josm.plugins.tracer.modules.lpis.LpisModule$LpisTracerTask.createTracedPolygonImpl(LpisModule.java:275)
    at org.openstreetmap.josm.plugins.tracer.TracerModule$AbstractTracerTask$6.run(TracerModule.java:348)

Ke změně došlo v commitu: https://github.com/openstreetmap/josm/commit/70baab40d3c2ba86bf1546126d267e8f1c0a0081

Funkce CombinePrimitiveResolverDialog.getInstance() byla odstraněna.

Komentář v kódu Traceru:

// Note 1: this method was copied from CombinePrimitiveResolverDialog.launchIfNecessary() and // adapted to resolution of tag sets only. It's not possible to use the original launchIfNecessary() // because (a) it requires primitives occurring in DataSet, (b) it tries to resolve relation // memberships, (c) it builds a list of DataSet Commands instead of the resolved set of tags.

// Note 2: this method relies on deprecated function CombinePrimitiveResolverDialog.getInstance(). // The function was deprecated by introduction of CombinePrimitiveResolverDialog.launchIfNecessary() // but it's still available for public use.

Maatts commented 7 years ago

Poslal jsem pull request, který řeší popsaný problém plus pár dalších deprecated funkcí a problémů, viz changelog. Po dvou letech nečinnosti jsem dost vařil z vody, takže prosím vyzkoušet před finálním releasem :-)

mkyral commented 7 years ago

Jasně. Otestuji a pustím. Ale asi až zítra. Díky moc.