Call to undefined method PhpParser\Node\Expr\Variable::getParts()
#610
Comments
|
Can you narrow it down to a certain vendor package? Because it's hard to detect which code structure causes this. |
<?php
/**
* This file is part of the Nette Framework (https://nette.org)
* Copyright (c) 2004 David Grudl (https://davidgrudl.com)
*/
declare(strict_types=1);
namespace Nette;
use Nette\Utils\ObjectHelpers;
/**
* Strict class for better experience.
* - 'did you mean' hints
* - access to undeclared members throws exceptions
* - support for @property annotations
* - support for calling event handlers stored in $onEvent via onEvent()
*/
trait SmartObject
{
/**
* @return mixed
* @throws MemberAccessException
*/
public function __call(string $name, array $args)
{
$class = static::class;
if (ObjectHelpers::hasProperty($class, $name) === 'event') { // calling event handlers
$handlers = $this->$name ?? null;
if (is_iterable($handlers)) {
foreach ($handlers as $handler) {
$handler(...$args);
}
} elseif ($handlers !== null) {
throw new UnexpectedValueException("Property $class::$$name must be iterable or null, " . get_debug_type($handlers) . ' given.');
}
return null;
}
ObjectHelpers::strictCall($class, $name);
}
/**
* @throws MemberAccessException
*/
public static function __callStatic(string $name, array $args)
{
ObjectHelpers::strictStaticCall(static::class, $name);
}
/**
* @return mixed
* @throws MemberAccessException if the property is not defined.
*/
public function &__get(string $name)
{
$class = static::class;
if ($prop = ObjectHelpers::getMagicProperties($class)[$name] ?? null) { // property getter
if (!($prop & 0b0001)) {
throw new MemberAccessException("Cannot read a write-only property $class::\$$name.");
}
$m = ($prop & 0b0010 ? 'get' : 'is') . ucfirst($name);
if ($prop & 0b10000) {
$trace = debug_backtrace(0, 1)[0]; // suppose this method is called from __call()
$loc = isset($trace['file'], $trace['line'])
? " in $trace[file] on line $trace[line]"
: '';
trigger_error("Property $class::\$$name is deprecated, use $class::$m() method$loc.", E_USER_DEPRECATED);
}
if ($prop & 0b0100) { // return by reference
return $this->$m();
} else {
$val = $this->$m();
return $val;
}
} else {
ObjectHelpers::strictGet($class, $name);
}
}
/**
* @throws MemberAccessException if the property is not defined or is read-only
*/
public function __set(string $name, mixed $value): void
{
$class = static::class;
if (ObjectHelpers::hasProperty($class, $name)) { // unsetted property
$this->$name = $value;
} elseif ($prop = ObjectHelpers::getMagicProperties($class)[$name] ?? null) { // property setter
if (!($prop & 0b1000)) {
throw new MemberAccessException("Cannot write to a read-only property $class::\$$name.");
}
$m = 'set' . ucfirst($name);
if ($prop & 0b10000) {
$trace = debug_backtrace(0, 1)[0]; // suppose this method is called from __call()
$loc = isset($trace['file'], $trace['line'])
? " in $trace[file] on line $trace[line]"
: '';
trigger_error("Property $class::\$$name is deprecated, use $class::$m() method$loc.", E_USER_DEPRECATED);
}
$this->$m($value);
} else {
ObjectHelpers::strictSet($class, $name);
}
}
/**
* @throws MemberAccessException
*/
public function __unset(string $name): void
{
$class = static::class;
if (!ObjectHelpers::hasProperty($class, $name)) {
throw new MemberAccessException("Cannot unset the property $class::\$$name.");
}
}
public function __isset(string $name): bool
{
return isset(ObjectHelpers::getMagicProperties(static::class)[$name]);
}
}L36 
|
|
Thanks. I can check. |
👍 My explain and how to fix was already replyed at composer-unused/symbol-parser#122 (comment) |
|
@llaville I didn't understand how to fix it in my projects from the comment. But I've found a way 😃 composer require "symfony/finder:^6.4"
composer require "composer-unused/symbol-parser:0.2.1"It would help to make composer-unused a scoped prefixed package, to drop these version conflicts compleetely. Like PHPStan, Rector and ECS do. |
|
@TomasVotruba Did you read my RFC at #630 ? |
|
@llaville Indeed, but didn't find solution there. |
|
@TomasVotruba Could you share your project (where you used |
|
@llaville Here I added the version locks: The |
|
@TomasVotruba Thanks for share This is what I've tried to explain by my comment (composer-unused/symbol-parser#122 (comment))
We should check that I've already applied this change on the new Goal was to unified SymbolCollector(s) and have only one : the BTW, A quick fix to solve the situation, with |
|
If you remove the https://github.com/rectorphp/swiss-knife/blob/main/composer.json#L20 constraint to use the latest version 0.2.2, you 'll get of course error (as previously explained) /shared/backups/github/swiss-knife $ vendor/bin/composer-unusedAnd with the patch/fix (I've provided), you'll get |
|
@icanhazstring |
|
@icanhazstring Thanks for new version https://github.com/composer-unused/symbol-parser/releases/tag/0.2.3, but the most important is missing to solve this issue. PS: this is the same issue as composer-unused/symbol-parser#122 |
|
Got it. If you would be so kind to just run a PR for that patch? Would gladly just release a new version 😉 |
|
Added to version 0.2.4 |
|
With this adventure, I've noticed that is was a bit difficult to know where the issue come from. I propose to add the package name processing on right side of the progress bar (when verbose mode is set to level 1 : That means for example, when trying to analyse https://github.com/rectorphp/swiss-knife/, with we can have this one : Agree for a PR ? |
|
Yes sounds like a good addition 👍 |
|
@TomasVotruba You can removed |
Describe the bug
I have encountered a crash
I run the unused with 112d25c
what other information should I acquire?
The text was updated successfully, but these errors were encountered: