Page MenuHomePhabricator
Create Task
Maniphest T321665

SX Publishing: Redirection to target article fails with "Bad title" exception
Closed, ResolvedPublicBUG REPORT
Actions

Description

Steps to replicate the issue:

Create a section translation (using SX) with target article that contains non-ASCII characters and publish it.

Here is a screencast of such a case in Greek (el) Wikipedia:

What happens?:
After successful publishing, the application redirects to an error page with "Bad Title" exception.

What should have happened instead?:
The application should redirect the user to the target page.

Details

SubjectRepoBranchLines +/-
CX3 Build 0.2.0+20221031mediawiki/extensions/ContentTranslationmaster+478 -284
SX: Fix redirection after successful publishing for non-ASCII titlesmediawiki/extensions/ContentTranslationmaster+5 -3
Customize query in gerrit

Related Objects
Search...

Event Timeline

ngkountas created this task.Oct 26 2022, 10:36 AM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptOct 26 2022, 10:36 AM
ngkountas claimed this task.Oct 26 2022, 10:37 AM
ngkountas triaged this task as High priority.
ngkountas moved this task from Priority: Unified Content & Section Translation to In Progress on the Language-Team (Language-2022-October-December) board.
ngkountas moved this task from In Progress to In Review on the Language-Team (Language-2022-October-December) board.Oct 26 2022, 11:35 AM
Nikerabbit added a project: Unplanned-Sprint-Work.Oct 26 2022, 1:48 PM
Nikerabbit added a parent task: T252542: Section Translation Editor: Preview and publish.Oct 26 2022, 1:52 PM
gerritbot added a comment.Oct 27 2022, 7:39 AM

Change 850004 had a related patch set uploaded (by Nik Gkountas; author: Nik Gkountas):

[mediawiki/extensions/ContentTranslation@master] SX: Fix redirection after successful publishing for non-ASCII titles

https://gerrit.wikimedia.org/r/850004

gerritbot added a project: Patch-For-Review.Oct 27 2022, 7:39 AM
Nikerabbit added a project: SectionTranslation.Oct 27 2022, 1:42 PM
gerritbot added a comment.Oct 30 2022, 12:53 PM

Change 850004 merged by Nik Gkountas:

[mediawiki/extensions/ContentTranslation@master] SX: Fix redirection after successful publishing for non-ASCII titles

https://gerrit.wikimedia.org/r/850004

ReleaseTaggerBot added a project: MW-1.40-notes (1.40.0-wmf.8; 2022-10-31).Oct 30 2022, 1:00 PM
Maintenance_bot removed a project: Patch-For-Review.Oct 30 2022, 1:30 PM
gerritbot added a comment.Oct 31 2022, 9:03 AM

Change 850987 had a related patch set uploaded (by Santhosh; author: Santhosh):

[mediawiki/extensions/ContentTranslation@master] CX3 Build 0.2.0+20221031

https://gerrit.wikimedia.org/r/850987

gerritbot added a project: Patch-For-Review.Oct 31 2022, 9:03 AM
gerritbot added a comment.Oct 31 2022, 10:25 AM

Change 850987 merged by jenkins-bot:

[mediawiki/extensions/ContentTranslation@master] CX3 Build 0.2.0+20221031

https://gerrit.wikimedia.org/r/850987

Maintenance_bot removed a project: Patch-For-Review.Oct 31 2022, 10:30 AM
ngkountas moved this task from In Review to Check after deployment on the Language-Team (Language-2022-October-December) board.Nov 2 2022, 3:14 PM
Nikerabbit moved this task from Check after deployment to Needs QA on the Language-Team (Language-2022-October-December) board.Nov 9 2022, 3:08 PM
EChukwukere-WMF subscribed.Nov 16 2022, 5:13 AM

Test status: QA PASS

this shows the expected sandbox name and not an error

EChukwukere-WMF moved this task from Needs QA to Done on the Language-Team (Language-2022-October-December) board.Nov 16 2022, 5:13 AM
Pginer-WMF closed this task as Resolved.Dec 1 2022, 9:16 AM
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL