Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Data Liberation] Add XML API, Stream API, WXR URL Rewriter API #1952

Merged
merged 36 commits into from
Oct 28, 2024
Merged

[Data Liberation] Add XML API, Stream API, WXR URL Rewriter API #1952

merged 36 commits into from
Oct 28, 2024

Conversation

adamziel
Copy link
Collaborator

@adamziel adamziel commented Oct 28, 2024 •
edited
Loading

A part of #1894. Follows up on #1893.

This PR brings in a few more PHP APIs that were initially explored outside of Playground so that they can be incubated in Playground. See the linked descriptions for more details about each API:

Testing instructions

  • Confirm the PHPUnit tests pass in CI
  • Confirm the test suite looks reasonabel
  • That's it for now! It's all new code that's not actually used anywhere in Playground yet. I just want to merge it to keep iterating and improving.

@adamziel
Data liberation: Kickoff the project
1ef710f
@adamziel
Port the URL rewriters from adamziel/site-transfer-protocol
234a8bf
@adamziel
Port WP_HTML_Processor et al. from WordPress
819febd
@adamziel
Move WordPress core files
0a6167b
@adamziel
Outline the next steps
826fe75
@adamziel
Add PHPCS and CBF
0633e6f
@adamziel
Update HTML API, fix unit tests
4406fcf
@adamziel
Merge branch 'trunk' into data-liberation-bring-in-php-parsers
0cfd334
@adamziel
Bump CI PHP version to 8.1
b90a9d6
@adamziel
Adjust the CI setup for PHP
081535b
@adamziel
Run npm instlal insteaf of installing just nx
aca88fe
@adamziel
Use the correct nx project name
897af50
@adamziel
Remove the network functions and only lint the src directory
f7679b0
@adamziel
Remove special casing for direct matching pathname prefixes
5b9ec7d
@adamziel
Fix linting errors
97fed71
@adamziel
Move the additional functions to pbpcbf.php
96c1ce4
@adamziel
Replace iterate_urls with url_matches
e15408a
@adamziel
Lint PHP
b788eea
@adamziel
Thoroughly test WP_URL_In_Text_Processor
b83933c
@adamziel
Enable tests for WP_Block_Markup_Processor
fb0204c
@adamziel
Enable all PHPUnit tests
b1ea8dc
@adamziel
Enable URLParserWHATWGComplianceTests
4335044
@adamziel
move $is_relative declaration clsoer to where it's used
91863ca
@adamziel
Add a single tricky test case for wp_rewrite_urls()
d2aeea4
@adamziel
Preserve urlencoded data in the rewritten path
60db1e1
@adamziel
Unit test urldecoding UTF-8 data
2da0386
@adamziel
Lint
54bea02
@adamziel
Remove messing with private WP_HTML_Tag_Processor attributes
54c901d
@adamziel
Remove the commented out dead code from WP_URL_In_Text_Processor
a62532b
@adamziel
Uncomment the public suffix list verification
238decd
Base automatically changed from data-liberation-bring-in-php-parsers to trunk October 28, 2024 23:14
@adamziel adamziel marked this pull request as ready for review October 28, 2024 23:29
@adamziel adamziel requested a review from a team as a code owner October 28, 2024 23:29
@adamziel adamziel mentioned this pull request Oct 28, 2024
Open
23 tasks
@adamziel adamziel merged commit 4ecf6cc into trunk Oct 28, 2024
10 checks passed
@adamziel adamziel deleted the data-liberation-xml-parsers branch October 28, 2024 23:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bgrgicak bgrgicak bgrgicak left review comments

Assignees
No one assigned
Labels
None yet
Projects
Playground Board
Status: Done
Milestone
Data Liberation: URL Rewriting
Development

Successfully merging this pull request may close these issues.
2 participants
@adamziel @bgrgicak