You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Trying to merge multiple regional feeds together with another, already merged feed resulted in duplicate schedule_ids for calendar.txt entries.
Steps to reproduce:
Merge a feed with two schedules with IDs T0, a-T0 with another feed with another, different schedule with ID T0.
Expected behavior:
Resulting feed has calendar entries with unique IDs.
Observed behavior:
Resulting feed has calendar entries with duplicate IDs a-T0.
Root cause
This behavior is caused by MergeSupport.renameAgencyAndId which prepends a prefix without further checking for uniqueness which might result in duplicates.
As long as no feed contains IDs starting with single letters followed by a hyphen (e.g. "a-"), this will not cause an issue. However, the documentation should describe this duplicate resolution strategy.
The text was updated successfully, but these errors were encountered:
Summary:
Trying to merge multiple regional feeds together with another, already merged feed resulted in duplicate schedule_ids for calendar.txt entries.
Steps to reproduce:
Merge a feed with two schedules with IDs T0, a-T0 with another feed with another, different schedule with ID T0.
Expected behavior:
Resulting feed has calendar entries with unique IDs.
Observed behavior:
Resulting feed has calendar entries with duplicate IDs a-T0.
Root cause
This behavior is caused by MergeSupport.renameAgencyAndId which prepends a prefix without further checking for uniqueness which might result in duplicates.
As long as no feed contains IDs starting with single letters followed by a hyphen (e.g. "a-"), this will not cause an issue. However, the documentation should describe this duplicate resolution strategy.
The text was updated successfully, but these errors were encountered: