reorganization of data(_nodist), images(_nodist), styles(_nodist), IDE and native files in a more practical file tree

[simon04]

Replying to Don-vip:

In 16006/josm:
see #18140 - reorganization of data(_nodist), images(_nodist), styles(_nodist), IDE and native files in a more practical file tree.

• Everything belonging to the jar is now in resources (data, images, styles)
• Everything not belonging to the jar is now in nodist (data, images, styles)
• Everything related to OS native functions is now in native (linux, macosx, windows)
• Everything related to an IDE is now in ide (eclipse, netbeans)

[simon04]

Awesome! This layout is imposed by Maven or ​Gradle and makes setting up an IDE much easier. Thank you!

[simon04]

Since we have to update the i18n process (i18n.install.dir in i18n/build.xml) anyways, should we move the .lang files to a separate directory, such as resources/data/i18n?

[simon04]

skyper in 18140#comment:36:

So all icon pathes in the wiki need to be adjusted.

stoecker in 18140#comment:37:

We probably can do this semi-automatic.

Also, the Trac logo URL needs to be updated.

[stoecker]

Replying to simon04:

Since we have to update the i18n process (i18n.install.dir in i18n/build.xml) anyways, should we move the .lang files to a separate directory, such as resources/data/i18n?

That would be bad as it would affects all plugins.

[stoecker]

Replying to simon04:

skyper in 18140#comment:36:

So all icon pathes in the wiki need to be adjusted.

stoecker in 18140#comment:37:

We probably can do this semi-automatic.

Question is: Do we want to take that chance to replace the image macro with an JOSM specific one, so that it auto-detects PNG/SVG and references directly to the JOSM image dir?

Also, the Trac logo URL needs to be updated.

Done.

[Hb---]

Main logo on front page is currently 404.

[stoecker]

Main logo on front page is currently 404.

Fixed.

@Vincent: I hope you know what a can of worms you opened with the reorganisation?

[stoecker]

@leni: Please use tickets and not the SPAM report feature to report issues with the wiki.

[leni]

@stoecker someone told me to use SPAM report for image... when to use it? only for attachments?

[stoecker]

Replying to leni:

@stoecker someone told me to use SPAM report for image... when to use it? only for attachments?

SPAM report should be used when the result is an admin deleting the "offending" stuff (i.e. you may also use it e.g. for accidentially created pages or obsolete images to get rid off or alike). Deleting is the only possible action. Everything else which may need feedback is better done via ticket.

[Don-vip]

@Dirk yes I know. This is something I wanted to do for a long time and never did because of fear of the worms. I have found enough courage to do it now Did I broke the i18n or not? I forgot about it yesterday.

[GerdP]

The command

ant clean dist 

no longer works with r16010 or newer, probably one of the recent changes made for this ticket.

epsg-compile:
    [mkdir] Created dir: C:\josm\core\build2
    [javac] Compiling 1 source file to C:\josm\core\build2

epsg:
    [touch] Creating C:\josm\core\resources\data\projection\custom-epsg
     [java] Exception in thread "main" java.lang.ExceptionInInitializerError
     [java]     at BuildProjectionDefinitions.initMap(BuildProjectionDefinitions.java:91)
     [java]     at BuildProjectionDefinitions.buildList(BuildProjectionDefinitions.java:109)
     [java]     at BuildProjectionDefinitions.main(BuildProjectionDefinitions.java:77)
     [java] Caused by: org.openstreetmap.josm.tools.JosmRuntimeException: java.io.IOException: Failed to open input stream for resource 'resource://data/projection/custom-epsg'
     [java]     at org.openstreetmap.josm.data.projection.Projections.<clinit>(Projections.java:175)
     [java]     ... 3 more
     [java] Caused by: java.io.IOException: Failed to open input stream for resource 'resource://data/projection/custom-epsg'
     [java]     at org.openstreetmap.josm.io.CachedFile.lambda$getInputStream$0(CachedFile.java:231)
     [java]     at java.util.Optional.orElseThrow(Optional.java:290)
     [java]     at org.openstreetmap.josm.io.CachedFile.getInputStream(CachedFile.java:231)
     [java]     at org.openstreetmap.josm.io.CachedFile.getContentReader(CachedFile.java:258)
     [java]     at org.openstreetmap.josm.data.projection.Projections.loadProjectionDefinitions(Projections.java:301)
     [java]     at org.openstreetmap.josm.data.projection.Projections.<clinit>(Projections.java:170)
     [java]     ... 3 more

