Pass user to revision.publish() in save_target

[efrysf]

Summary

When save_target() publishes a translated page or draftable snippet, it defers the publish via transaction.on_commit(new_revision.publish). Because publish() is called without arguments, the resulting PageLogEntry has a null user_id, making it impossible to determine who published translated content from the audit log.

This passes the user parameter through to publish() on both the Page and DraftStateMixin code paths (models.py lines 804 and 815).

Context

We discovered this while diagnosing a production issue where bulk PO file imports were triggering DataError on auto-redirect creation. All 36 publish events in PageLogEntry had null user_id, which made it difficult to trace the activity back to the users who uploaded the PO files. The user was already being passed to save_revision() on the lines immediately above, but was not forwarded to the subsequent publish() call.

Test plan

• Existing test_save_target passes (calls save_target() without user, so user=None is forwarded — same behavior as before)
• Full tox suite (python3.14-django5.2-wagtail7.2-sqlite): 444 tests, 0 new failures (1 pre-existing failure in test_edit_nested_snippet_translation unrelated to this change)
• Verify PageLogEntry records the user after publishing a translated page via PO file import

🤖 Generated with Claude Code