Skip to content

[BUGFIX] Refactored unwrapping of USER_INT#869

Merged
lukaszuznanski merged 1 commit intomasterfrom
task/improve-headless-wrap
Mar 25, 2026
Merged

[BUGFIX] Refactored unwrapping of USER_INT#869
lukaszuznanski merged 1 commit intomasterfrom
task/improve-headless-wrap

Conversation

@twoldanski
Copy link
Copy Markdown
Collaborator

  • simplified handling
  • improved regex to be faster & more reliable trying not hitting limits and return nulls
  • add fallback if that happens
  • leverage new PHP power if ext:headless running on newer PHP version

@coveralls
Copy link
Copy Markdown

coveralls commented Feb 20, 2026

Pull Request Test Coverage Report for Build 23532981460

Details

  • 32 of 36 (88.89%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.2%) to 73.04%

Changes Missing Coverage Covered Lines Changed/Added Lines %
Classes/Utility/HeadlessUserInt.php 32 36 88.89%
Totals Coverage Status
Change from base Build 23532947375: -0.2%
Covered Lines: 1146
Relevant Lines: 1569

💛 - Coveralls

- simplified handling
- improved regex to be faster & more reliable trying not hitting limits and return nulls
- add fallback if that happens
- leverage new PHP power if ext:headless running on newer PHP version
@lukaszuznanski lukaszuznanski force-pushed the task/improve-headless-wrap branch from 601209b to a8eed12 Compare March 25, 2026 09:01
@lukaszuznanski lukaszuznanski merged commit c36c6b6 into master Mar 25, 2026
14 checks passed
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.

3 participants