Skip to content

Conversation

@Morgy93
Copy link
Collaborator

@Morgy93 Morgy93 commented Jan 30, 2026

This pull request consists of minor code quality and consistency improvements across several files, as well as a small fix to template escaping. The most significant change is the consistent use of the $escaper object for URL escaping in the frontend template to improve security and maintain best practices.

Frontend template security and consistency:

  • Updated src/view/frontend/templates/inspector.phtml to use the $escaper object instead of $block for URL escaping in both the stylesheet and script tags, ensuring proper output sanitization. [1] [2]

Code formatting and style improvements:

  • Fixed formatting in several PHP files by adding proper line breaks before docblocks and after method closures in CheckCommand.php, BuildCommand.php, and the NodePackageManager constructor for better readability and adherence to coding standards. [1] [2] [3] [4]

@Morgy93 Morgy93 requested a review from dermatz January 30, 2026 19:16
@Morgy93 Morgy93 enabled auto-merge (squash) January 30, 2026 19:16
@Morgy93 Morgy93 merged commit 3f9048a into main Jan 30, 2026
12 checks passed
@Morgy93 Morgy93 deleted the phpcbf-fixes branch January 30, 2026 20:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants