Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
Marc Würth
@ravage84
@retanoj no, there is no such feature, yet
Michał Kalinowski
@kalinowski5
Hi!
Are there any plans for new release/tag?
It would be nice to have recent features (like baselines) in PHIVE.
Kyle
@kylekatarnls
Tobias van Beek
@tvbeek
@kalinowski5 So today is the day :)
Mark Vaughn
@MarkVaughn
Any chance we could tag 2.10.1 with the xdebug handler support. It also conflicts with https://github.com/phan/phan v4
Marc Würth
@ravage84
@MarkVaughn is there a reason we couldn't?
Mark Vaughn
@MarkVaughn
I don't think so, I just never commandeered a release for the project
Tobias van Beek
@tvbeek
If you can create a pull request for the changelog and the build.properties (like phpmd/phpmd#880 ) I can release a new version this week.
Marc Würth
@ravage84
@MarkVaughn go forward, please. Let's use that opportunity to take your release virginity :laughing:
Mark Vaughn
@MarkVaughn
Tobias van Beek
@tvbeek
👍congratulations with your first release.
Anders Jenbo
@AJenbo

I made a proposal for how we could add support for making PHPMD a viable tool for verifying ISO-25010 compliance: phpmd/phpmd#895

Please let me know if you have any feedback or suggestions.

Jerry
@DevJ3rry_gitlab
hey hey, any news on php8 compatibility ?
Tobias van Beek
@tvbeek
@DevJ3rry_gitlab you can already use PHPMD on PHP 8 but there are some open issues, see: https://github.com/phpmd/phpmd/labels/php8.x they will be handled if someone has time to do that. But feel free to create a PR to fix the issues :smile:
Natan Felles
@natanfelles
Hello! Set maximum value of ExcessiveClassComplexity rule is not working in my project.
It is my config: <rule ref="rulesets/codesize.xml/ExcessiveClassComplexity">
<properties>
<property name="maximum" value="80"/>
</properties>
</rule>
But always returns the message: The class Session has an overall complexity of 56 which is very high. The configured complexity threshold is 50.
I have other custom rules configured and all are working.
I'm running PHP 8.0.7
Natan Felles
@natanfelles
Do you have any tips or can I open an issue?
Tobias van Beek
@tvbeek
@natanfelles Do you also have the codesize.xml in general in your ruleset? I can suspect that can create special options. Else it should be nice if you could create an issue with your ruleset added.
Natan Felles
@natanfelles
Oh, bingo! A had a general codesize.xml: <rule ref="rulesets/codesize.xml"/>
Removing it solved the problem. Very thanks @tvbeek ! :D
Tobias van Beek
@tvbeek
@natanfelles No problem, I'm happy that I could help :)
pramohs
@pramohs:matrix.org
[m]
Invalid field modifiers given, allowed modifiers are IS_PUBLIC, IS_PROTECTED, IS_PRIVATE and IS_STATIC while trying to run a phpmd
Kyle
@kylekatarnls
Hello @pramohs:matrix.org github.com/phpmd/phpmd/issues will be a better fit to investigate this properly, we'll need the command a minimum code chunk that triggers the error and the stack trace of the error to investigate.
Federico Moyá
@fede-moya:matrix.org
[m]
Hello @MarkVaughn , federico from CodeClimate here. In answer to your request I have just released an update of the phpmd plugin. It would be great if you could test it and report if anything is not working as it should. https://docs.codeclimate.com/docs/phpmd Regards.
Marc Würth
@ravage84
@fede-moya:matrix.org thank you for your effort and reaching out on us.
It was me, who sent the request.
How can we make sure, it stays up to date? We more or less regularly update PHPMD (& the underlying PDepend).
Marc Würth
@ravage84
@fede-moya:matrix.org teh PHPMD plugin seems to work. Thanks!
Federico Moyá
@fede-moya:matrix.org
[m]
@ravage84 Great news I'm happy to hear that. So in order to keep it updated the best would be for you folks to open prs with the updates. https://github.com/codeclimate/codeclimate-phpmd I can commit to review those PRs and merged them and I can also commit to complete those reviews relatively fast. Sounds good ?
porokhovnichenko
@porokhovnichenko
Hello everyone, feel so happy to find this community! I hope to find an answer to my question here - when constructor promotion property is used and it's body is empty, something like: public function __construct( private int $id) {} I get a message "Avoid unused parameter such as '$id'". Is it expected behavior for PHP 8.0.10 and phpmd 2.10.2? I am confused with this a bit, I didn't expect to see the message.
Kyle
@kylekatarnls
Hello, is it phpmd 2.10.2 from phar or from composer? If composer, can you also check the pdepend version?
porokhovnichenko
@porokhovnichenko
@kylekatarnls Hello, 2.10.2 from composer, pdepend version is 2.10.0.
Tobias van Beek
@tvbeek
@porokhovnichenko I suspect it is the same problem as: phpmd/phpmd#884 can you verify it with the dev-master composer version?
porokhovnichenko
@porokhovnichenko
@tvbeek Thanks for your interest, yes, it looks the same. And with dev-master I still get the same message about unused parameter.
porokhovnichenko
@porokhovnichenko
@tvbeek But if I install pdepend from composer with dev-master version it is not reproduced any more
Kyle
@kylekatarnls
Yes, sorry, both are needed, PDepend will expose a util to tell if a parameter is promoted, and PHPMD use it to whitelist in the unused-parameter rule.
We'll release PDepend, then a new PHPMD version that will take the last PDepend one too.
porokhovnichenko
@porokhovnichenko
Thanks a lot to all of you!
Wonzbak
@wonzbak
Hello
I've a warning at the class level: The class MyClass has an overall complexity of ....
Is there a way to deactive only the warning of the class, but not on the method of a class
Wonzbak
@wonzbak
?
Tobias van Beek
@tvbeek
@wonzbak you can suppress a specific rule, see: https://phpmd.org/documentation/suppress-warnings.html
I think that will be enough for your case (or change the code 😉)
Wonzbak
@wonzbak
Yes I would like change the code, but suppress warning is always a question of time,
I finaly add the warning: @SuppressWarnings(PHPMD.ExcessiveClassComplexity) at the class level, instead of the wrong one CyclomaticComplexity
thx for the response