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

White page after "Next" on creating new poll #46

Closed
ghost opened this issue Mar 18, 2016 · 2 comments
Closed

White page after "Next" on creating new poll #46

ghost opened this issue Mar 18, 2016 · 2 comments

Comments

@ghost
Copy link

ghost commented Mar 18, 2016

Using Polls 0.7.0 on oC 8.2.3 hitting "Next" on the "Create new poll" page leads to a white page.

Steps to reproduce:

  1. Create new poll
  2. Enter simple title and description
  3. Leave the defaults "Registered users only", "Event schedule" and no "Expires"
  4. Choose one single day and one single hour
  5. Turn the single cell to green
  6. Hit "Next"

After doing this steps the /index.php/apps/polls/create page redirects to an empty/white /index.php/apps/polls/. Hitting Enter in the URL bar (not reloading the page with F5) refreshes the page and shows the newly created poll.

There is absolutely nothing logged in the data/owncloud.log, php error log or nginx error log so this is not an internal server error.

The only interesting part could be:

xxx - - [19/Mar/2016:00:25:07 +0100] "POST /index.php/apps/polls/insert HTTP/1.1" 307 0 "-" "Mozilla/5.0"
xxx - - [19/Mar/2016:00:25:07 +0100] "POST /index.php/apps/polls/ HTTP/1.1" 405 0 "-" "Mozilla/5.0"

where it seems that the POST for inserting the poll is redirected to a POST on /index.php/apps/polls/ which is failing with a 405/Method Not Allowed.

The same is also happening when e.g. deleting a poll.

@v1r0x
Copy link
Collaborator

v1r0x commented Mar 18, 2016

This is a bug in oc core and has been fixed for 9.0. For older versions you have to edit this line: https://github.com/owncloud/core/blob/v8.2.3/lib/public/appframework/http/redirectresponse.php#L47 and replace Http::STATUS_TEMPORARY_REDIRECT with Http::STATUS_SEE_OTHER.

@v1r0x v1r0x closed this as completed Mar 18, 2016
@ghost
Copy link
Author

ghost commented Mar 18, 2016

Grmlll, nevermind. Thought this is not related to owncloud/core#22497 as thought thats already included in 8.2.3. But i was wrong. :-/ Seems that is a "breaking change" which isn't backported because of that.

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

1 participant