Skip to content

[17.0][OU-FIX] l10n_de_mis_*_report: Rename XML-IDs pre-merge#5314

Merged
pedrobaeza merged 1 commit intoOCA:17.0from
Tecnativa:17.0-ou-fix-l10n_de_mis_report-xml_id
Aug 15, 2025
Merged

[17.0][OU-FIX] l10n_de_mis_*_report: Rename XML-IDs pre-merge#5314
pedrobaeza merged 1 commit intoOCA:17.0from
Tecnativa:17.0-ou-fix-l10n_de_mis_report-xml_id

Conversation

@pedrobaeza
Copy link
Member

Both l10n_de_mis_skr03_report and l10n_de_mis_skr04_report have the same XML-IDs for their data records, so merging them into l10n_de_mis_report generates duplicated XML-IDs if both are installed, crashing the system.

But only if you have one of them installed, all the previous MIS definition records will be tried to be removed, giving the corresponding error if they have been used.

Let's fix it doing a previous renaming of all its XML-IDs.

@Tecnativa TT41859

Both l10n_de_mis_skr03_report and l10n_de_mis_skr04_report have the same
XML-IDs for their data records, so merging them into l10n_de_mis_report
generates duplicated XML-IDs if both are installed, crashing the system.

But only if you have one of them installed, all the previous MIS
definition records will be tried to be removed, giving the corresponding
error if they have been used.

Let's fix it doing a previous renaming of all its XML-IDs.

TT41859
@pedrobaeza pedrobaeza added this to the 17.0 milestone Aug 15, 2025
@pedrobaeza
Copy link
Member Author

Tested and working.

@pedrobaeza pedrobaeza merged commit 03bd8c9 into OCA:17.0 Aug 15, 2025
3 of 4 checks passed
@pedrobaeza pedrobaeza deleted the 17.0-ou-fix-l10n_de_mis_report-xml_id branch August 15, 2025 12:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant