Skip to content

Conversation

@iMattPro
Copy link
Member

@iMattPro iMattPro commented Dec 11, 2025

PHPBB-17591

Checklist:

  • Correct branch: master for new features; 3.3.x for fixes
  • Tests pass
  • Code follows coding guidelines: master and 3.3.x
  • Commit follows commit message format

Tracker ticket:

https://tracker.phpbb.com/browse/PHPBB-17591

@private-packagist
Copy link

phpBB/composer.lock

Dev Package changes

Package Operation From To About
squizlabs/php_codesniffer upgrade 3.13.5 4.0.1 diff

Settings · Docs · Powered by Private Packagist

@iMattPro
Copy link
Member Author

Tests do pass. Just CI-BS™ again.

@LukeWCS
Copy link
Contributor

LukeWCS commented Dec 11, 2025

Tested with EC 1.11.0-dev, CS 4.0.1, and PPSSE 4.x bc680bc.

A developer provided me with an extension that had already been adapted for phpBB 4.0.0-a1. I used this extension to run the test with Ext Check, as this test suite automatically switches between PPSSE 3.x and 4.x. This allowed me to conduct a real-world test.

Without the fix, PPSSE 4.x reported 14 false positives. With #6908, no false positives were reported. 👍

@marc1706 marc1706 added this to the 4.0.0-a2 milestone Dec 24, 2025
@marc1706 marc1706 merged commit 9530445 into phpbb:master Dec 24, 2025
35 checks passed
@iMattPro iMattPro deleted the ticket/17591-master branch December 24, 2025 20:17
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