BUILD FAILED
C:\josm\core\build.xml:1046: Java returned: 1

[stoecker]

Question is: Do we want to take that chance to replace the image macro with an JOSM specific one, so that it auto-detects PNG/SVG and references directly to the JOSM image dir?

comments?

[skyper]

Replying to stoecker:

Question is: Do we want to take that chance to replace the image macro with an JOSM specific one, so that it auto-detects PNG/SVG and references directly to the JOSM image dir?

+1

• there is no image dir but some within trunk, some in the wiki like (external) presets and many elsewhere, including (external) presets and plugins.
• personally the name of the icon is my major problem when searching, the path is only minor priority.

[Klumbumbus]

Replying to stoecker:

Question is: Do we want to take that chance to replace the image macro with an JOSM specific one, so that it auto-detects PNG/SVG and references directly to the JOSM image dir?

Yes, that would definitely reduce work load after changes of #15240.
Would the internal help browser manage it well? Would it create lots of more work if we leave trac one day?

[stoecker]

Replying to Klumbumbus:

Replying to stoecker:

Question is: Do we want to take that chance to replace the image macro with an JOSM specific one, so that it auto-detects PNG/SVG and references directly to the JOSM image dir?

Yes, that would definitely reduce work load after changes of #15240.

I'll have a look then.

Would the internal help browser manage it well?

Yes. It does not know of the wiki language, it reads HTML.

Would it create lots of more work if we leave trac one day?

Well, that would be so much work, that I don't consider it at all. Staying on an old trac version is the much easier choice.

[stoecker]

Replying to skyper:

Replying to stoecker:

Question is: Do we want to take that chance to replace the image macro with an JOSM specific one, so that it auto-detects PNG/SVG and references directly to the JOSM image dir?

+1

• there is no image dir but some within trunk, some in the wiki like (external) presets and many elsewhere, including (external) presets and plugins.

This is only for JOSM images, i.e. these which are accessed with "source:...". The main idea is to make them behave like JOSM does with auto-detection of image extension. Everything else will be unchanged.

[Klumbumbus]

Replying to stoecker:

Would it create lots of more work if we leave trac one day?

Well, that would be so much work, that I don't consider it at all. Staying on an old trac version is the much easier choice.

OK (I thought if we switch from svn to git we would leave trac entirely.)

[Klumbumbus]

stoecker in 18140#comment:37:

We probably can do this semi-automatic.

Rather are forced to as updating the links is not doable manually https://josm.openstreetmap.de/search?q=Image%28source%3Atrunk%2Fimages
1k pages, so probably 10k links :)

[simon04]

Replying to stoecker:

Replying to simon04:

Since we have to update the i18n process (i18n.install.dir in i18n/build.xml) anyways, should we move the .lang files to a separate directory, such as resources/data/i18n?

That would be bad as it would affects all plugins.

The Ant task plugintrans runs i18n.pl [...] --basedir=${plugin.dir}/${dir}/data/, thus, plugins are unaffected from changing ${i18n.install.dir} which is only used in the task coretrans.

[stoecker]

Replying to Don-vip:

Did I broke the i18n or not?

What do you think?
[o35345:35346]

I forgot about it yesterday.

And many many more... :-)

[stoecker]

Replying to simon04:

Replying to stoecker:

Replying to simon04:

Since we have to update the i18n process (i18n.install.dir in i18n/build.xml) anyways, should we move the .lang files to a separate directory, such as resources/data/i18n?

That would be bad as it would affects all plugins.

The Ant task plugintrans runs i18n.pl [...] --basedir=${plugin.dir}/${dir}/data/, thus, plugins are unaffected from changing ${i18n.install.dir} which is only used in the task coretrans.

Please stop thinking only from source side. If we change the directory where translations are stored then we also need to do this for the plugins, as core and plugins use the same mechanisms. And having different places for source and final jar would actually be opposite to what is the goal of this directory change so that's also no option.

[simon04]

In 16012/josm:

see #18845 - Move lang to resources/data/trans/

[simon04]

In 16013/josm:

see #18845 - I18n update

[simon04]

Replying to stoecker:

Please stop thinking only from source side.

Ups, I was just preparing my commits and overlooked your reply…

If we change the directory where translations are stored then we also need to do this for the plugins, as core and plugins use the same mechanisms.

Actually not, see https://josm.openstreetmap.de/changeset/16012/josm/trunk/src/org/openstreetmap/josm/tools/I18n.java

If the move to resources/data/trans/ is not an option, I'm going to revert my changes.

For i18n: [o35347], [o35348].

[Klumbumbus]

Replying to Klumbumbus:

stoecker in 18140#comment:37:

We probably can do this semi-automatic.

Rather are forced to as updating the links is not doable manually https://josm.openstreetmap.de/search?q=Image%28source%3Atrunk%2Fimages
1k pages, so probably 10k links :)

Another thing to consider: even if all image paths were updated semi automatic then the revision of all translated pages will be outdated. In best case the revision of the translated page would be increased by one during the image path update in case the translated page was up to date before. And do not increase if the page was outdated already before. Not sure if this is possible.

[stoecker]

In 16014/josm:

revert translation file move, see #18845

[stoecker]

In 16015/josm:

move nodist/data/trans one level up, see #18845

[Klumbumbus]

JOSM imagery pages can't be edited.
Warnung: Ungültige Wiki-Seite: XML validation for map failed: [Errno 2] No such file or directory: '/home/josm/auto/svn_josm/core/data/maps.xsd'

[stoecker]

Replying to Klumbumbus:

JOSM imagery pages can't be edited.
Warnung: Ungültige Wiki-Seite: XML validation for map failed: [Errno 2] No such file or directory: '/home/josm/auto/svn_josm/core/data/maps.xsd'

Yes. Found it a few seconds ago. Fixed?

[stoecker]

I think the important parts on the server side should be fixed now. We'll see tomorrow when latest is built.

[stoecker]

[o35349]

[GerdP]

Replying to GerdP:

The command

ant clean dist 

no longer works with r16010 or newer, probably one of the recent changes made for this ticket.

epsg-compile:
    [mkdir] Created dir: C:\josm\core\build2
    [javac] Compiling 1 source file to C:\josm\core\build2

epsg:
    [touch] Creating C:\josm\core\resources\data\projection\custom-epsg
     [java] Exception in thread "main" java.lang.ExceptionInInitializerError
     [java]     at BuildProjectionDefinitions.initMap(BuildProjectionDefinitions.java:91)
     [java]     at BuildProjectionDefinitions.buildList(BuildProjectionDefinitions.java:109)
     [java]     at BuildProjectionDefinitions.main(BuildProjectionDefinitions.java:77)
     [java] Caused by: org.openstreetmap.josm.tools.JosmRuntimeException: java.io.IOException: Failed to open input stream for resource 'resource://data/projection/custom-epsg'
     [java]     at org.openstreetmap.josm.data.projection.Projections.<clinit>(Projections.java:175)
     [java]     ... 3 more
     [java] Caused by: java.io.IOException: Failed to open input stream for resource 'resource://data/projection/custom-epsg'
     [java]     at org.openstreetmap.josm.io.CachedFile.lambda$getInputStream$0(CachedFile.java:231)
     [java]     at java.util.Optional.orElseThrow(Optional.java:290)
     [java]     at org.openstreetmap.josm.io.CachedFile.getInputStream(CachedFile.java:231)
     [java]     at org.openstreetmap.josm.io.CachedFile.getContentReader(CachedFile.java:258)
     [java]     at org.openstreetmap.josm.data.projection.Projections.loadProjectionDefinitions(Projections.java:301)
     [java]     at org.openstreetmap.josm.data.projection.Projections.<clinit>(Projections.java:170)
     [java]     ... 3 more

BUILD FAILED
C:\josm\core\build.xml:1046: Java returned: 1

Probably caused by me: I've manually removed \data\projection\custom-epsg
With a clean checkout you should see the same problem.

[simon04]

Replying to simon04:

Replying to stoecker:

Please stop thinking only from source side.

Ups, I was just preparing my commits and overlooked your reply…

@stoecker, I didn't mean to overrule you. After investigating comment:21, I was preparing my commits and didn't check for new comments on this issue. Only after committing, I've read your reply.

[stoecker]

@Vincent: Most of the big worm are back inside, happy hunting for the smaller ones ;-)

[Klumbumbus]

Replying to stoecker:

Replying to Klumbumbus:

JOSM imagery pages can't be edited.
Warnung: Ungültige Wiki-Seite: XML validation for map failed: [Errno 2] No such file or directory: '/home/josm/auto/svn_josm/core/data/maps.xsd'

