Skip to content

Restore public processor class declarations - #305

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

Restore public processor class declarations#305
adamziel merged 1 commit into
trunkfrom
adamziel/restore-public-processor-class-declarations

Conversation

@adamziel

@adamziel adamziel commented Aug 7, 2026

Copy link
Copy Markdown
Collaborator

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. 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 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.

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.
@adamziel
adamziel force-pushed the adamziel/restore-public-processor-class-names branch from 43f22b5 to 95a86a9 Compare August 7, 2026 16:03
adamziel added a commit that referenced this pull request Aug 7, 2026
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. #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, syntax
checks, direct-loader checks, and root-autoload checks pass.
Base automatically changed from adamziel/restore-public-processor-class-names to trunk August 7, 2026 16:41
@adamziel
adamziel force-pushed the adamziel/restore-public-processor-class-declarations branch from de42d76 to 5ebb420 Compare August 7, 2026 16:42
@adamziel
adamziel merged commit 9ec1e48 into trunk Aug 7, 2026
81 of 83 checks passed
@adamziel
adamziel deleted the adamziel/restore-public-processor-class-declarations branch August 7, 2026 17:21
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