Skip to content

Move processor implementations to their public paths - #303

Merged
adamziel merged 1 commit into
trunkfrom
adamziel/restore-public-processor-class-names
Aug 7, 2026
Merged

Move processor implementations to their public paths#303
adamziel merged 1 commit into
trunkfrom
adamziel/restore-public-processor-class-names

Conversation

@adamziel

@adamziel adamziel commented Aug 7, 2026

Copy link
Copy Markdown
Collaborator

Moves the retained URL-in-text, HTML, and XML implementation files out of the PHP/ sidecar directories and into their public processor paths.

Files changed contains exactly four 100% renames with zero additions and zero deletions. No implementation byte changes in this PR.

Stack

  1. Vacate public processor paths for exact file moves #304 — vacate the destination paths
  2. Move processor implementations to their public paths #303 — move four byte-identical implementation files
  3. Restore public processor class declarations #305 — restore the public class declarations and remove temporary loaders

Testing

The Data Liberation, XML, and Markdown suites pass. PHPCS, syntax checks, direct-loader checks, and root-autoload checks pass.

@adamziel
adamziel force-pushed the adamziel/restore-public-processor-implementations branch 2 times, most recently from 140e205 to 43f22b5 Compare August 7, 2026 15:23
@adamziel
adamziel force-pushed the adamziel/restore-public-processor-class-names branch from 6f2ef88 to de42d76 Compare August 7, 2026 15:23
@adamziel adamziel closed this Aug 7, 2026
@adamziel
adamziel force-pushed the adamziel/restore-public-processor-class-names branch from de42d76 to 43f22b5 Compare August 7, 2026 15:56
@adamziel adamziel reopened this Aug 7, 2026
@adamziel adamziel changed the title Restore public processor class declarations Move processor implementations to their public paths Aug 7, 2026
@adamziel
adamziel changed the base branch from adamziel/restore-public-processor-implementations to adamziel/prepare-processor-file-moves August 7, 2026 15:56
adamziel added a commit that referenced this pull request Aug 7, 2026
Moves the four public alias loaders into temporary `loaders/`
directories so #303 can present the implementation relocations as exact
Git renames.

The temporary loaders accept either adjacent stack state: the
implementations under `PHP/` in this PR, or the public paths after #303.
The implementation files receive only a PHPCS directive that moves with
their unchanged bytes. #305 removes the loaders and directives while
restoring the public class declarations.

## Stack

1. #304 — vacate the destination paths
2. #303 — move four byte-identical implementation files
3. #305 — restore the public class declarations and remove temporary
loaders

After each base PR merges, retarget the next PR to `trunk`.

## Testing

The Data Liberation, XML, and Markdown suites pass. PHPCS, syntax
checks, direct-loader checks, and root-autoload checks pass.
Base automatically changed from adamziel/prepare-processor-file-moves to trunk August 7, 2026 16:02
@adamziel
adamziel force-pushed the adamziel/restore-public-processor-class-names branch from 43f22b5 to 95a86a9 Compare August 7, 2026 16:03
@adamziel
adamziel merged commit 38adac0 into trunk Aug 7, 2026
81 of 83 checks passed
@adamziel
adamziel deleted the adamziel/restore-public-processor-class-names branch August 7, 2026 16:41
adamziel added a commit that referenced this pull request Aug 7, 2026
Renames the four relocated `PHP*` implementation classes to their public
names and removes the temporary loaders introduced by #304.

With #303 merged, Files changed contains only the class declaration
cleanup and temporary-loader removal. The resulting tree contains
neither the retired `PHP*` class names nor the `PHP/` and `loaders/`
directories.

## Stack

1. #304 — vacate the destination paths
2. #303 — move four byte-identical implementation files
3. #305 — restore the public class declarations and remove temporary
loaders

## Testing

The Data Liberation, XML, and Markdown suites pass. PHPCS and syntax
checks pass. A root-autoload check confirms the public classes load, the
retired `PHP*` classes do not, and `WP_HTML_Processor` remains a
`WP_HTML_Tag_Processor` subclass.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant