PHPStan failed with "No files found to analyse"

Created on 12 May 2021, over 3 years ago
Updated 3 October 2023, about 1 year ago

Problem/Motivation

After analyzing a custom module via the web UI i get this output:

Steps to reproduce

Installed the dev version via composer require 'drupal/upgrade_status:3.x-dev@dev'.
Installed core dev via composer require --dev drupal/core-dev:8.9.14.
Go to the admin page, check a custom module, run scan.

Some context on the installed packages:

  • drupal/core: 8.9.14
  • drupal/core-dev: 8.9.14
  • phpstan/phpstan: 0.12.86
  • mglaman/phpstan-drupal: 0.12.9
  • phpstan/phpstan-deprecation-rules: 0.12.6
  • phpunit/phpunit: 7.5.20

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

๐Ÿ› Bug report
Status

Active

Version

4.0

Component

Code

Created by

๐Ÿ‡ฉ๐Ÿ‡ชGermany bahuma20 Germany

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • ๐Ÿ‡ญ๐Ÿ‡ฐHong Kong kt2ssh

    I got the same problem and found the unexpected <?php tag from phpstan result.

  • Status changed to Active over 1 year ago
  • ๐Ÿ‡ซ๐Ÿ‡ทFrance fgm Paris, France

    It looks like a similar problem I just met: I have a custom theme that extends a base theme but does not include a single line of PHP, so when running an upgrade_status 4.0 scan, I get this error

    PHPStan command failed:
    /opt/homebrew/Cellar/php@8.1/8.1.21/bin/php /Users/fgm/src/OSInet/org/riff/blog/./vendor/bin/phpstan analyse --memory-limit=1500M --error-format=json --configuration=/tmp/upgrade_status/deprecation_testing.neon /Users/fgm/src/OSInet/org/riff/blog/web/themes/osinet/riffblogt
    
    Command output:
    Empty.
    
    Command error:
    ! [NOTE] No files found to analyse. [WARNING] This will cause a non-zero exit code in PHPStan 2.0.
    

    Adding an almost empty (theme).theme file fixed the issue for me.

  • ๐Ÿ‡ญ๐Ÿ‡บHungary Gรกbor Hojtsy Hungary

    That there was no file found to analyse sounds like a legitimate problem. At least it does not say everything was ok because there was nothing to check if it was ok. I guess for themes that could be a realistic scenario that no PHP file was found, but it would only be reasonable for modules very rarely. Should we have a special case for this situation? We would still need to somehow surface that there was no file scanned, but maybe a nicer error handler for this case?

  • ๐Ÿ‡ซ๐Ÿ‡ทFrance fgm Paris, France

    I think one could imagine a module only consisting of libraries, so with a mymodule.libraries.yml and a bunch of CSS / JS files pointed by those, so no PHP code either.

    That's probably not common but does not have to be an error. It's a bit similar to the "theme extends a base theme, only changing CSS / JS".

  • ๐Ÿ‡ญ๐Ÿ‡บHungary Gรกbor Hojtsy Hungary

    Let's focus this issue on the no files case then. The others don't seem to have more info on why they would happen.

  • ๐Ÿ‡ญ๐Ÿ‡บHungary Gรกbor Hojtsy Hungary

    However how we represent this situation is a question. That no file was found could be an error or it could be a legitimate thing to ignore. So we should IMHO report it, but maybe not as a raw error, but more like a warning level? It would still show it as one problem to resolve though, so those that want to make that go away would "need to" add an empty module file or somesuch for the error to go away. We can maybe discourage that in the message?

  • ๐Ÿ‡บ๐Ÿ‡ธUnited States tobby

    FWIW, I'm having the same issue. I've got a module with only my_module.info.yml and my_module.libraries.yml files, so I get the "No files found to analyse" warning. I created a temporary my_module.module file, and I was able to validate that the module is indeed D10-ready.

    I agree with Gรกbor -- maybe this "no file" issue can be a warning or even an info level. And place it in a "Compatible with next major Drupal core version (with caveats)" category instead of "Fix manually". My thinking is that a module with no executable code is technically compatible with Drupal 10...

  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia sahilgidwani Jaipur
    1. I am getting the error for PHP Stan command failed, as mentioned in the issue description, but not getting any error related to "No files found to analyse".
    2. When I debugged the analyze method of the DeprecationAnalyzer class, I found that the Process class is not returning output on line 373, and as a result, we are getting null in $json.
    3. Further, the condition check implemented for checking any JSON errors is returning true, and it is showing errors, although the PHP stan command is being executed successfully without any errors.

    Anyone can please look into this and implement a solution to resolve the issue mentioned above.

Production build 0.71.5 2024