Last modified: 2014-06-30 01:03:46 UTC
This commit removed parentheses from extracts: https://gerrit.wikimedia.org/r/#/c/132663/ However, the space before the parentheses is not removed, which results in an incorrect space, often directly before punctuation. Example: http://en.wikifur.com/wiki/Category:NordicFuzzCon_staff For the article "Vappu", which starts: "Vappu (born August 7), also known as Lexy, Alexis and Alexistar..." The extract is: "Vappu , also known as Lexy, Alexis and Alexistar..." There should not be a space after "Vappu". I think you could do something like "(\s?)" before the rest of the regex to remove this.
The following commit addressed this issue - https://gerrit.wikimedia.org/r/#/c/135759/ Have you upgraded and still facing this problem?
Yes, we are running on the current HEAD. That commit removed parentheses and the material within them. It left the space immediately before the parentheses when the parentheses was followed by punctuation, such as a comma or full stop. (This was not handled by the regexp that was replaced, either.) The line: extract = extract.replace(/\s+/g, ' '); // Remove extra white spaces will only replace two or more spaces with one. It does not remove the single space between "Vappu" and "," in the example above.
Understood. I'll add failing test cases for the example you mentioned and look into this. Thank you!
Cool! If you do make changes, it might also be worth revisiting the name of the function and the accompanying comment, since what is being removed is not what I'd call brackets - [] or {} - but parentheses - () - and also their content. I know (...now, per https://en.wikipedia.org/wiki/Bracket ) that parentheses are technically a type of bracket, but in that case I would expect other brackets to be affected as well. Perhaps something like removeParenthesizedContent()? (Another possibility is that other types of brackets should be removed, but that'd be an entirely separate bug.)
> I know (...now, per https://en.wikipedia.org/wiki/Bracket ) that parentheses are technically a type of bracket, but in that case I would expect other brackets to be affected as well. Perhaps something like removeParenthesizedContent()? This patch corrected the name of the function and some variable names too.
> This patch corrected the name of the function and some variable names too. Sorry! Forgot to add the link - https://gerrit.wikimedia.org/r/#/c/141661/
