Fix resource operations corrupting package-level multilang translations

[Copilot]

When a resource is created, updated, or deleted, CKAN internally calls package_update, triggering after_dataset_update with a pkg_dict sourced from the core package table — which holds the last-saved language's text, not the current session language's translation. The multilang extension blindly overwrites the current-language package_multilang entries with this stale data.

Changes

ckanext/multilang/plugin.py

• Added before_resource_create/update/delete hooks that set a '__multilang_skip_pkg_update' flag on the shared CKAN context dict before the internal package_update is called
• after_dataset_update now skips the package multilang update when this flag is present
• after_resource_create/update and new after_resource_delete clear the flag after the resource operation completes

def before_resource_update(self, context, current, resource):
    context[self._RESOURCE_OP_FLAG] = True

def after_dataset_update(self, context, obj_dict):
    if context.get(self._RESOURCE_OP_FLAG):
        log.debug('Skipping package multilang update: triggered by a resource operation')
        return
    # ... normal package multilang update

def after_resource_update(self, context, obj_dict):
    context.pop(self._RESOURCE_OP_FLAG, None)
    # ... normal resource multilang update

The flag is passed cleanly through the context (the same dict CKAN threads through before_resource_* → package_update → after_dataset_update → after_resource_*), requiring no thread-local storage.

ckanext/multilang/tests/test_resource_update_fix.py

• New test module with 13 unit tests covering flag set/clear behaviour for each hook and full lifecycle simulations for resource create, update, and delete flows

[Copilot]

Pull request overview

Fixes a multilang regression in the CKAN extension where resource CRUD operations could overwrite package-level translations with stale package-table values during CKAN’s internal package_update flow.

Changes:

• Added a resource-operation context flag so after_dataset_update skips package multilang writes during resource create/update/delete.
• Cleared that flag in the corresponding resource after-hooks.
• Added regression-oriented tests for flag lifecycle behavior and simulated resource hook flows.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.

File	Description
ckanext/multilang/plugin.py	Adds the context-flag guard around package multilang updates during resource operations.
ckanext/multilang/tests/test_resource_update_fix.py	Adds unit tests covering flag set/clear behavior and simulated create/update/delete sequences.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.