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

EZEE-2533: Error HTTP 400 trying to preview Form during creation #892

Merged
merged 1 commit into from
Mar 19, 2019

Conversation

mikadamczyk
Copy link
Contributor

@mikadamczyk mikadamczyk commented Mar 11, 2019

Question Answer
Tickets https://jira.ez.no/browse/EZEE-2533
Bug fix? yes
New feature? no
BC breaks? no
Tests pass? yes
Doc needed? no
License GPL-2.0

https://github.com/ezsystems/ezplatform-form-builder/pull/143

Checklist:

  • Coding standards ($ composer fix-cs)
  • Ready for Code Review

@ViniTou
Copy link
Contributor

ViniTou commented Mar 11, 2019

What about user root location or some other custom ones?

@mikadamczyk
Copy link
Contributor Author

This is not needed in context of this bug, but maybe it is worth to add other custom root locations. What do you think about it?

@mikadamczyk mikadamczyk force-pushed the EZEE-2533 branch 2 times, most recently from c852345 to 28ce778 Compare March 12, 2019 10:45
@mikadamczyk mikadamczyk requested a review from webhdx March 12, 2019 11:48
@mikadamczyk mikadamczyk force-pushed the EZEE-2533 branch 2 times, most recently from c69e332 to 9c45ada Compare March 12, 2019 12:27
continue;
$context = new SiteaccessPreviewVoterContext($location, $versionInfo, $siteaccess, $languageCode);
foreach ($this->siteaccessPreviewVoters as $siteaccessPreviewVoter) {
if (true === $siteaccessPreviewVoter->vote($context)) {
Copy link
Contributor

Choose a reason for hiding this comment

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

We can skip comparing

@lserwatka lserwatka merged commit 71ead62 into ezsystems:1.4 Mar 19, 2019
@mikadamczyk
Copy link
Contributor Author

merged up

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

7 participants