Yes. Found it a few seconds ago. Fixed?

Yes.

[simon04]

In 16017/josm:

revert translation file move, see #18845 (update CORE_TRANS_DIRECTORY)

[Don-vip]

In 16018/josm:

see #18845 - fix epsg and taginfo goals

[Don-vip]

Replying to stoecker:

Replying to Don-vip:

Did I broke the i18n or not?

What do you think?

By "broken" I was afraid to have uploaded to Launchpad incomplete translations, thus deleting thousands of translated strings. I'm relieved it didn't happen this way.

And many many more... :-)

Thanks for the help! :)

[Don-vip]

Replying to stoecker:

@Vincent: Most of the big worm are back inside, happy hunting for the smaller ones ;-)

Thanks! I'm on it :)

[simon04]

There are two failing tests, https://josm.openstreetmap.de/jenkins/job/JOSM/lastCompletedBuild/testReport/:

• org.openstreetmap.josm.data.VersionTest.testGetAgentString
• org.openstreetmap.josm.data.validation.tests.MapCSSTagCheckerTest.testPreprocessing

[Don-vip]

I'm on it

[Don-vip]

In 16019/josm:

see #18845 - fix unit test

[Don-vip]

Replying to simon04:

• org.openstreetmap.josm.data.VersionTest.testGetAgentString

Fixed, my fault.

• org.openstreetmap.josm.data.validation.tests.MapCSSTagCheckerTest.testPreprocessing

This one I don't understand. The test is OK for me. Do you have an idea?

[GerdP]

Replying to Don-vip:

In 16018/josm:

see #18845 - fix epsg and taginfo goals

Thanks,

ant clean dist

works again on Windows system.

[GerdP]

Replying to Don-vip:

Replying to simon04:

• org.openstreetmap.josm.data.VersionTest.testGetAgentString

Fixed, my fault.

• org.openstreetmap.josm.data.validation.tests.MapCSSTagCheckerTest.testPreprocessing

This one I don't understand. The test is OK for me. Do you have an idea?

IIGTR target test is missing a dependency on create-revision.
I can reproduce the problem with a clean checkout and

ant compile-test
ant -Ddefault-junit-includes="**/**/MapCSSTag*Test.class" test

After this the needed file REVISION is missing in the build directory.

[GerdP]

In 16028/josm:

see #18845: fix target compile-test dependencies

[Hb---]

Replying to

stoecker: ... replace the image macro with an JOSM specific on ?

Replying to Klumbumbus:

1k pages, so probably 10k links :)

if all image paths were updated semi automatic, then the revision of all translated pages will be outdated. ...

A semiautomated solution for the wiki image source links seems fine. The active translators can handle this. For the non active it is unimportant if a page is 20 or 21 versions behind.

Please inform how the decision is. If it is still open, please give notice too.

[stoecker]

My plan currently is:

• Get the new link-variant working for tests (today)
• extract the what-to-change list from the SQL directly
• update my "automatically overflood trac with hundred requests script" to change the pages accordingly (As requested updating the translated pages revsion by one when applicable)
• start the script
• present all the broken links found during this process for manual fixing.

[stoecker]

Please try:

[[JOSMImage(openlocation)]] [[JOSMImage(audio-fwd)]]

See docs at WikiMacros#JOSMImage-macro

Please test.

[Don-vip]

Replying to GerdP:

In 16028/josm:

see #18845: fix target compile-test dependencies

I removed the test-compile dependency on "dist" especially to avoid the call to create-revision, as it causes problems with the server Makefile. I will try to see why this test needs the REVISION file, it's strange.

[GerdP]

OK. See also #18858.

[Don-vip]

In 16033/josm:

see #18845 - make MapCSSTagCheckerTest work without REVISION file

[Don-vip]

In 16036/josm:

see #18845, fix #18858 - fix ant build

[Klumbumbus]

Replying to stoecker:

Please test.

inline seems to not work:

heading text

heading text

Can't really test @rev as all images were moved, however I think there is not a single usage of that code in the wiki anyway.

[stoecker]

Replying to Klumbumbus:

Replying to stoecker:

Please test.

inline seems to not work:

heading text

heading text

Really :-)

Can't really test @rev as all images were moved, however I think there is not a single usage of that code in the wiki anyway.

That makes no sense anyway. If you have a specific revision you can use the normal macro, as this revision will never change.

[Klumbumbus]

Great. From my POV the JOSMImage macro is fine.

[Hb---]

Replying to stoecker:

The png is fine, but the svg image audio-fwd is not shown in JOSM Help browser, neither as block nor inline. But this may be related to the help system and not to the new macro.

[Klumbumbus]

Replying to Hb---:

Replying to stoecker:

The png is fine, but the svg image audio-fwd is not shown in JOSM Help browser,

These and the ones in the headings are displayed fine for me in the josm internal help browser.

[skyper]

Replying to stoecker:

...
Please test.

Mmh, should [[JOSMImage(relation)]] work () or only [[JOSMImage(data/relation)]] () ?

[stoecker]

Replying to skyper:

Mmh, should [[JOSMImage(relation)]] work () or only [[JOSMImage(data/relation)]] () ?

See docs: WikiMacros#JOSMImage-macro: The first parameter is a path inside trunk/resources/images/ without the extension. So the first can't work, as there is no relation.xxx inside the path.

[Hb---]

Replying to Klumbumbus:
All image types are fine after updating to 1.8.0_242-b08, AdoptOpenJDK, OpenJDK 64-Bit Server VM.

Replying to stoecker:

The first parameter is a path inside trunk/resources/images/ without the extension.

I chocked on that sentence too. Suggestion:

The first parameter is a file path starting at trunk/resources/images/ without the extension.

[stoecker]

Replying to Hb---:

The first parameter is a file path starting at trunk/resources/images/ without the extension.

Well, saying "file path" is same for me as saying "female woman". That's implied. I changed the inside: The first parameter is a path in the directory

[Klumbumbus]

We should probably not change image links like in https://josm.openstreetmap.de/wiki/Help/Menu/Help?action=diff&version=38 and https://josm.openstreetmap.de/wiki/De%3AHelp/Menu/Help?action=diff&version=23 as this might complicate the planned semiautomatic update. The goal is anyway not [[Image(source:/trunk/resources/images/dialogs/search.png)]] but [[JOSMImage(dialogs/search)]]

Dirk?

[stoecker]

Replying to Klumbumbus:

We should probably not change image links like in https://josm.openstreetmap.de/wiki/Help/Menu/Help?action=diff&version=38 and https://josm.openstreetmap.de/wiki/De%3AHelp/Menu/Help?action=diff&version=23 as this might complicate the planned semiautomatic update.

I have no issue with that. Manual fixes simply mean that the script has to do less ;-)

The goal is anyway not [[Image(source:/trunk/resources/images/dialogs/search.png)]]

These case will also be converted to JOSMImage macro when found.

but [[JOSMImage(dialogs/search)]]

Dirk?

I hope I find time this weekend for this task.

[Klumbumbus]

I get Warnung: Ungültige Wiki-Seite: XML validation for style failed: [Errno 2] No such file or directory: '/home/josm/auto/svn_josm/core/resources/data/mappaint-style.xsd'
when trying to edit wiki:/Styles/LegacyStandard

[skyper]

[[BadReport]] needs to be updated.

[Don-vip]

In 16050/josm:

see #18845 - see #16860 - directory cleanup

[Don-vip]

In 16051/josm:

see #18845 - integration tests need REVISION file

[stoecker]

Replying to Klumbumbus:

I get Warnung: Ungültige Wiki-Seite: XML validation for style failed: [Errno 2] No such file or directory: '/home/josm/auto/svn_josm/core/resources/data/mappaint-style.xsd'
when trying to edit wiki:/Styles/LegacyStandard

Oh, that bug must be there for a long time. Fixed it.

[Don-vip]

In 16064/josm:

see #18845 - see #18880 - update Netbeans files

[leni]

there are two different images for the validator: in dialogs directory and in preferences directory

I modified https://josm.openstreetmap.de/wiki/Rules by changing the macro for the Validator image; I replaced the "image" macro by the "JOSMImage" macro without modifying either the directory or the original parameters:

at the top of the page from dialogs directory with parameter (dialogs/validator,middle,margin-right=20)
and after the table from preferences directory with parameter (preferences/validator,15)

but the display is different between wiki and JOSM Help browser :
in the wiki page the result is as above; when in JOSM Help browser first image is smaller than the second one.

I use W10 and Firefox

[Hb---]

Replying to leni:
On the Rules page the first occurrence of the image had no width attribute.
But it did have a style attribute (middle) which the second has not. This may be the cause. Additionally the image resizing in the Help browser has at least one bug #15864.

Replying to stoecker:

... saying "file path" is same for me as saying "female woman". That's implied.

Well, that Redmond OS had blown this fact away from me.

[leni]

differents Rules Validator - wiki and JOSM Help browser

[leni]

Replying to Hb---:

Replying to leni:
On the Rules page the first occurrence of the image had no width attribute.
But it did have a style attribute (middle) which the second has not. This may be the cause.

The change you made in the rules page didn't change anything, it wasn't meant to be that.

Additionally the image resizing in the Help browser has at least one bug #15864.

Sorry, it must be a duplicate with #15864 (I joined image)

[skyper]

@ Dirk:

Sorry, totally forgot about it but it would be nice if link= could be automatically set and be overridden with a value if needed. Also middle would be nice as default. Right now, you have to added it to almost every image.

[stoecker]

All Updates

Report any errors (I hope none :-)

[stoecker]

Replying to skyper:

@ Dirk:

Sorry, totally forgot about it but it would be nice if link= could be automatically set and be overridden with a value if needed. Also middle would be nice as default. Right now, you have to added it to almost every image.

I'm no magician. I can do straightforward changes with such a script, but no complicated argument handling. Now it's to late anyway.

[stoecker]

Either my script had an error in the detection or there are really no dead image links anywhere (any png/svg issue was fixed automatically).

[Klumbumbus]

Replying to stoecker:

All Updates

Hah, even that 1430 points cheat code didn't bring you the first place at https://josm.openstreetmap.de/stats/wiki#EditsByAuthor ;)

[stoecker]

Replying to stoecker:

Replying to skyper:

@ Dirk:

Sorry, totally forgot about it but it would be nice if link= could be automatically set and be overridden with a value if needed. Also middle would be nice as default. Right now, you have to added it to almost every image.

I'm no magician. I can do straightforward changes with such a script, but no complicated argument handling. Now it's to late anyway.

I can add "middle,link=" as default argument to JOSMImage() when no arguments are given. Would that be fine? Klumbumbus?

[stoecker]

Replying to Klumbumbus:

Replying to stoecker:

All Updates

Hah, even that 1430 points cheat code didn't bring you the first place at https://josm.openstreetmap.de/stats/wiki#EditsByAuthor ;)

Well, in the second column I am :-)

[Klumbumbus]

Replying to stoecker:

I can add "middle,link=" as default argument to JOSMImage() when no arguments are given. Would that be fine? Klumbumbus?

I think so.

[skyper]

Replying to Klumbumbus:

Replying to stoecker:

I can add "middle,link=" as default argument to JOSMImage() when no arguments are given. Would that be fine? Klumbumbus?

I think so.

That is what I had in mind.

[skyper]

Replying to stoecker:

Replying to Klumbumbus:

Replying to stoecker:

All Updates

Hah, even that 1430 points cheat code didn't bring you the first place at https://josm.openstreetmap.de/stats/wiki#EditsByAuthor ;)

Well, in the second column I am :-)

Would you mind, to give me the user right to view the stats, again. Thanks

[stoecker]

Replying to skyper:

Replying to Klumbumbus:

Replying to stoecker:

I can add "middle,link=" as default argument to JOSMImage() when no arguments are given. Would that be fine? Klumbumbus?

I think so.

That is what I had in mind.

Done.

[Don-vip]

In 16126/josm:

see #18845 - fix build.xml for script targets, jar file is not required

[Don-vip]

Is everything fixed, or do we still need to do something?

[simon04]

In 16140/josm:

see #18140 - build.xml: use ${resources.dir}

[leni]

thank you for this work and stay safe.

[skyper]

@Dirk:
The defaults for [[JOSMImage()]] are lost, if you add another option.

[[JOSMImage(dialogs/delete,64)]] should be the same as [[JOSMImage(dialogs/delete,64,middle,link=)]] but it is not.

[stoecker]

Replying to skyper:

@Dirk:
The defaults for [[JOSMImage()]] are lost, if you add another option.

As documented, read WikiMacros#JOSMImage-macro.

[[JOSMImage(dialogs/delete,64)]] should be the same as [[JOSMImage(dialogs/delete,64,middle,link=)]] but it is not.

No, as it would be impossible to get rid of the two options if needed.

[skyper]

Replying to stoecker:

No, as it would be impossible to get rid of the two options if needed.

