Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Invalid characters passed to "octdec" #690

Closed
6 tasks done
r-man opened this issue Nov 25, 2023 · 1 comment · Fixed by #691
Closed
6 tasks done

Invalid characters passed to "octdec" #690

r-man opened this issue Nov 25, 2023 · 1 comment · Fixed by #691
Assignees
Labels
Milestone

Comments

@r-man
Copy link

r-man commented Nov 25, 2023

  • PHP Depend version: 2.15.1
  • PHP Version: 8.2.8
  • Installation type: composer
  • Operating System / Distribution & Version: Ubuntu 23.10

Current Behavior

PHP Deprecated: Invalid characters passed for attempted conversion, these have been ignored in /var/www/html/vendor/pdepend/pdepend/src/main/php/PDepend/Source/Language/PHP/AbstractPHPParser.php on line 8219

Expected Behavior

No deprecation messages.

Steps To Reproduce:

  1. You need to enable E_DEPRECATED in the error_reporting directive.
  2. Create PaymentDTO.php
class PaymentDTO
{
    private float $amount = 0.00;
}
  1. Run
vendor/bin/pdepend --summary-xml=summary.xml PaymentDTO.php

Checks before submitting

  • Be sure that there isn't already an issue about this. See: Issues list
  • Be sure that there isn't already a pull request about this. See: Pull requests
  • I have added every step to reproduce the bug.
  • If possible I added relevant code examples.
  • This issue is about 1 bug and nothing more.
  • The issue has a descriptive title. For example: "Default value is not available in AST when it's an array".
@kylekatarnls kylekatarnls self-assigned this Nov 25, 2023
@kylekatarnls kylekatarnls added this to the 2.16.0 milestone Nov 25, 2023
@kylekatarnls
Copy link
Member

Bug confirmed, I should be able to provide a fix quickly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging a pull request may close this issue.

2 participants