id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc
17201	fix common typo in progressMonitor beginTask()	GerdP	team	"Due to copy+paste we have quite a lot of sources with this error pattern:
Instead of 
{{{
#!java
  progressMonitor.beginTask(tr(""Contacting Server..."", 10));
}}}
it should be 
{{{
#!java
  progressMonitor.beginTask(tr(""Contacting Server...""), 10);
}}}
The effect is that many progress monitors show no progress because the default tick number is 10000.
I found this in the o5m plugin and in some core sources.

Other plugins matching the search pattern `""beginTask\(tr.*[0..9]+\)\)""` are: 
{{{
plugins\cadastre-fr\src\org\openstreetmap\josm\plugins\fr\cadastre\download\CadastreServerReader.java
plugins\poly\src\poly\DownloadPolyTask.java
plugins\imagery-xml-bounds\src\org\openstreetmap\josm\plugins\imageryxmlbounds\io\JosmServerLocationReader.java
plugins\pbf\src\org\openstreetmap\josm\plugins\pbf\io\PbfServerReader.java
}}}

There may be more where the tick count is not given as a numerical constant."	task	new	normal		Core			progressmonitor	gaben