Page MenuHomePhabricator

ssastry (Subramanya Sastry)
User

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Wednesday

  • Clear sailing ahead.

User Details

User Since
Oct 7 2014, 5:34 AM (266 w, 6 d)
Availability
Available
LDAP User
Subramanya Sastry
MediaWiki User
SSastry (WMF) [ Global Accounts ]

Recent Activity

Today

ssastry lowered the priority of T238387: Associative array collapses to a non-associative form when keys represent contiguous integer values causing JSON output diffs from Normal to Low.

As the above snippet shows, $a becomes a regular array and when json-encoded, the encoding uses regular array syntax. This is likely a narrow edge case but something worth investigation. This is likely to be only fixed with typed token properties.

Mon, Nov 18, 2:04 AM · Parsoid-PHP
ssastry renamed T238387: Associative array collapses to a non-associative form when keys represent contiguous integer values causing JSON output diffs from PHP vs JS falsy check difference to Associative array collapses to a non-associative form when keys represent contiguous integer values causing JSON output diffs.
Mon, Nov 18, 2:02 AM · Parsoid-PHP
ssastry added a comment to T238387: Associative array collapses to a non-associative form when keys represent contiguous integer values causing JSON output diffs.

This is triggered by a wikitext error in a table row. | ... ||0 | 0. That last 0| should have been ||0.
But, this gives us an easy test case:

[subbu@earth:~/work/wmf/parsoid] echo -e "{|\n|1||0|0\n|}" | node bin/parse.js --body_only
<table data-parsoid='{"dsr":[0,13,2,2]}'>
<tbody data-parsoid='{"dsr":[3,11,0,0]}'><tr data-parsoid='{"autoInsertedStart":true,"dsr":[3,10,0,0]}'><td data-parsoid='{"dsr":[3,5,1,0]}'>1</td><td data-parsoid='{"stx":"row","a":{"0":null},"sa":{"0":""},"dsr":[5,10,4,0]}'>0</td></tr>
</tbody></table>
Mon, Nov 18, 1:47 AM · Parsoid-PHP
ssastry claimed T238463: strlen() expects parameter 1 to be string, array given.
Mon, Nov 18, 12:45 AM · Patch-For-Review, Parsoid-PHP

Sat, Nov 16

ssastry moved T238463: strlen() expects parameter 1 to be string, array given from Backlog to Bugs on the Parsoid-PHP board.
Sat, Nov 16, 5:23 AM · Patch-For-Review, Parsoid-PHP
ssastry triaged T238463: strlen() expects parameter 1 to be string, array given as Normal priority.
Sat, Nov 16, 5:22 AM · Patch-For-Review, Parsoid-PHP
ssastry created T238463: strlen() expects parameter 1 to be string, array given.
Sat, Nov 16, 5:22 AM · Patch-For-Review, Parsoid-PHP

Fri, Nov 15

ssastry moved T238456: Missing implementation to post Parsoid/PHP lints to production database from Backlog to Porting on the Parsoid-PHP board.
Fri, Nov 15, 11:29 PM · MediaWiki-extensions-Linter, Parsoid-PHP
ssastry moved T238457: PHP Notice: Trying to get property 'start' of non-object from Backlog to Bugs on the Parsoid-PHP board.
Fri, Nov 15, 11:29 PM · Parsoid-PHP
ssastry triaged T238457: PHP Notice: Trying to get property 'start' of non-object as Normal priority.
Fri, Nov 15, 11:29 PM · Parsoid-PHP
ssastry created T238457: PHP Notice: Trying to get property 'start' of non-object.
Fri, Nov 15, 11:29 PM · Parsoid-PHP
ssastry added a project to T238456: Missing implementation to post Parsoid/PHP lints to production database: MediaWiki-extensions-Linter.
Fri, Nov 15, 10:34 PM · MediaWiki-extensions-Linter, Parsoid-PHP
ssastry updated subscribers of T238456: Missing implementation to post Parsoid/PHP lints to production database.
Fri, Nov 15, 10:34 PM · MediaWiki-extensions-Linter, Parsoid-PHP
ssastry added a subtask for T237326: Make Parsoid/PHP cluster read-write to ensure lints discovered by Parsoid/PHP are stored in the DB: T238456: Missing implementation to post Parsoid/PHP lints to production database.
Fri, Nov 15, 10:33 PM · Core Platform Team, User-WDoran, Parsoid-PHP
ssastry added a parent task for T238456: Missing implementation to post Parsoid/PHP lints to production database: T237326: Make Parsoid/PHP cluster read-write to ensure lints discovered by Parsoid/PHP are stored in the DB.
Fri, Nov 15, 10:33 PM · MediaWiki-extensions-Linter, Parsoid-PHP
ssastry triaged T238456: Missing implementation to post Parsoid/PHP lints to production database as High priority.
Fri, Nov 15, 10:33 PM · MediaWiki-extensions-Linter, Parsoid-PHP
eprodromou awarded T236930: API Developer supports different request media types a Party Time token.
Fri, Nov 15, 9:20 PM · Patch-For-Review, Core Platform Team Workboards (Green), Story, CPT Initiatives (Parsoid REST API in PHP (CDP2)), MediaWiki-REST-API, Parsoid-PHP
ssastry added a comment to T229074: Preparing VisualEditor for Parsoid-PHP switch.

@Ryasmeen: the 'new page creation' bug is now resolved and new code is on beta cluster as of y'day. Let us know if you find any other bugs. Or, a confirmation that you could run all your QA tests successfully! Thanks!

Fri, Nov 15, 8:43 PM · Editing QA, Patch-For-Review, MW-1.35-notes (1.35.0-wmf.8; 2019-11-26), VisualEditor (Current work), Core Platform Team, Parsoid-PHP
ssastry closed T237569: Linter extension is currently incompatible with Parsoid/PHP, a subtask of T237326: Make Parsoid/PHP cluster read-write to ensure lints discovered by Parsoid/PHP are stored in the DB, as Resolved.
Fri, Nov 15, 8:39 PM · Core Platform Team, User-WDoran, Parsoid-PHP
ssastry closed T237569: Linter extension is currently incompatible with Parsoid/PHP as Resolved.

We went with option #1 here. We don't expect perf. issues but given our instrumentation, we'll know if something is off here and can revisit this.

Fri, Nov 15, 8:39 PM · Parsoid-Linter, MediaWiki-extensions-Linter, Parsoid-PHP
ssastry added a subtask for T229015: Tracking: Direct live production traffic at Parsoid/PHP: T237326: Make Parsoid/PHP cluster read-write to ensure lints discovered by Parsoid/PHP are stored in the DB.
Fri, Nov 15, 8:34 PM · Core Platform Team, User-WDoran, Parsoid-PHP
ssastry added a parent task for T237326: Make Parsoid/PHP cluster read-write to ensure lints discovered by Parsoid/PHP are stored in the DB: T229015: Tracking: Direct live production traffic at Parsoid/PHP.
Fri, Nov 15, 8:34 PM · Core Platform Team, User-WDoran, Parsoid-PHP
ssastry removed a parent task for T237461: Parsoid/PHP LintLogger has gaps relative to Parsoid/JS: T229015: Tracking: Direct live production traffic at Parsoid/PHP.
Fri, Nov 15, 8:33 PM · Patch-For-Review, Parsoid-PHP
ssastry removed a subtask for T229015: Tracking: Direct live production traffic at Parsoid/PHP: T237461: Parsoid/PHP LintLogger has gaps relative to Parsoid/JS.
Fri, Nov 15, 8:33 PM · Core Platform Team, User-WDoran, Parsoid-PHP
ssastry added a parent task for T237461: Parsoid/PHP LintLogger has gaps relative to Parsoid/JS: T229015: Tracking: Direct live production traffic at Parsoid/PHP.
Fri, Nov 15, 8:33 PM · Patch-For-Review, Parsoid-PHP
ssastry added a subtask for T229015: Tracking: Direct live production traffic at Parsoid/PHP: T237461: Parsoid/PHP LintLogger has gaps relative to Parsoid/JS.
Fri, Nov 15, 8:33 PM · Core Platform Team, User-WDoran, Parsoid-PHP
ssastry added a comment to T229075: Preparing ContentTranslation for Parsoid-PHP switch.

Language team: As of yesterday, new code is on both beta cluster and production that can handle new pages. You may have already inferred it by the resolution of the blocking subtask, but just in case you hadn't, this is FYI about it.

Fri, Nov 15, 8:29 PM · Core Platform Team, Language-Team (Language-2019-October-December), ContentTranslation, Parsoid-PHP
ssastry closed T236930: API Developer supports different request media types as Resolved.

Thanks @eprodromou and @tstarling for the attention and pointers.

Fri, Nov 15, 8:24 PM · Patch-For-Review, Core Platform Team Workboards (Green), Story, CPT Initiatives (Parsoid REST API in PHP (CDP2)), MediaWiki-REST-API, Parsoid-PHP
ssastry closed T236930: API Developer supports different request media types, a subtask of T229015: Tracking: Direct live production traffic at Parsoid/PHP, as Resolved.
Fri, Nov 15, 8:24 PM · Core Platform Team, User-WDoran, Parsoid-PHP
ssastry closed T236864: Call to a member function getArticleID() on null as Resolved.
Fri, Nov 15, 6:49 PM · Parsoid-PHP
ssastry moved T238435: Output differences in dates on a jawiki page - possible argument parsing diffs in citation templates in <ref> tags from Backlog to Bugs on the Parsoid-PHP board.
Fri, Nov 15, 6:32 PM · Parsoid-PHP
ssastry triaged T238435: Output differences in dates on a jawiki page - possible argument parsing diffs in citation templates in <ref> tags as Normal priority.
Fri, Nov 15, 6:31 PM · Parsoid-PHP
ssastry moved T238420: Whitespace parsing issue in arguments (alt="" vs alt=" ") from Bugs to Parsoid/JS bugs on the Parsoid-PHP board.
Fri, Nov 15, 5:52 PM · Parsoid-PHP
ssastry lowered the priority of T238420: Whitespace parsing issue in arguments (alt="" vs alt=" ") from Normal to Low.
Fri, Nov 15, 5:51 PM · Parsoid-PHP
ssastry added a comment to T238420: Whitespace parsing issue in arguments (alt="" vs alt=" ").

Unsurprisingly, Parsoid/PHP once again matches core rendering here and Parsoid/JS is the one that differs.

Fri, Nov 15, 5:51 PM · Parsoid-PHP
ssastry updated the task description for T238420: Whitespace parsing issue in arguments (alt="" vs alt=" ").
Fri, Nov 15, 5:50 PM · Parsoid-PHP
ssastry moved T238422: Attribute char encoding differences from Backlog to Parsoid/JS bugs on the Parsoid-PHP board.
Fri, Nov 15, 5:37 PM · Parsoid-PHP
ssastry triaged T238422: Attribute char encoding differences as Low priority.
Fri, Nov 15, 5:37 PM · Parsoid-PHP
ssastry lowered the priority of T238419: Quote parsing difference from Normal to Low.
Fri, Nov 15, 5:35 PM · Parsoid-PHP
ssastry added a comment to T238419: Quote parsing difference.

Removed 4 duplicate comments that actually went through while Phab was unresponsive.

Fri, Nov 15, 5:34 PM · Parsoid-PHP
ssastry added a comment to T238419: Quote parsing difference.
Fri, Nov 15, 5:33 PM · Parsoid-PHP
ssastry added a comment to T238419: Quote parsing difference.
Fri, Nov 15, 5:27 PM · Parsoid-PHP
ssastry moved T238419: Quote parsing difference from Bugs to Parsoid/JS bugs on the Parsoid-PHP board.
Fri, Nov 15, 5:25 PM · Parsoid-PHP
ssastry added a comment to T238419: Quote parsing difference.
Fri, Nov 15, 5:25 PM · Parsoid-PHP
ssastry added a comment to T238419: Quote parsing difference.
Fri, Nov 15, 5:22 PM · Parsoid-PHP
ssastry added a comment to T238419: Quote parsing difference.

Looking at the output of https://uk.wikipedia.org/wiki/Hitman:_Contracts, Parsoid/PHP matches the current rendering. Parsoid/JS seems to be at fault here.

Fri, Nov 15, 5:21 PM · Parsoid-PHP
ssastry closed T237103: PHP Notice: Undefined offset: 2 as Resolved.
Fri, Nov 15, 5:16 PM · Parsoid-PHP
ssastry moved T238419: Quote parsing difference from Backlog to Bugs on the Parsoid-PHP board.
Fri, Nov 15, 5:13 PM · Parsoid-PHP
ssastry moved T238420: Whitespace parsing issue in arguments (alt="" vs alt=" ") from Backlog to Bugs on the Parsoid-PHP board.
Fri, Nov 15, 5:13 PM · Parsoid-PHP
ssastry triaged T238420: Whitespace parsing issue in arguments (alt="" vs alt=" ") as Normal priority.
Fri, Nov 15, 5:11 PM · Parsoid-PHP
ssastry updated the task description for T238419: Quote parsing difference.
Fri, Nov 15, 5:10 PM · Parsoid-PHP
ssastry triaged T238419: Quote parsing difference as Normal priority.
Fri, Nov 15, 5:09 PM · Parsoid-PHP
ssastry added a comment to T237103: PHP Notice: Undefined offset: 2.

I see a bunch of similar errors in the parsoid-tests logstash board: https://logstash.wikimedia.org/app/kibana#/dashboard/parsoid-tests?_g=h@97fe121&_a=h@d8af28b

Fri, Nov 15, 12:28 AM · Parsoid-PHP
ssastry added a comment to T237103: PHP Notice: Undefined offset: 2.

I see a bunch of similar errors in the parsoid-tests logstash board: https://logstash.wikimedia.org/app/kibana#/dashboard/parsoid-tests?_g=h@97fe121&_a=h@d8af28b

Fri, Nov 15, 12:27 AM · Parsoid-PHP

Thu, Nov 14

ssastry edited projects for T237306: PEG error: Expected end of input or tlb but "\r" found., added: Parsoid; removed Parsoid-PHP.
Thu, Nov 14, 11:19 PM · Parsoid
ssastry moved T238388: Lot of diffs: Both Parsoid/JS & Parsoid/PHP look badly broken from Backlog to Bugs on the Parsoid-PHP board.
Thu, Nov 14, 11:18 PM · Parsoid-PHP
ssastry triaged T238388: Lot of diffs: Both Parsoid/JS & Parsoid/PHP look badly broken as Normal priority.
Thu, Nov 14, 11:17 PM · Parsoid-PHP
ssastry moved T238387: Associative array collapses to a non-associative form when keys represent contiguous integer values causing JSON output diffs from Backlog to Bugs on the Parsoid-PHP board.
Thu, Nov 14, 11:14 PM · Parsoid-PHP
ssastry triaged T238387: Associative array collapses to a non-associative form when keys represent contiguous integer values causing JSON output diffs as Normal priority.
Thu, Nov 14, 11:14 PM · Parsoid-PHP
ssastry edited projects for T237538: Merge Disambiguation in core or add hook, added: Parsoid; removed Parsoid-PHP.
Thu, Nov 14, 11:09 PM · Parsoid, MediaWiki-extensions-Disambiguator
ssastry triaged T237754: Adjust TimedMedia url handling (getAPIData) to match legacy parser as Normal priority.
Thu, Nov 14, 11:09 PM · Parsoid, MW-1.35-notes (1.35.0-wmf.8; 2019-11-26), TimedMediaHandler
ssastry added a comment to T229077: Preparing MCS for Parsoid-PHP switch.

Hi @ssastry -
Two potential issues I noticed when running the tests with the header to request the php output:

  1. Previously scheme-less urls for videos now have a scheme. For example, on enwiki Hummingbird/810247947 the URLs for videos had their prefixes change from //upload.wikimedia.org/ to https://upload.wikimedia.org/
  2. The title in the href of the link tag with rel="dc:isVersionOf now uses spaces instead of underscores. For example, on enwiki User:BSitzmann_(WMF)/MCS/Test/Frankenstein/778666613 the href changed from //en.wikipedia.org/wiki/User%3ABSitzmann_(WMF)/MCS/Test/Frankenstein to //en.wikipedia.org/wiki/User%3ABSitzmann%20(WMF)/MCS/Test/Frankenstein
Thu, Nov 14, 9:39 PM · Patch-For-Review, Product-Infrastructure-Team-Backlog (Kanban), Mobile-Content-Service, Parsoid-PHP
ssastry removed a subtask for T235217: Parsoid should use protocol-relative URLs for media: T237754: Adjust TimedMedia url handling (getAPIData) to match legacy parser.
Thu, Nov 14, 9:10 PM · Parsoid-PHP
ssastry removed a parent task for T237754: Adjust TimedMedia url handling (getAPIData) to match legacy parser: T235217: Parsoid should use protocol-relative URLs for media.
Thu, Nov 14, 9:10 PM · Parsoid, MW-1.35-notes (1.35.0-wmf.8; 2019-11-26), TimedMediaHandler
ssastry closed T238161: [Regression pre-wmf.6] Cannot create a new page on Beta cluster shows error "Error contacting the Parsoid/RESTBase server (HTTP 500)", a subtask of T229074: Preparing VisualEditor for Parsoid-PHP switch, as Resolved.
Thu, Nov 14, 9:08 PM · Editing QA, Patch-For-Review, MW-1.35-notes (1.35.0-wmf.8; 2019-11-26), VisualEditor (Current work), Core Platform Team, Parsoid-PHP
ssastry closed T238161: [Regression pre-wmf.6] Cannot create a new page on Beta cluster shows error "Error contacting the Parsoid/RESTBase server (HTTP 500)", a subtask of T229075: Preparing ContentTranslation for Parsoid-PHP switch, as Resolved.
Thu, Nov 14, 9:08 PM · Core Platform Team, Language-Team (Language-2019-October-December), ContentTranslation, Parsoid-PHP
ssastry closed T238161: [Regression pre-wmf.6] Cannot create a new page on Beta cluster shows error "Error contacting the Parsoid/RESTBase server (HTTP 500)" as Resolved.

This fix is now on beta and I verified it works by creating https://en.wikipedia.beta.wmflabs.org/wiki/T238161_test

Thu, Nov 14, 9:08 PM · User-Ryasmeen, Verified, Core Platform Team Workboards (Clinic Duty Team), Parsoid-PHP, Patch-For-Review
ssastry added a comment to T234549: "Properly" address missing srcText issues in PageConfigFrame.

I'd like to keep the clarity of SelserData, that accessing the page frame while serializing should throw, unless we're doing selser.

Thu, Nov 14, 7:54 PM · Parsoid

Wed, Nov 13

ssastry claimed T238161: [Regression pre-wmf.6] Cannot create a new page on Beta cluster shows error "Error contacting the Parsoid/RESTBase server (HTTP 500)".
Wed, Nov 13, 11:25 PM · User-Ryasmeen, Verified, Core Platform Team Workboards (Clinic Duty Team), Parsoid-PHP, Patch-For-Review
ssastry reassigned T238161: [Regression pre-wmf.6] Cannot create a new page on Beta cluster shows error "Error contacting the Parsoid/RESTBase server (HTTP 500)" from mobrovac to Arlolra.
Wed, Nov 13, 9:10 PM · User-Ryasmeen, Verified, Core Platform Team Workboards (Clinic Duty Team), Parsoid-PHP, Patch-For-Review
ssastry added a comment to T229077: Preparing MCS for Parsoid-PHP switch.

I think Parsoid/PHP is better code here since it handles titles consistently but let me check in with others to see what ought to be canonical title output here.

Wed, Nov 13, 4:17 PM · Patch-For-Review, Product-Infrastructure-Team-Backlog (Kanban), Mobile-Content-Service, Parsoid-PHP
ssastry added a comment to T229077: Preparing MCS for Parsoid-PHP switch.

Will look into this.

Wed, Nov 13, 4:15 PM · Patch-For-Review, Product-Infrastructure-Team-Backlog (Kanban), Mobile-Content-Service, Parsoid-PHP
ssastry added a comment to T229077: Preparing MCS for Parsoid-PHP switch.

Got it. Looks like Parsoid/PHP normalizes titles whereas Parsoid/JS emits it as given.

Wed, Nov 13, 4:12 PM · Patch-For-Review, Product-Infrastructure-Team-Backlog (Kanban), Mobile-Content-Service, Parsoid-PHP
ssastry added a comment to T238218: Control whitespace in injected wikitext for multi-line comments.

One quick obvious thought. Maybe a "data-ve-multiline-reply-hack" attribute to the list-item span which is similar to your data-parsoid="": hack, but something which is less-hacky (since we don't want editors dealing with data-parsoid). We can remove support for this attribute once the multi-line list syntax lands. The "-hack" serves as a reminder to us all not to keep it around.

Wed, Nov 13, 3:12 PM · OWC2020, Parsoid
ssastry added a comment to T229077: Preparing MCS for Parsoid-PHP switch.

Hi @ssastry -
Two potential issues I noticed when running the tests with the header to request the php output:

Wed, Nov 13, 2:04 PM · Patch-For-Review, Product-Infrastructure-Team-Backlog (Kanban), Mobile-Content-Service, Parsoid-PHP
ssastry added a comment to T229074: Preparing VisualEditor for Parsoid-PHP switch.

Editing team: We want to catch problems sooner than later, so checking in about updates on how testing is going here.

We started testing today, with plans to finish on Thursday.

Wed, Nov 13, 1:38 AM · Editing QA, Patch-For-Review, MW-1.35-notes (1.35.0-wmf.8; 2019-11-26), VisualEditor (Current work), Core Platform Team, Parsoid-PHP

Tue, Nov 12

ssastry added a comment to T229077: Preparing MCS for Parsoid-PHP switch.

Hi Product Infra Team: We want to catch problems sooner than later, so checking in about updates on how testing is going here. Anything that you need from us? Or, is everything looking good on your end?

Tue, Nov 12, 10:53 PM · Patch-For-Review, Product-Infrastructure-Team-Backlog (Kanban), Mobile-Content-Service, Parsoid-PHP
ssastry added a comment to T235777: Redlink differences between Parsoid/PHP and Parsoid/JS.

So, in logstash, I see the following warnings emitted by Parsoid/JS: Batch request The value passed for "batch" contains invalid or non-normalized data. Textual data should be valid, NFC-normalized Unicode without C0 control characters other than HT (\t), LF (\n), and CR (\r). for the page frwiki:Ṳ́, which then leads to this warning emtited by the AddRedLinks pass: We should have data for the title: Ś.

Tue, Nov 12, 7:21 PM · Parsoid-PHP
ssastry added a comment to T229075: Preparing ContentTranslation for Parsoid-PHP switch.

Language Team: We want to catch problems sooner than later, so checking in about updates on how testing is going here. Anything that you need from us? Or, is everything looking good on your end?

Tue, Nov 12, 6:17 PM · Core Platform Team, Language-Team (Language-2019-October-December), ContentTranslation, Parsoid-PHP
ssastry added a comment to T229074: Preparing VisualEditor for Parsoid-PHP switch.

Editing team: We want to catch problems sooner than later, so checking in about updates on how testing is going here. Anything that you need from us? Or, is everything looking good except for that one bug that @matmarax found ( T237666: Parsoid/PHP fails for transforms for new pages with slashes in the title ) ?

Tue, Nov 12, 6:15 PM · Editing QA, Patch-For-Review, MW-1.35-notes (1.35.0-wmf.8; 2019-11-26), VisualEditor (Current work), Core Platform Team, Parsoid-PHP

Mon, Nov 11

ssastry renamed T237886: Argument 1 passed to Parsoid\Utils\PHPUtils::encodeURIComponent() must be of the type string, integer given from Argument 1 passed to Parsoid\Utils\PHPUtils::encodeURIComponent() must be of the type string, integer given, called in /srv/deployment/parsoid/deploy-cache/revs/7e86f835f6ae15749b3665620891f5ec8d03311f/src/src/Wt2Html/TT/ExtensionHandler.php on line 109 to Argument 1 passed to Parsoid\Utils\PHPUtils::encodeURIComponent() must be of the type string, integer given.
Mon, Nov 11, 11:35 PM · Wikimedia-production-error, Parsoid-PHP
ssastry assigned T237693: Port templatedata mocha tests to phpunit to Sbailey.
Mon, Nov 11, 11:30 PM · Patch-For-Review, Parsoid-PHP
ssastry moved T238030: Fortuitous bug-fix in Parsoid/PHP (due to PHP DOM & Domino DOM differences) from Backlog to Parsoid/JS bugs on the Parsoid-PHP board.
Mon, Nov 11, 11:29 PM · Parsoid-PHP
ssastry updated the task description for T238030: Fortuitous bug-fix in Parsoid/PHP (due to PHP DOM & Domino DOM differences).
Mon, Nov 11, 11:28 PM · Parsoid-PHP
ssastry triaged T238030: Fortuitous bug-fix in Parsoid/PHP (due to PHP DOM & Domino DOM differences) as Low priority.
Mon, Nov 11, 11:23 PM · Parsoid-PHP
ssastry moved T238022: Parsoid/JS use of \w \s \b etc is inconsistent with PHP's behavior when the 'u' regexp modifier is used, which leads to selective serializer output differences between Parsoid/PHP & Parsoid/JS in some scenarios from Backlog to Parsoid/JS bugs on the Parsoid-PHP board.
Mon, Nov 11, 8:40 PM · Patch-For-Review, Parsoid-PHP
ssastry triaged T238022: Parsoid/JS use of \w \s \b etc is inconsistent with PHP's behavior when the 'u' regexp modifier is used, which leads to selective serializer output differences between Parsoid/PHP & Parsoid/JS in some scenarios as Low priority.
Mon, Nov 11, 8:39 PM · Patch-For-Review, Parsoid-PHP
ssastry created T238022: Parsoid/JS use of \w \s \b etc is inconsistent with PHP's behavior when the 'u' regexp modifier is used, which leads to selective serializer output differences between Parsoid/PHP & Parsoid/JS in some scenarios.
Mon, Nov 11, 8:39 PM · Patch-For-Review, Parsoid-PHP

Sun, Nov 10

ssastry moved T237886: Argument 1 passed to Parsoid\Utils\PHPUtils::encodeURIComponent() must be of the type string, integer given from Backlog to Bugs on the Parsoid-PHP board.
Sun, Nov 10, 8:26 PM · Wikimedia-production-error, Parsoid-PHP
ssastry triaged T237886: Argument 1 passed to Parsoid\Utils\PHPUtils::encodeURIComponent() must be of the type string, integer given as Normal priority.
Sun, Nov 10, 8:26 PM · Wikimedia-production-error, Parsoid-PHP
ssastry created T237886: Argument 1 passed to Parsoid\Utils\PHPUtils::encodeURIComponent() must be of the type string, integer given.
Sun, Nov 10, 8:26 PM · Wikimedia-production-error, Parsoid-PHP

Fri, Nov 8

ssastry claimed T237569: Linter extension is currently incompatible with Parsoid/PHP.
Fri, Nov 8, 10:57 PM · Parsoid-Linter, MediaWiki-extensions-Linter, Parsoid-PHP
ssastry closed T235231: Parsoid/JS video tag has a "seek" parameter in the URL that Parsoid/PHP video tag output doesn't as Resolved.
Fri, Nov 8, 8:51 PM · Parsoid-PHP
ssastry closed T235231: Parsoid/JS video tag has a "seek" parameter in the URL that Parsoid/PHP video tag output doesn't, a subtask of T229015: Tracking: Direct live production traffic at Parsoid/PHP, as Resolved.
Fri, Nov 8, 8:51 PM · Core Platform Team, User-WDoran, Parsoid-PHP

Thu, Nov 7

ssastry moved T237693: Port templatedata mocha tests to phpunit from Backlog to Porting on the Parsoid-PHP board.
Thu, Nov 7, 10:14 PM · Patch-For-Review, Parsoid-PHP
ssastry triaged T237693: Port templatedata mocha tests to phpunit as Normal priority.
Thu, Nov 7, 10:14 PM · Patch-For-Review, Parsoid-PHP
ssastry removed a subtask for T229015: Tracking: Direct live production traffic at Parsoid/PHP: T237556: Detect html2wt reqs issued to Parsoid/PHP with data-parsoid blobs generated by Parsoid/JS and issue a HTTP 421.
Thu, Nov 7, 9:39 PM · Core Platform Team, User-WDoran, Parsoid-PHP
ssastry removed a parent task for T237556: Detect html2wt reqs issued to Parsoid/PHP with data-parsoid blobs generated by Parsoid/JS and issue a HTTP 421: T229015: Tracking: Direct live production traffic at Parsoid/PHP.
Thu, Nov 7, 9:39 PM · Parsoid-PHP
ssastry added a comment to T234280: Edits made via VE on a translatable page removes untouched content in <translate> tags.

I think I see the exact problem here. In the Parsoid HTML (https://www.mediawiki.org/api/rest_v1/page/html/Developer_Advocacy), there are two elements with about="#mwt67" and different data-mw:
...
They are in different sections, so while weird, this doesn't immediately break things. But after unwrapping the sections (ve.unwrapParsoidSections(document.body, undefined)), these two elements become siblings, so the two about-groups get merged into one, and the data-mw from the second element is lost.
Is this a bug in Parsoid?

Thu, Nov 7, 9:24 PM · VisualEditor (Current work), VisualEditor-MediaWiki, Parsoid
ssastry added a comment to T234242: [betalabs-regression] Cannot add/update info on Structured discussion boards.

Likely caused by this: T231569#5502186 .. to resolve port conflicts between Parsoid/PHP and Parsoid/JS in the beta cluster. @Catrope found that Parsoid/JS is listening on 8001, not 8000 as Flow is configured.

Thu, Nov 7, 9:17 PM · Beta-Cluster-Infrastructure, Growth-Team, StructuredDiscussions