I see, thanks. Might need to set a default size once we have more bigger .svg icons.

[Klumbumbus]

Replying to stoecker:

Either my script had an error in the detection or there are really no dead image links anywhere

I just noticed a broken image link on wiki:/Help/Action/About and its translated pages. (No big deal to fix this by hand though. Just in case you want to investigate.)

[skyper]

Replying to Klumbumbus:

Replying to stoecker:

Either my script had an error in the detection or there are really no dead image links anywhere

I just noticed a broken image link on wiki:/Help/Action/About and its translated pages. (No big deal to fix this by hand though. Just in case you want to investigate.)

This is one rare special case. In general, everything went smooth. Found more broken links to source than images. Thanks

[stoecker]

Replying to skyper:

Replying to Klumbumbus:

Replying to stoecker:

Either my script had an error in the detection or there are really no dead image links anywhere

I just noticed a broken image link on wiki:/Help/Action/About and its translated pages. (No big deal to fix this by hand though. Just in case you want to investigate.)

This is one rare special case. In general, everything went smooth. Found more broken links to source than images. Thanks

Probably because of the "josm" at the beginning of the link. I'll have to rerun the script to fix these cases as well. I'll try tomorrow, don't have the necessary software here ATM.

[skyper]

If rerun, please:

• remove default values from [[JOSMImage()]] if no other option is present. That are most changes regarding images, atm.
• additionally, look for ^source: in links and adjust them.

[stoecker]

Replying to skyper:

If rerun, please:

• remove default values from [[JOSMImage()]] if no other option is present. That are most changes regarding images, atm.

I'll only change links I forgot, not the ones I already changed. But I'll keep this in mind.

• additionally, look for ^source: in links and adjust them.

I have no idea what that means.

[skyper]

Replying to stoecker:

Replying to skyper:

If rerun, please:

• remove default values from [[JOSMImage()]] if no other option is present. That are most changes regarding images, atm.

I'll only change links I forgot, not the ones I already changed. But I'll keep this in mind.

Ok.

• additionally, look for ^source: in links and adjust them.

I have no idea what that means.

See search and the first change in Styles, e.g. [source:trunk/styles/standard/elemstyles.mapcss JOSM standard] to [source:/trunk/resources/styles/standard/elemstyles.mapcss JOSM standard]

[stoecker]

Replying to Klumbumbus:

Replying to stoecker:

Either my script had an error in the detection or there are really no dead image links anywhere

I just noticed a broken image link on wiki:/Help/Action/About and its translated pages. (No big deal to fix this by hand though. Just in case you want to investigate.)

Fixed with 1 and 2

Search for source links results in following remaining errors:

• Error with src/org/openstreetmap/josm/data/SelectionChangedListener.java in DevelopersGuide/DevelopingPlugins
• Error with src/org/openstreetmap/josm/layer/OsmDataLayer.java in DevelopersGuide/DevelopingPlugins
• Error with src/org/openstreetmap/josm/plugin/Plugin.java in DevelopersGuide/DevelopingPlugins
• Error with data/validator/words.cfg in Help/Preferences/Validator
• Error with data/words.cfg in wiki:Ja:Help/Preferences/Validator
• Error with images_nodist/misc/gray_check.svg in wiki:Nl:Help/Plugin/Wikipedia
• Error with data/validator/words.cfg in wiki:Nl:Help/Preferences/Validator
• Error with src/org/openstreetmap/josm/resources/data/validation/tests in wiki:Nl:Help/Validator/MapCSSTagChecker
• Error with data/validator/words.cfg in wiki:Ru:Help/Preferences/Validator
• Error with data/tagging-preset.xsd in wiki:Ru:TaggingPresets
• Error with data/tagging-preset.xsd in wiki:Ru:TaggingPresets
• Error with data/words.cfg in wiki:Uk:Help/Preferences/Validator

They are not easily fixed, but need updates of the texts.

[simon04]

On Plugins, the yes/no icons are missing.

[GerdP]

I've fixed the words.cfg problems

[stoecker]

Replying to simon04:

On Plugins, the yes/no icons are missing.

Fixed.

[stoecker]

Replying to GerdP:

I've fixed the words.cfg problems

Hmm, I was SURE this file was gone. Probably mass-edits after a long work-day are a bit sub-optimal :-)

[GerdP]

It's still used in TagChecker

[Don-vip]

In 16239/josm:

see #18845 - adapt relative paths in Windows installer scripts