Page MenuHomePhabricator
Feed Advanced Search

Wed, May 1

labster committed rECSEc66f2dd3b688: Simplify using nextUntil with callback function (authored by ashcslmn).
Simplify using nextUntil with callback function
Wed, May 1, 9:25 AM
labster committed rECSE287b196e1c0a: Simplify using nextUntil with callback function (authored by ashcslmn).
Simplify using nextUntil with callback function
Wed, May 1, 9:22 AM
labster committed rEUSP7647ef69f99a: build: Updating mediawiki/mediawiki-phan-config to 0.14.0 (authored by LibUp-bot).
build: Updating mediawiki/mediawiki-phan-config to 0.14.0
Wed, May 1, 8:23 AM
labster committed rEUSPf267572b25cb: Update service wiring for MW 1.42.
Update service wiring for MW 1.42
Wed, May 1, 4:24 AM

Mon, Apr 29

labster committed rECSEad16c747edc0: Simplify using nextUntil with callback function (authored by ashcslmn).
Simplify using nextUntil with callback function
Mon, Apr 29, 5:32 AM

Apr 8 2024

labster added a comment to T361991: Regex search doesn't work with `/` delimiter in maintenance/grep.php .

Yeah, I have had a Gerrit account for seven years but I have finally gotten around to figuring out how it works. Hopefully this is what you need.

Apr 8 2024, 1:57 AM · MW-1.41-notes, MW-1.40-notes, MW-1.39-notes, MW-1.42-notes (1.42.0-wmf.26; 2024-04-09), Patch-For-Review, MediaWiki-Maintenance-system, MW-1.42-release, MW-1.41-release, MW-1.40-release, MW-1.39-release

Apr 6 2024

labster renamed T361991: Regex search doesn't work with `/` delimiter in maintenance/grep.php from Regex flag doesn't work with `/` delimiter in maintenance/grep.php to Regex search doesn't work with `/` delimiter in maintenance/grep.php .
Apr 6 2024, 3:14 AM · MW-1.41-notes, MW-1.40-notes, MW-1.39-notes, MW-1.42-notes (1.42.0-wmf.26; 2024-04-09), Patch-For-Review, MediaWiki-Maintenance-system, MW-1.42-release, MW-1.41-release, MW-1.40-release, MW-1.39-release
labster created T361991: Regex search doesn't work with `/` delimiter in maintenance/grep.php .
Apr 6 2024, 3:12 AM · MW-1.41-notes, MW-1.40-notes, MW-1.39-notes, MW-1.42-notes (1.42.0-wmf.26; 2024-04-09), Patch-For-Review, MediaWiki-Maintenance-system, MW-1.42-release, MW-1.41-release, MW-1.40-release, MW-1.39-release

Oct 4 2022

labster added a comment to T236809: Refactor Parser.php to allow alternate parser (Parsoid).

The release notes currently say:

* The following parser-related hooks have been deprecated:
  […]
  - ParserSectionCreate
    * No replacement; <section> tag wrapping will be done by core in future.
  - ParserPreSaveTransformComplete
    * No replacement; Content::preSaveTransform() provides for customizable PSTs
  - BeforeParserrenderImageGallery
    * No replacement; MediaHandler provides for customizable media rendering

I'm not exited to see "no replacement". What I wish we would have done is to first get rid of deprecated calls in codebases we actively monitor, and only then hard-deprecate stuff. What are the next steps now? Rewrite code that relies on these hooks? Who is supposed to do this? When? Consuming which budget? And a rewrite seems what's now necessary to keep the Wigo3 extension running.

I don't own any of the codebases in question, so I have not much reason to care about them. I'm not even paid to look after them. But what I care about is that we try to help volunteers as good as we can. And I think we can do better.

Oct 4 2022, 3:50 AM · Parsoid-Read-Views, MW-1.36-notes (1.36.0-wmf.8; 2020-09-08), MediaWiki-extensions-Variables, MW-1.35-notes (1.35.0-wmf.36; 2020-06-09), Parsing-Active-Work, Parsoid, MediaWiki-Parser

Dec 28 2020

labster added a comment to T155336: CategoryTree shows dropdown when subcategory size is negative.

