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

\Magento\Framework\App\Helper\AbstractHelper leaking undeclared property #4326

Merged
merged 2 commits into from
May 14, 2016
Merged

Conversation

nevvermind
Copy link
Contributor

/cc #2103

@vkublytskyi vkublytskyi added the PS label Apr 27, 2016
@nevvermind
Copy link
Contributor Author

Erm..

The command "sh -c "if [ '$TEST_SUITE' = 'integration_part_1' ] || [ '$TEST_SUITE' = 'integration_part_2' ]; then cd dev/tests/integration/; bash IntegationTestsForTravis.sh 2; fi"" exited with 0.

34.02s$ sh -c "if [ '$TEST_SUITE' = 'integration_part_1' ]; then cd dev/tests/integration/; ./../../../vendor/bin/phpunit -c phpunit.xml.travis1; fi"

PHP Parse error:  syntax error, unexpected '.', expecting ',' or ';' in /home/travis/build/magento/magento2/dev/tests/integration/testsuite/Magento/Framework/Search/Adapter/Mysql/AdapterTest.php on line 39

Parse error: syntax error, unexpected '.', expecting ',' or ';' in /home/travis/build/magento/magento2/dev/tests/integration/testsuite/Magento/Framework/Search/Adapter/Mysql/AdapterTest.php on line 39

How did that get in there?

@adragus-inviqa
Copy link
Contributor

/cc #4343

@paliarush
Copy link
Contributor

Internal ticket was created: MAGETWO-52561

@piotrekkaminski
Copy link
Contributor

@paliarush can we merge now? Build is green.

@magento-team magento-team merged commit 6975a69 into magento:develop May 14, 2016
@nevvermind nevvermind deleted the patch-1 branch May 20, 2016 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants