Skip to content

Conversation

@samsonasik
Copy link
Member

@samsonasik samsonasik commented Oct 11, 2025

@TomasVotruba this is test for reproduce issue

btw, I use fork for this repo and I don't have write permission to this repo.

@TomasVotruba
Copy link
Member

btw, I use fork for this repo and I don't have write permission to this repo.

Fixed 👍

<?php

include __DIR__ . '/../../../../vendor/rector/rector/preload.php';
include __DIR__ . '/../../../../vendor/autoload.php';
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This structure should never be used manually. Rector autoload should not be exposed.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is just the way I can imagine it reproduced, but actual applications may use include over include_once for autoload somewhere that it automatically make preload.php included more than once. Some magic framework things

@TomasVotruba
Copy link
Member

Let's wait for the reproducer from the issue reporter.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants