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

Errors during renaming #2518

Open
leo-unglaub opened this issue Jan 23, 2024 · 17 comments · Fixed by #2559
Open

Errors during renaming #2518

leo-unglaub opened this issue Jan 23, 2024 · 17 comments · Fixed by #2559
Labels

Comments

@leo-unglaub
Copy link

Hey,
since a few days i get a lot of errors while renaming some classes. I am on the latest version from git (HEAD) and i am using VS Code as an editor.

[Error - 15:14:00] Request textDocument/codeAction failed.
  Message: Exception [Phpactor\LanguageServer\Core\Workspace\Exception\UnknownDocument] Unknown text document "file:///home/leo/code/website-laravel/app/Models/User.php" at /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Core/Workspace/Workspace.php#45
  Code: -32603

Here is the full stacktrace. I also tried deleting everything in ~/.cache/phpactor and reindexing the project, still the same error.
Greetings
Leo

[Error - 15:14:00] Request textDocument/codeAction failed.
  Message: Exception [Phpactor\LanguageServer\Core\Workspace\Exception\UnknownDocument] Unknown text document "file:///home/leo/code/website-laravel/app/Models/User.php" at /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Core/Workspace/Workspace.php#45
  Code: -32603 
#0 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Handler/TextDocument/CodeActionHandler.php(54): Phpactor\LanguageServer\Core\Workspace\Workspace->get()
#1 [internal function]: Phpactor\LanguageServer\Handler\TextDocument\CodeActionHandler->Phpactor\LanguageServer\Handler\TextDocument\{closure}()
#2 /home/leo/code/github.com/phpactor/phpactor/vendor/amphp/amp/lib/Coroutine.php(118): Generator->send()
#3 /home/leo/code/github.com/phpactor/phpactor/vendor/amphp/amp/lib/Internal/Placeholder.php(149): Amp\Coroutine->Amp\{closure}()
#4 /home/leo/code/github.com/phpactor/phpactor/vendor/amphp/amp/lib/Coroutine.php(123): Amp\Coroutine->resolve()
#5 /home/leo/code/github.com/phpactor/phpactor/vendor/amphp/amp/lib/Internal/Placeholder.php(149): Amp\Coroutine->Amp\{closure}()
#6 /home/leo/code/github.com/phpactor/phpactor/vendor/amphp/amp/lib/Deferred.php(53): Amp\Promise@anonymous->resolve()
#7 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Core/Server/ResponseWatcher/DeferredResponseWatcher.php(21): Amp\Deferred->resolve()
#8 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Middleware/ResponseHandlingMiddleware.php(31): Phpactor\LanguageServer\Core\Server\ResponseWatcher\DeferredResponseWatcher->handle()
#9 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Core/Middleware/RequestHandler.php(37): Phpactor\LanguageServer\Middleware\ResponseHandlingMiddleware->process()
#10 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Middleware/MethodAliasMiddleware.php(30): Phpactor\LanguageServer\Core\Middleware\RequestHandler->handle()
#11 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Core/Middleware/RequestHandler.php(37): Phpactor\LanguageServer\Middleware\MethodAliasMiddleware->process()
#12 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Middleware/CancellationMiddleware.php(49): Phpactor\LanguageServer\Core\Middleware\RequestHandler->handle()
#13 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Core/Middleware/RequestHandler.php(37): Phpactor\LanguageServer\Middleware\CancellationMiddleware->process()
#14 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Middleware/ShutdownMiddleware.php(68): Phpactor\LanguageServer\Core\Middleware\RequestHandler->handle()
#15 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Core/Middleware/RequestHandler.php(37): Phpactor\LanguageServer\Middleware\ShutdownMiddleware->process()
#16 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Middleware/InitializeMiddleware.php(52): Phpactor\LanguageServer\Core\Middleware\RequestHandler->handle()
#17 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Core/Middleware/RequestHandler.php(37): Phpactor\LanguageServer\Middleware\InitializeMiddleware->process()
#18 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Middleware/ErrorHandlingMiddleware.php(34): Phpactor\LanguageServer\Core\Middleware\RequestHandler->handle()
#19 [internal function]: Phpactor\LanguageServer\Middleware\ErrorHandlingMiddleware->Phpactor\LanguageServer\Middleware\{closure}()
#20 /home/leo/code/github.com/phpactor/phpactor/vendor/amphp/amp/lib/Coroutine.php(67): Generator->current()
#21 /home/leo/code/github.com/phpactor/phpactor/vendor/amphp/amp/lib/functions.php(96): Amp\Coroutine->__construct()
#22 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Middleware/ErrorHandlingMiddleware.php(32): Amp\call()
#23 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Core/Middleware/RequestHandler.php(37): Phpactor\LanguageServer\Middleware\ErrorHandlingMiddleware->process()
#24 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Core/Dispatcher/Dispatcher/MiddlewareDispatcher.php(29): Phpactor\LanguageServer\Core\Middleware\RequestHandler->handle()
#25 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Core/Server/LanguageServer.php(223): Phpactor\LanguageServer\Core\Dispatcher\Dispatcher\MiddlewareDispatcher->dispatch()
#26 [internal function]: Phpactor\LanguageServer\Core\Server\LanguageServer->Phpactor\LanguageServer\Core\Server\{closure}()
#27 /home/leo/code/github.com/phpactor/phpactor/vendor/amphp/amp/lib/Coroutine.php(67): Generator->current()
#28 /home/leo/code/github.com/phpactor/phpactor/vendor/amphp/amp/lib/functions.php(96): Amp\Coroutine->__construct()
#29 /home/leo/code/github.com/phpactor/phpactor/vendor/amphp/amp/lib/functions.php(121): Amp\call()
#30 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Core/Server/LanguageServer.php(221): Amp\asyncCall()
#31 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Core/Server/LanguageServer.php(214): Phpactor\LanguageServer\Core\Server\LanguageServer->dispatchRequest()
#32 [internal function]: Phpactor\LanguageServer\Core\Server\LanguageServer->Phpactor\LanguageServer\Core\Server\{closure}()
#33 /home/leo/code/github.com/phpactor/phpactor/vendor/amphp/amp/lib/Coroutine.php(118): Generator->send()
#34 /home/leo/code/github.com/phpactor/phpactor/vendor/amphp/amp/lib/Internal/Placeholder.php(149): Amp\Coroutine->Amp\{closure}()
#35 /home/leo/code/github.com/phpactor/phpactor/vendor/amphp/amp/lib/Coroutine.php(123): Amp\Coroutine->resolve()
#36 /home/leo/code/github.com/phpactor/phpactor/vendor/amphp/amp/lib/Internal/Placeholder.php(149): Amp\Coroutine->Amp\{closure}()
#37 /home/leo/code/github.com/phpactor/phpactor/vendor/amphp/amp/lib/Deferred.php(53): Amp\Promise@anonymous->resolve()
#38 /home/leo/code/github.com/phpactor/phpactor/vendor/amphp/byte-stream/lib/ResourceInputStream.php(109): Amp\Deferred->resolve()
#39 /home/leo/code/github.com/phpactor/phpactor/vendor/amphp/amp/lib/Loop/Driver.php(119): Amp\ByteStream\ResourceInputStream::Amp\ByteStream\{closure}()
#40 /home/leo/code/github.com/phpactor/phpactor/vendor/amphp/amp/lib/Loop/Driver.php(72): Amp\Loop\Driver->tick()
#41 /home/leo/code/github.com/phpactor/phpactor/vendor/amphp/amp/lib/Loop.php(95): Amp\Loop\Driver->run()
#42 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Core/Server/LanguageServer.php(111): Amp\Loop::run()
#43 /home/leo/code/github.com/phpactor/phpactor/lib/Extension/LanguageServer/Command/StartCommand.php(50): Phpactor\LanguageServer\Core\Server\LanguageServer->run()
#44 /home/leo/code/github.com/phpactor/phpactor/vendor/symfony/console/Command/Command.php(298): Phpactor\Extension\LanguageServer\Command\StartCommand->execute()
#45 /home/leo/code/github.com/phpactor/phpactor/vendor/symfony/console/Application.php(1040): Symfony\Component\Console\Command\Command->run()
#46 /home/leo/code/github.com/phpactor/phpactor/vendor/symfony/console/Application.php(301): Symfony\Component\Console\Application->doRunCommand()
#47 /home/leo/code/github.com/phpactor/phpactor/lib/Application.php(48): Symfony\Component\Console\Application->doRun()
#48 /home/leo/code/github.com/phpactor/phpactor/vendor/symfony/console/Application.php(171): Phpactor\Application->doRun()
#49 /home/leo/code/github.com/phpactor/phpactor/bin/phpactor(46): Symfony\Component\Console\Application->run()
#50 {main}
@przepompownia
Copy link
Contributor

I haven't looked closely yet - I am also familiar with this or a similar error, but I get it only between subsequent renaming actions. In most of my cases reindexing allows me to rename file again without error.

@dantleech
Copy link
Collaborator

dantleech commented Jan 26, 2024

yes probably an issue with stale index, which should be a hot topic as it's something I run into multiple times a day (although in a different context) :) i'll try and investigate.

@dantleech dantleech added the bug label Jan 26, 2024
@przepompownia
Copy link
Contributor

przepompownia commented Feb 10, 2024

@dantleech is it possible that the index will then become stale due to lack of notification from the client for both the old and new (renamed) files?

@dantleech
Copy link
Collaborator

dantleech commented Feb 21, 2024

yes that is possible. Phpactor will use a number of strategies to detect file changes, CoC will actually send the notifications to Phpactor via. LSP notifications, neovim lsp does not, if that's not possible it will try inotify-watch and watchman, falling back to find at regular intervals. On top of that, just to be safe Phpactor will rescan the project for changed files and index them every 5 minutes.

But probably we can reacticely remove non-existing files from the index when such errors occur.

@przepompownia
Copy link
Contributor

I still haven't answered myself which LSP notification about deleted file should be enough to detect the deletion on Phpactor side (or how to verify if any watcher implementation reported such event).

@przepompownia
Copy link
Contributor

neovim/neovim#24975 - on the Neovim side we probably also have some missing notification to send after renaming.

@dantleech
Copy link
Collaborator

dantleech commented Feb 24, 2024

Hopefully fixed in #2559 re-open if not.

dantleech added a commit that referenced this issue Feb 24, 2024
* GH-2518: Do not crash if reference does not exist

* Fix div be zero
@leo-unglaub
Copy link
Author

So far it works perfectly. I took a huge codebase and renamed as much as i could find in as little time as possible. No errors, it worked perfect so far :)

Big thanks for tracking this down!

@leo-unglaub
Copy link
Author

leo-unglaub commented Feb 25, 2024

Damn, i was to fast. I managed to trigger an error during renaming again. I used an Enum with 30+ cases in it and renamed them, the first 20 or so worked fine, but then i got an error:

[Error - 05:27:47] Request textDocument/rename failed.
  Message: Exception [Error] Call to a member function getTraceAsString() on null at /home/leo/code/github.com/phpactor/phpactor/lib/Extension/LanguageServerRename/Handler/RenameHandler.php#81
  Code: -32603 
#0 [internal function]: Phpactor\Extension\LanguageServerRename\Handler\RenameHandler->Phpactor\Extension\LanguageServerRename\Handler\{closure}()
#1 /home/leo/code/github.com/phpactor/phpactor/vendor/amphp/amp/lib/Coroutine.php(67): Generator->current()
#2 /home/leo/code/github.com/phpactor/phpactor/vendor/amphp/amp/lib/functions.php(96): Amp\Coroutine->__construct()
#3 /home/leo/code/github.com/phpactor/phpactor/lib/Extension/LanguageServerRename/Handler/RenameHandler.php(57): Amp\call()
#4 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Core/Handler/HandlerMethodRunner.php(86): Phpactor\Extension\LanguageServerRename\Handler\RenameHandler->rename()
#5 [internal function]: Phpactor\LanguageServer\Core\Handler\HandlerMethodRunner->Phpactor\LanguageServer\Core\Handler\{closure}()
#6 /home/leo/code/github.com/phpactor/phpactor/vendor/amphp/amp/lib/Coroutine.php(67): Generator->current()
#7 /home/leo/code/github.com/phpactor/phpactor/vendor/amphp/amp/lib/functions.php(96): Amp\Coroutine->__construct()
#8 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Core/Handler/HandlerMethodRunner.php(72): Amp\call()
#9 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Middleware/HandlerMiddleware.php(38): Phpactor\LanguageServer\Core\Handler\HandlerMethodRunner->dispatch()
#10 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Core/Middleware/RequestHandler.php(37): Phpactor\LanguageServer\Middleware\HandlerMiddleware->process()
#11 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Middleware/ResponseHandlingMiddleware.php(35): Phpactor\LanguageServer\Core\Middleware\RequestHandler->handle()
#12 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Core/Middleware/RequestHandler.php(37): Phpactor\LanguageServer\Middleware\ResponseHandlingMiddleware->process()
#13 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Middleware/MethodAliasMiddleware.php(37): Phpactor\LanguageServer\Core\Middleware\RequestHandler->handle()
#14 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Core/Middleware/RequestHandler.php(37): Phpactor\LanguageServer\Middleware\MethodAliasMiddleware->process()
#15 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Middleware/CancellationMiddleware.php(49): Phpactor\LanguageServer\Core\Middleware\RequestHandler->handle()
#16 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Core/Middleware/RequestHandler.php(37): Phpactor\LanguageServer\Middleware\CancellationMiddleware->process()
#17 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Middleware/ShutdownMiddleware.php(68): Phpactor\LanguageServer\Core\Middleware\RequestHandler->handle()
#18 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Core/Middleware/RequestHandler.php(37): Phpactor\LanguageServer\Middleware\ShutdownMiddleware->process()
#19 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Middleware/InitializeMiddleware.php(56): Phpactor\LanguageServer\Core\Middleware\RequestHandler->handle()
#20 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Core/Middleware/RequestHandler.php(37): Phpactor\LanguageServer\Middleware\InitializeMiddleware->process()
#21 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Middleware/ErrorHandlingMiddleware.php(34): Phpactor\LanguageServer\Core\Middleware\RequestHandler->handle()
#22 [internal function]: Phpactor\LanguageServer\Middleware\ErrorHandlingMiddleware->Phpactor\LanguageServer\Middleware\{closure}()
#23 /home/leo/code/github.com/phpactor/phpactor/vendor/amphp/amp/lib/Coroutine.php(67): Generator->current()
#24 /home/leo/code/github.com/phpactor/phpactor/vendor/amphp/amp/lib/functions.php(96): Amp\Coroutine->__construct()
#25 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Middleware/ErrorHandlingMiddleware.php(32): Amp\call()
#26 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Core/Middleware/RequestHandler.php(37): Phpactor\LanguageServer\Middleware\ErrorHandlingMiddleware->process()
#27 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Core/Dispatcher/Dispatcher/MiddlewareDispatcher.php(29): Phpactor\LanguageServer\Core\Middleware\RequestHandler->handle()
#28 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Core/Server/LanguageServer.php(223): Phpactor\LanguageServer\Core\Dispatcher\Dispatcher\MiddlewareDispatcher->dispatch()
#29 [internal function]: Phpactor\LanguageServer\Core\Server\LanguageServer->Phpactor\LanguageServer\Core\Server\{closure}()
#30 /home/leo/code/github.com/phpactor/phpactor/vendor/amphp/amp/lib/Coroutine.php(67): Generator->current()
#31 /home/leo/code/github.com/phpactor/phpactor/vendor/amphp/amp/lib/functions.php(96): Amp\Coroutine->__construct()
#32 /home/leo/code/github.com/phpactor/phpactor/vendor/amphp/amp/lib/functions.php(121): Amp\call()
#33 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Core/Server/LanguageServer.php(221): Amp\asyncCall()
#34 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Core/Server/LanguageServer.php(214): Phpactor\LanguageServer\Core\Server\LanguageServer->dispatchRequest()
#35 [internal function]: Phpactor\LanguageServer\Core\Server\LanguageServer->Phpactor\LanguageServer\Core\Server\{closure}()
#36 /home/leo/code/github.com/phpactor/phpactor/vendor/amphp/amp/lib/Coroutine.php(118): Generator->send()
#37 /home/leo/code/github.com/phpactor/phpactor/vendor/amphp/amp/lib/Internal/Placeholder.php(149): Amp\Coroutine->Amp\{closure}()
#38 /home/leo/code/github.com/phpactor/phpactor/vendor/amphp/amp/lib/Coroutine.php(123): Amp\Coroutine->resolve()
#39 /home/leo/code/github.com/phpactor/phpactor/vendor/amphp/amp/lib/Internal/Placeholder.php(149): Amp\Coroutine->Amp\{closure}()
#40 /home/leo/code/github.com/phpactor/phpactor/vendor/amphp/amp/lib/Deferred.php(53): Amp\Promise@anonymous->resolve()
#41 /home/leo/code/github.com/phpactor/phpactor/vendor/amphp/byte-stream/lib/ResourceInputStream.php(101): Amp\Deferred->resolve()
#42 /home/leo/code/github.com/phpactor/phpactor/vendor/amphp/amp/lib/Loop/NativeDriver.php(327): Amp\ByteStream\ResourceInputStream::Amp\ByteStream\{closure}()
#43 /home/leo/code/github.com/phpactor/phpactor/vendor/amphp/amp/lib/Loop/NativeDriver.php(124): Amp\Loop\NativeDriver->selectStreams()
#44 /home/leo/code/github.com/phpactor/phpactor/vendor/amphp/amp/lib/Loop/Driver.php(138): Amp\Loop\NativeDriver->dispatch()
#45 /home/leo/code/github.com/phpactor/phpactor/vendor/amphp/amp/lib/Loop/Driver.php(72): Amp\Loop\Driver->tick()
#46 /home/leo/code/github.com/phpactor/phpactor/vendor/amphp/amp/lib/Loop.php(95): Amp\Loop\Driver->run()
#47 /home/leo/code/github.com/phpactor/phpactor/vendor/phpactor/language-server/lib/Core/Server/LanguageServer.php(111): Amp\Loop::run()
#48 /home/leo/code/github.com/phpactor/phpactor/lib/Extension/LanguageServer/Command/StartCommand.php(50): Phpactor\LanguageServer\Core\Server\LanguageServer->run()
#49 /home/leo/code/github.com/phpactor/phpactor/vendor/symfony/console/Command/Command.php(298): Phpactor\Extension\LanguageServer\Command\StartCommand->execute()
#50 /home/leo/code/github.com/phpactor/phpactor/vendor/symfony/console/Application.php(1040): Symfony\Component\Console\Command\Command->run()
#51 /home/leo/code/github.com/phpactor/phpactor/vendor/symfony/console/Application.php(301): Symfony\Component\Console\Application->doRunCommand()
#52 /home/leo/code/github.com/phpactor/phpactor/lib/Application.php(48): Symfony\Component\Console\Application->doRun()
#53 /home/leo/code/github.com/phpactor/phpactor/vendor/symfony/console/Application.php(171): Phpactor\Application->doRun()
#54 /home/leo/code/github.com/phpactor/phpactor/bin/phpactor(46): Symfony\Component\Console\Application->run()
#55 {main}

I am on f4a79b0. I do not have permissions to reopen the issue.

@dantleech
Copy link
Collaborator

I mean it is a different error...

@dantleech dantleech reopened this Feb 25, 2024
@dantleech
Copy link
Collaborator

it was trying to show an error, but the error handling was broken - I've updated master and it should at least show the error in the IDE now.

@przepompownia
Copy link
Contributor

przepompownia commented Feb 25, 2024

By coincidence, I encountered the same place in #2531 both in RenameHandler and FileRenameHandler (edit: false - I used error handling from RenameHandler 😁 ).
`
https://github.com/phpactor/phpactor/pull/2531/files#diff-ae4533ad2220106dce4b7d9a7651096361723c94c0e58b2d0c29027eb20c2386R81

@leo-unglaub
Copy link
Author

I managed to get the full error message:

Could not find corresponding reference to member name "DatabaseUserUpdated" in document "file:///<home>/website/app/Models/TaskTypes.php" at offset 576

I also tried to run phpactor with xdebug attached to it so i could give you better debugging information. But the problem is that for me xdebug does not trigger on PHP AMP forced processed. For some reason its only active on the main thread and then gets disabled. And i cannot enable it globally, because then i get a lot of errors from the code i am working in not beeing found.

@przepompownia
Copy link
Contributor

Searching by the message string takes me here:

protected function renameEdit(Location $location, ?ByteOffsetRange $range, string $originalName, string $newName): LocatedTextEdit
{
$referenceDocument = $this->locator->get($location->uri());
$range = $this->getRenameRange($referenceDocument, $location->range()->start());
if (null === $range) {
throw new CouldNotRename(sprintf(
'Could not find corresponding reference to member name "%s" in document "%s" at offset %s',
$originalName,
$referenceDocument->uri()->__toString(),
$location->range()->start()->toInt()
));
}

@przepompownia
Copy link
Contributor

przepompownia commented Apr 17, 2024

@leo-unglaub please check it again after merging #2627. Did this class member (or reference to it) have any attribute?

@leo-unglaub
Copy link
Author

I am not sure what project this was in, but i did a lot of renaming with #2627 and did not see this error. So maybe/hopefully your patch fixed this :) I will keep an eye on it and report back. Thanks so much :)

@przepompownia
Copy link
Contributor

Feedback also very appreciated, thank you too.

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

Successfully merging a pull request may close this issue.

3 participants