id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc
18123	[PATCH] Add Yandex as a imagery source in osm-obj-info	taylor.smock	Rub21	"Yandex has given permission for OSM use ( see https://wiki.openstreetmap.org/wiki/Contributors#Yandex.Panoramas ).


I've got a pull request here: https://github.com/JOSM/osm-obj-info/pull/9, and I've attached the patch from `git format-patch -1 HEAD`.

This is related to https://github.com/JOSM/osm-obj-info/issues/8 .

For the plugins that are on GitHub, should I file bugs and patches github or on trac?

The patch refactors the calls to open a browser to another function (which de-duplicates a significant portion of the code), and looking at it again, it can probably be modified further so that many of the functions no longer exist. For example:
{{{
#!java
lbLinkYandex.addMouseListener(new MouseAdapter() {
    @Override
    public void mouseClicked(MouseEvent e) {
        OSMObjInfoActions.openImageInBrowser(""https://yandex.com/maps/?l=stv,sta&ll={longitude},{latitude}&z=18"", lbMapillary.getText());
    }
});
}}}
with `OSMObjInfoActions.openImageInBrowser` being
{{{
#!java
public static void openImageInBrowser(String imageUrl, String coords) {
    if (coords == null || coords.isEmpty()) return;
    String[] arrCoords = coords.split("","");
    openInBrowser(imageUrl.replace(""{latitude}"", arrCoords[0]).replace(""{longitude}"", arrCoords[1]));
}
}}}

instead of

{{{
#!java
lbLinkYandex.addMouseListener(new MouseAdapter() {
    @Override
    public void mouseClicked(MouseEvent e) {
        OSMObjInfoActions.openInBrowserYandex(lbMapillary.getText());
    }
});
}}}"	enhancement	closed	normal		Plugin osm-obj-info		fixed	yandex