It is a public wiki where it could be seen three years ago for a couple months. It has since disappeared -- not sure if there was some maint script that has been run in the meantime, but there have been several MW version upgrades since then. Likely pages added to the category, too. Can no longer reproduce, sorry.

Dec 28 2020, 8:49 AM · MediaWiki-extensions-CategoryTree

Jan 10 2018

labster added a comment to T119908: [RfC]: Migrate code review / management from Gerrit to Phabricator.

It's not as big of a a barrier to contribution as you think. Generally, when I want to commit something on a project hosted by WMF, I push a commit to somewhere else like Github, and then ask someone else who understands Gerrit to merge it. Usually, this is the extension developer, and from there they can either do git remote add or download a patch file.

Jan 10 2018, 4:48 AM · TechCom-RFC (TechCom-RFC-Closed), Developer-Advocacy, releng-201516-q3, Gerrit, Gerrit-Migration, Phabricator, Release-Engineering-Team

Sep 16 2017

labster added a comment to T173835: Invalid magic word 'cscore' when trying to rebuild LC with Contribution Scores.

This one is marked as resolved in our tracker at Miraheze, @Reception123. Is it ok to close this bug, or did someone find the cause?

Sep 16 2017, 9:07 PM · ContributionScores

Aug 28 2017

labster created T174373: Extension:ArticleToCategory2 -- HTML comments are not valid JS.
Aug 28 2017, 7:10 PM · MediaWiki-extensions-ArticleToCategory2

May 21 2017

labster added a comment to T150634: README.md mentions $gcwikis but the code doesn't.

Huh. I didn't expect when I wrote this ticket that it would be resolved by deleting all the documentation. Thanks, I guess?

May 21 2017, 3:43 AM · Documentation, Technical-Debt, GlobalContribs

Apr 17 2017

labster added a comment to T152929: WikiForum overwrites $wgCaptchaTriggers when adding a new permission.

Well, I'll fire our sysadmins just as soon as I start paying them :P

Apr 17 2017, 8:33 PM · MW-1.31-release-notes (WMF-deploy-2017-10-31 (1.31.0-wmf.6)), ConfirmEdit (CAPTCHA extension), MW-1.30-release-notes (WMF-deploy-2017-08-29 (1.30.0-wmf.16)), Patch-For-Review, Social-Tools, WikiForum

Mar 25 2017

labster added a comment to T161387: Change AutoCreateCategoryPages to not have to grab entire namespace.

That credit works for me.

Mar 25 2017, 7:41 PM · MediaWiki-extensions-AutoCreateCategoryPages
labster added a comment to T161387: Change AutoCreateCategoryPages to not have to grab entire namespace.

Hi, I'm here, a little. Just going to say that caching the entire category namespace sounds like a nightmare, if only because of the cache invalidation problem. So I just removed the TODO, because that seems like the Wrong Thing.

Mar 25 2017, 8:49 AM · MediaWiki-extensions-AutoCreateCategoryPages

Feb 13 2017

labster added a comment to T152858: Replace YUI color picker in Special:FanBoxes.

Just want to let you know that this task is what kept us from installing this extension on allthetropes.org. Using an unsupported third-party library doesn't inspire confidence from a privacy point-of-view.

Feb 13 2017, 8:52 PM · JavaScript, Technical-Debt, Social-Tools, FanBoxes

Feb 10 2017

labster created T157770: Extension:CreateRedirect doesn't handle categories gracefully.
Feb 10 2017, 5:44 AM · MediaWiki-extensions-CreateRedirect
labster added a comment to T28898: CreateRedirect extension needs code cleanup.

Just a comment: this seems like a cleanup here would be a easy task. The body file is under 200 lines of code.

Feb 10 2017, 5:36 AM · MediaWiki-extensions-CreateRedirect, Technical-Debt

Jan 18 2017

labster added a comment to T154076: MWExceptionRenderer::reportOutageHTML() produces invalid HTML.

Just because the issue has come up again and I've done more reading since then... apparently <html> and <body> elements aren't required for valid HTML 5. But the <!DOCTYPE html> is missing, and that is required. Same suggested solution as in the comment above.

Jan 18 2017, 3:25 AM · MediaWiki-General

Jan 15 2017

labster created T155336: CategoryTree shows dropdown when subcategory size is negative.
Jan 15 2017, 7:41 AM · MediaWiki-extensions-CategoryTree

Dec 29 2016

labster added a comment to T154106: Hardcoded username in resources/js/twitter.js.

Thanks for the quick turnaround!

Dec 29 2016, 3:13 AM · good first task, Gamepress

Dec 25 2016

labster created T154106: Hardcoded username in resources/js/twitter.js.
Dec 25 2016, 10:46 AM · good first task, Gamepress
labster added a comment to T154076: MWExceptionRenderer::reportOutageHTML() produces invalid HTML.

OK @Aklapper, I'll bite. Convince me that the second part is a separate task, and I'll open another task. I'm rather thinking that if the call to reportOutageHTML() moved into reportHTML() ... right around here ... that all of the above will be solved at the same time.

Dec 25 2016, 10:29 AM · MediaWiki-General

Dec 24 2016

labster updated the task description for T154076: MWExceptionRenderer::reportOutageHTML() produces invalid HTML.
Dec 24 2016, 5:24 AM · MediaWiki-General
labster created T154076: MWExceptionRenderer::reportOutageHTML() produces invalid HTML.
Dec 24 2016, 5:22 AM · MediaWiki-General

Dec 17 2016

labster added a comment to T153557: Extension:MassEditRegex branch REL1_28 is incompatible with MW 1.28.

Ah, OK. You just said "already fixed in master" and assigned a duplicate task that is already closed, but didn't say that there was a fix for this branch. I assumed that you didn't understand the request.

Dec 17 2016, 10:49 PM · MediaWiki-extensions-MassEditRegex
labster changed the status of T153557: Extension:MassEditRegex branch REL1_28 is incompatible with MW 1.28 from Duplicate to Declined.

If you're not going to fix the problem, don't just close the ticket resolved. I'm taking this to mean that you're declining to fix this issue.

Dec 17 2016, 10:43 PM · MediaWiki-extensions-MassEditRegex
labster created T153557: Extension:MassEditRegex branch REL1_28 is incompatible with MW 1.28.
Dec 17 2016, 9:24 PM · MediaWiki-extensions-MassEditRegex

Dec 12 2016

labster added a comment to T135367: Potential query optimization on getComments.

Your guide seems pretty good, I'll give it a shot the next time I think of a patch. Actually, BlogPageClass.php is taunting me with:

Dec 12 2016, 10:52 PM · Performance Issue, Social-Tools, MediaWiki-extensions-Comments
labster added a comment to T143511: Integrate closable/openable content sections in Refreshed skin.

Nope. The sectioning is done here:
https://github.com/wikimedia/mediawiki-extensions-MobileFrontend/blob/master/includes/MobileFormatter.php#L541 That looks complicated.

Dec 12 2016, 8:05 AM · Refreshed, Brickimedia
labster added a comment to T146114: Refreshed is zoomable on mobile, which allows the user interface to break .

As of iOS 10, all pages are zoomable. There is no API to prevent zooming on a page.

Dec 12 2016, 3:42 AM · Brickimedia, Refreshed
labster created T152939: Refreshed: Header floats above content with wikitable in site notice.
Dec 12 2016, 3:22 AM · CSS, Design, Refreshed

Dec 11 2016

labster added a project to T152929: WikiForum overwrites $wgCaptchaTriggers when adding a new permission: WikiForum.
Dec 11 2016, 10:04 PM · MW-1.31-release-notes (WMF-deploy-2017-10-31 (1.31.0-wmf.6)), ConfirmEdit (CAPTCHA extension), MW-1.30-release-notes (WMF-deploy-2017-08-29 (1.30.0-wmf.16)), Patch-For-Review, Social-Tools, WikiForum
labster created T152929: WikiForum overwrites $wgCaptchaTriggers when adding a new permission.
Dec 11 2016, 10:03 PM · MW-1.31-release-notes (WMF-deploy-2017-10-31 (1.31.0-wmf.6)), ConfirmEdit (CAPTCHA extension), MW-1.30-release-notes (WMF-deploy-2017-08-29 (1.30.0-wmf.16)), Patch-For-Review, Social-Tools, WikiForum

Dec 5 2016

labster added a comment to T146712: all kinds of mixed issues with miraheze table (was: allthetropes is not updating on wikistats).

You chose the worst day to start working on this as our db server decided to crash today. I'll get back to you when we're back online.

Dec 5 2016, 10:38 PM · Patch-For-Review, VPS-project-Wikistats

Nov 14 2016

labster created T150634: README.md mentions $gcwikis but the code doesn't.
Nov 14 2016, 5:27 AM · Documentation, Technical-Debt, GlobalContribs

Nov 5 2016

labster created T150082: Extension:SiteScout needless use of eval in JS.
Nov 5 2016, 9:21 AM · Social-Tools

Oct 16 2016

labster created T148316: PHP warning with exactly one avatar uploaded.
Oct 16 2016, 4:25 AM · Patch-For-Review, Social-Tools, RandomUsersWithAvatars

Oct 7 2016

labster added a comment to T135367: Potential query optimization on getComments.

Can anyone take over maintainership for this extension?

Oct 7 2016, 10:02 PM · Performance Issue, Social-Tools, MediaWiki-extensions-Comments

Oct 5 2016

labster changed the status of T146564: With VisualEditor beta-disabled, pipes are missing on other extensions using SkinEditSectionLinks from Stalled to Open.

I don't understand. How is mw-editsection-divider an internal class for VE if it's in includes/skins/Skin.php? And if SkinEditSectionLinks is not a public API, why is it mentioned in docs/hooks.txt as replacement for a different public API?

Oct 5 2016, 2:52 AM · VisualEditor-MediaWiki, VisualEditor

Sep 26 2016

labster created T146712: all kinds of mixed issues with miraheze table (was: allthetropes is not updating on wikistats).
Sep 26 2016, 11:07 PM · Patch-For-Review, VPS-project-Wikistats

Sep 25 2016

labster created T146564: With VisualEditor beta-disabled, pipes are missing on other extensions using SkinEditSectionLinks.
Sep 25 2016, 1:08 AM · VisualEditor-MediaWiki, VisualEditor

Sep 21 2016

labster added a comment to T142314: Null byte in old versions of Replace Text may cause arbitrary execution.

In terms of the DOS attack @Bawolff mentioned, maybe ini_set on some PCRE options would be useful?

Sep 21 2016, 6:15 AM · Security, MediaWiki-extensions-ReplaceText

Sep 3 2016

labster placed T135367: Potential query optimization on getComments up for grabs.

No activity in over 3 months. If no one is interested in merging this code, can they please decline the task?

Sep 3 2016, 8:01 PM · Performance Issue, Social-Tools, MediaWiki-extensions-Comments

Aug 7 2016

labster added a comment to T142314: Null byte in old versions of Replace Text may cause arbitrary execution.

If you are unable to upgrade to PHP 5.5.9, then you can use MediaWiki 1.23.13, which requires PHP version 5.3.2 or later

Aug 7 2016, 7:17 AM · Security, MediaWiki-extensions-ReplaceText
labster added a comment to T73027: RegExp search and replace doesn't work with MS Sql SERVER.

As far as I can tell, SQL Server does not support regular expressions -- or at least not without using a CLR function.

Aug 7 2016, 1:59 AM · MSSQL, MediaWiki-extensions-ReplaceText
labster added a project to T142314: Null byte in old versions of Replace Text may cause arbitrary execution: MediaWiki-extensions-ReplaceText.
Aug 7 2016, 1:10 AM · Security, MediaWiki-extensions-ReplaceText
labster created T142314: Null byte in old versions of Replace Text may cause arbitrary execution.
Aug 7 2016, 1:10 AM · Security, MediaWiki-extensions-ReplaceText

Jul 31 2016

labster created T141706: Skin:Erudite: Is method msgWikiNoEdit() still needed?.
Jul 31 2016, 5:23 AM · MediaWiki-skins-Erudite

Jul 21 2016

labster added a comment to T132125: Cannot remove a site by editing $wgSiteMatrixSites.

Thanks, this is great work. Sorry I didn't reply earlier, as I've been swamped by $dayjob stuff. I do have some other suggestions, but I'll reply in T140941.

Jul 21 2016, 9:30 PM · MW-1.28-release (WMF-deploy-2016-07-26_(1.28.0-wmf.12)), Patch-For-Review, MediaWiki-Configuration, SiteMatrix

May 22 2016

labster added a comment to T133511: Extension:MsCalendar XSS vulnerability.

I just suggested updating the extension because I thought it would be easiest. If you want, you just just do some HTML filtering in the original extension -- a good place to start would be here:

May 22 2016, 10:49 PM · User-RhinosF1, Security, MediaWiki-extensions-MsCalendar, Security-Extensions, Vuln-XSS
labster added a comment to T133511: Extension:MsCalendar XSS vulnerability.

We're nearing 2 months since I first reported the vulnerability to the authors. Is there any WMF guidance as to how long before I can publicly disclose the vulnerability?

May 22 2016, 1:31 AM · User-RhinosF1, Security, MediaWiki-extensions-MsCalendar, Security-Extensions, Vuln-XSS
labster updated subscribers of T133511: Extension:MsCalendar XSS vulnerability.
May 22 2016, 1:26 AM · User-RhinosF1, Security, MediaWiki-extensions-MsCalendar, Security-Extensions, Vuln-XSS

May 16 2016

labster created T135367: Potential query optimization on getComments.
May 16 2016, 9:44 AM · Performance Issue, Social-Tools, MediaWiki-extensions-Comments

May 9 2016

labster added a comment to T134546: CommentIgnoreList warning about extension message alias.

OK. If wfLoadExtension is recommended, then there was a docs bug on https://www.mediawiki.org/wiki/Extension:Comments I went ahead and added registration=1 to the install template.

May 9 2016, 8:35 AM · Patch-For-Review, Social-Tools, MediaWiki-extensions-Comments

May 6 2016

labster added a comment to T134546: CommentIgnoreList warning about extension message alias.

Oh, right, I have $wgShowDevelopmentWarnings = true;, but doesn't everyone? 🤓

May 6 2016, 2:55 AM · Patch-For-Review, Social-Tools, MediaWiki-extensions-Comments
labster created T134546: CommentIgnoreList warning about extension message alias.
May 6 2016, 2:50 AM · Patch-For-Review, Social-Tools, MediaWiki-extensions-Comments

May 2 2016

labster added a comment to T134153: Swap order of statements to save a DB query.

The same thing applies at https://git.wikimedia.org/blob/mediawiki%2Fextensions%2FComments/master/CommentClass.php#L175 and L317

May 2 2016, 9:26 AM · Patch-For-Review, MediaWiki-extensions-Comments, Social-Tools
labster created T134153: Swap order of statements to save a DB query.
May 2 2016, 8:28 AM · Patch-For-Review, MediaWiki-extensions-Comments, Social-Tools

May 1 2016

labster added a comment to T131515: extension.json is poorly documented.

Agreed, documentation is extremely poor.

May 1 2016, 7:27 AM · MediaWiki-Documentation, Documentation

Apr 25 2016

labster added a comment to T133511: Extension:MsCalendar XSS vulnerability.

That was me informing the authors. With the same message as above. Felipe Schenone replied back to me with this message on 30 March:

Apr 25 2016, 12:36 AM · User-RhinosF1, Security, MediaWiki-extensions-MsCalendar, Security-Extensions, Vuln-XSS

Apr 24 2016

labster created T133511: Extension:MsCalendar XSS vulnerability.
Apr 24 2016, 10:31 PM · User-RhinosF1, Security, MediaWiki-extensions-MsCalendar, Security-Extensions, Vuln-XSS
labster added a comment to T132125: Cannot remove a site by editing $wgSiteMatrixSites.

If it is intentional, I suppose we could host a fork of this. All we'd have to do is delete a few lines from the extension.json file, track upstream otherwise, and warn people outside the WMF to use our fork instead of the official one. Somehow this solution seems suboptimal, but I'd be willing to do it.

Apr 24 2016, 4:38 AM · MW-1.28-release (WMF-deploy-2016-07-26_(1.28.0-wmf.12)), Patch-For-Review, MediaWiki-Configuration, SiteMatrix

Apr 8 2016

labster created T132125: Cannot remove a site by editing $wgSiteMatrixSites.
Apr 8 2016, 2:31 AM · MW-1.28-release (WMF-deploy-2016-07-26_(1.28.0-wmf.12)), Patch-For-Review, MediaWiki-Configuration, SiteMatrix