Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix EZP-25280: ParentDepth Limitation fails on content creation #1706

Merged
merged 4 commits into from
Jul 1, 2016

Conversation

pspanja
Copy link
Contributor

@pspanja pspanja commented Jul 1, 2016

This PR resolves https://jira.ez.no/browse/EZP-25280

As described in the comment on the issue, the problem comes from missing Location on publishing the first Content draft. Solution is similar to what was already done for LocationLimitationType.

Added failing/fixed integration test and missing unit tests.

@@ -35,7 +35,7 @@ COMPOSER_UPDATE=""
# solr package search API integration tests
if [ "$TEST_CONFIG" = "phpunit-integration-legacy-solr.xml" ] ; then
echo "> Require ezsystems/ezplatform-solr-search-engine:dev-master"
composer require --no-update ezsystems/ezplatform-solr-search-engine:dev-master
composer require --no-update ezsystems/ezplatform-solr-search-engine:~1.0.3
Copy link
Contributor

@andrerom andrerom Jul 1, 2016

Choose a reason for hiding this comment

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

you can probably say ^1.0.0@dev and composer will pick right one depending on current kernel.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That works, tested locally, thanks. Will update and keep when merging to master.

@pspanja pspanja force-pushed the fix-EZP-25280-parent-depth-content-create-6.4 branch from 1f631bd to 7ae302b Compare July 1, 2016 12:52
@pspanja pspanja merged commit d9bf9e6 into 6.4 Jul 1, 2016
@pspanja pspanja deleted the fix-EZP-25280-parent-depth-content-create-6.4 branch July 1, 2016 13:20
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants