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

setup:di:compile breaks on 0 indexed object arrays #36

Open
ddoddsr opened this issue Jul 6, 2017 · 6 comments
Open

setup:di:compile breaks on 0 indexed object arrays #36

ddoddsr opened this issue Jul 6, 2017 · 6 comments

Comments

@ddoddsr
Copy link

ddoddsr commented Jul 6, 2017

Built a new and untouched 2.1.7
Loaded Umc_Base on it
Ran module:enable setup:upgrade

and ran setup:di:compile and received this message

[Exception] Warning: ltrim() expects parameter 1 to be string, array given in /Users/dandodd/Projects/tpmco.dev/setup/src/Magento/Setup/Module/Di/Compiler/Config/Chain/BackslashTrim.php on line 61

This looks like a Magento issue with 0 indexed objects. https://github.com/magento/magento2/issues/8803

I thought about changing the index 0 to 1 and 1 to 2 & etc but that would likely break something else.
What do you think about trying that?

The Umc_Base does run so I build a test Module and enabled upgraded and received this error while compiling
Compilation was started.
Interceptors generation... 4/7 [================>-----------] 57% 3 mins 334.2 MiBMiBB08.2 MiB
[RuntimeException]
Class Tpmco\Test\Model\UploaderPool does not exist

@UltimateModuleCreator
Copy link
Owner

UltimateModuleCreator commented Jul 7, 2017 via email

@ddoddsr
Copy link
Author

ddoddsr commented Jul 7, 2017

2.0.0-beta2
I was using the Umc_Base with the latest commit "Pull Merge Request #35 ... from parent/relation"
I added the Tpmco_Test to app/code/Tpmco/Test ...

Tpmco_Test.xml.zip

@ddoddsr
Copy link
Author

ddoddsr commented Aug 6, 2017

I think my issues began at compile of Umc_Base.


bin/magento  module:enable Umc_Base
bin/magento setup:upgrade 
bin/magento setup:di:compile

And it had an exception:

Compilation was started.
Area configuration aggregation... 5/7 [====================>-------]  71% 3 mins 172.0 MiB0 MiB

                                                                                                                                                                               
  [Exception]                                                                                                                                                                  
  Warning: ltrim() expects parameter 1 to be string, array given in /Users/dandodd/Projects/umc.dev/setup/src/Magento/Setup/Module/Di/Compiler/Config/Chain/BackslashTrim.php  
   on line 61          

SO I don't get a clean install of Umc_Base.

@lanrosta
Copy link

lanrosta commented Nov 1, 2017

Any word on this one Dev's? Same issue here on Magento ce 217.

@lanrosta
Copy link

lanrosta commented Nov 2, 2017

Fix: I had to start a new project using the magento 2.2.1-dev (from magento 2 github), then redownloaded this repo (as .zip on 11-02-2017) to get this issue resolved. Magento 217 just doesn't want to work with this UMC 2 so well when it comes to setup:di:compile. Kept getting this same ltrim() error using that one.

@UltimateModuleCreator
Copy link
Owner

@lanrosta Sorry, but I didn't have too much spare time to maintain this repo for now. I'm one-man-show on this. The issue will remain open until I get some spare time to work on it. Thanks for reporting this.

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

No branches or pull requests

3 participants