-
Notifications
You must be signed in to change notification settings - Fork 40
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
Comments
Hello Dan.
First of all, thanks for reporting this. getting help with the testing from
the community helps me a lot.
I developed the latest version of the UMC on Magento 2.2-dev, the version
available on github and I didn't get the compilation error.
I will try on 2.1.7.
But I need your help even more.
What version of the `Umc_Base` module are you using?
I just released yesteday 2.0.0-beta2. I don't know if you got the latest
version or the `beta1` version.
As for the generated module, please send me the file
`var/umc/modules/Tpmco_Test.xml`. This way I can reproduce the module on my
side and investigate the issue further.
Thank you again for all your help.
Marius.
…On Thu, Jul 6, 2017 at 7:44 PM, Dan Dodd ***@***.***> wrote:
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.
magento/magento2#8803 <http://url>
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
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#36>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AK3PT1es6i8pQf6nc04IlilbE47Ub6Ajks5sLQ8GgaJpZM4OP5jH>
.
|
2.0.0-beta2 |
I think my issues began at compile of Umc_Base.
And it had an exception:
SO I don't get a clean install of Umc_Base. |
Any word on this one Dev's? Same issue here on Magento ce 217. |
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. |
@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. |
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
The text was updated successfully, but these errors were encountered: