-
Notifications
You must be signed in to change notification settings - Fork 22
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
Circular reference detected for service "import_app_simple.simple" #170
Comments
Hi @David-Magebinary I'll check this issue and keep you updated. Thanks for reporting. |
@David-Magebinary I've tested it with version 3.8.7 with the following command ~/workspace/techdivision/import-cli-simple/bin/import-simple import:create:ok-file \
&& ~/workspace/techdivision/import-cli-simple/bin/import-simple \
import:products:inventory:msi \
--magento-edition=EE \
--magento-version=2.3.4 \
--db-pdo-dsn="mysql:host=127.0.0.1;dbname=magento_ee_234;charset=utf8" \
--db-username=root \
--db-password=xxxx \
--source-dir=var/importexport from within the Magento installation directory, and it works without any problem. So I've some questions:
Hope your answers will help me to allow me to reproduce the issue. Thanks for supporting |
@wagnert Outside of the Magento installation directory |
@David-Magebinary As M2IF was usually optimized within the Magento PHP environment, it has not been ready for PHP 7.4, as Magento itself, yet. We're working on this, but it'll take some time to make it work. |
@wagnert Do you have a solution or the cause of the issue in your mind? I can submit a pull request but I have to know what is your preferred solution for fixing this issue. |
@David-Magebinary I need to investigate more into the PHP 7.4 problems (there are several others). Did you try the PHAR file? |
@David-Magebinary Did you found the time to test the PHAR file meanwhile? :-) |
Hello @wagnert , David is away for a few months. I can test it for you or provide you with the docker container if that's helpful. Thanks, |
@Adel-Magebinary This would be awesome 🙂 Thanks |
** Exception that I got **
[Symfony\Component\DependencyInjection\Exception\ServiceCircularReferenceException] Circular reference detected for service "import_app_simple.simple", path: "import_app_simple.simple -> import.modules -> import.module.plugin".
How to reproduce the issue:
After generating the ok file, was trying to import the stock information.
./vendor/bin/import-simple import:products:inventory:msi --magento-edition CE --magento-version 2.3.3 --db-pdo-dsn mysql:host=172.17.0.1\;port=3306\;dbname=biv_m2 --db-username root --db-password xxxxx --source-dir /xx/xxxx
** Dirty fix solution:**
<argument type="service" id="import_app_simple.simple"/>
in the file/vendor/techdivision/import/symfony/Resources/config/services.xml
TechDivision\Import\Modules\PluginModule
classI did not use Symfony before, not really sure what the error is related to. Looks like it is a looping injecting issue. eg. parent -> child -> child -> parent
The text was updated successfully, but these errors were encountered: