#18725 closed enhancement (fixed)
Adjust output of SyncEditorLayerIndex to reduce workload when URLs change
Reported by: | Klumbumbus | Owned by: | team |
---|---|---|---|
Priority: | normal | Milestone: | 20.02 |
Component: | Unit tests | Version: | |
Keywords: | Cc: | stoecker, Don-vip |
Description
r15850 was a good idea, however it creates constant additional workload too (strip the error message at the correct position and add "...")
After thinking about it again I would prefer if we drop the URL completely (as already suggested by Dirk) as we have already the imagery name, which should be unique too.
If someone adapts the script, I care for changing ImageryCompareIgnores.
However we should keep the URL output for the following error message types:
- +++ ELI-URL is not unique:
- +++ JOSM-URL is not unique:
- +++ JOSM-Mirror-URL is not unique:
- * URL for id "+idj+" differs ("+urle+"):
- *** URLs found in ELI but not in JOSM
- *** URLs found in JOSM but not in ELI
For the following types the URL should could be replaced by the name
- +++ ELI-URL uses {z} instead of {zoom}:
- +++ JOSM-URL uses {z} instead of {zoom}:
Attachments (0)
Change History (6)
comment:1 by , 5 years ago
comment:2 by , 5 years ago
Changed it a bit, mainly replacing URL by ID. Check if this is ok. That was the best solution which included mainly automatic ignore file conversion.
follow-up: 4 comment:3 by , 5 years ago
* Name differs
and * Type differs
should use the id instead of the url I think. The rest seems fine.
comment:4 by , 5 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Replying to Klumbumbus:
* Name differs
and* Type differs
should use the id instead of the url I think. The rest seems fine.
No. That's not good. These messages are based on and thus tied to the URL. I'll leave it as is for now and we'll see what future brings :-)
comment:6 by , 5 years ago
Milestone: | → 20.02 |
---|
In 15878/josm: