Page MenuHomePhabricator

Parenthetical: New edge case with nested brackets
Closed, ResolvedPublic


A new edge case has been identified and a test case has been captured in WIP: Improve parenthetical handling.

There is a problem in the removeNestedParentheticals function - it does not correctly strip a parenthetical nested within other parenthetical.

This leads to invalid summaries on the following examples:

Related Objects

Event Timeline

A new edge case has been identified:

What's the edge case?

Change 379930 had a related patch set uploaded (by Jdlrobson; owner: Jdlrobson):
[mediawiki/services/mobileapps@master] Fix removeNestedParentheticals parenthetical handling

Change 379930 merged by jenkins-bot:
[mediawiki/services/mobileapps@master] Fix removeNestedParentheticals parenthetical handling

Mentioned in SAL (#wikimedia-operations) [2017-10-03T22:26:53Z] <bsitzmann@tin> Finished deploy [mobileapps/deploy@82aa7d6]: Update mobileapps to 5dc0c02 (T175762 T177001 T176525 T176517 T176519) (duration: 06m 14s)

Jdlrobson claimed this task.