Modify

Opened 6 weeks ago

Closed 6 weeks ago

Last modified 6 weeks ago

#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 Changed 6 weeks ago by stoecker

In 15878/josm:

see #18725 - adjust sync script output

comment:2 Changed 6 weeks ago by stoecker

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.

comment:3 Changed 6 weeks ago by Klumbumbus

* Name differs and * Type differs should use the id instead of the url I think. The rest seems fine.

comment:4 in reply to:  3 Changed 6 weeks ago by stoecker

Resolution: fixed
Status: newclosed

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:5 Changed 6 weeks ago by Klumbumbus

OK.

comment:6 Changed 6 weeks ago by Don-vip

Milestone: 20.02

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.