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

Files Interface in CMS fails to load, Internal Server Error #2257

Closed
jenniferaslan opened this issue Sep 10, 2018 · 9 comments
Closed

Files Interface in CMS fails to load, Internal Server Error #2257

jenniferaslan opened this issue Sep 10, 2018 · 9 comments

Comments

@jenniferaslan
Copy link

jenniferaslan commented Sep 10, 2018

Steps to reproduce the issue in an SS4 site:

  1. Add a has_one Image::class to Page.php

  2. Add an UploadField for that Image to getCMSFields

  3. Using Firefox, log into the CMS and go to a Page page

  4. Right-click anywhere and choose Inspect Element from the contextual menu

  5. In the Developer Tools window, choose the Network tab (it may help to pop this out into its own window)

  6. In the CMS, view the interface in Split mode

  7. Attach an image using the UploadField

  8. Save the page

  9. Click the UploadField to see the interface for choosing an image and see that the selected image is shown in the right-hand side of the dialog
    whenitworks

  10. Close the dialog

  11. Refresh the page

  12. Click the UploadField to see the interface for choosing an image and see that this time the right-hand side of the dialog is empty except for a message about internal server error.
    internalservererror

  13. Look over at the Network tab in the Developer Tools window to see the error 500 that just happened
    error500
    500error.txt

  14. Close the dialog for choosing an image

  15. Click the Files button in the far left column of the CMS

  16. Notice that the right side of the interface fails to load
    brokenfilesinterface

  17. Look at the Network tab of the Developer Tools window to see that the last item that loaded is “types” and the final two items that usually appear after “types” failed to load.
    Normal Loading:
    whenfilesloadsalltheway
    Incomplete Loading:
    stoppedloading

  18. Log out of the CMS and log back in and it will all work okay until you happen to refresh the page with the Split view open.

I don’t think this happens if you work only in Edit mode. I think this has something to do with ?stage=Stage.

composershow.txt

@jenniferaslan
Copy link
Author

I have received feedback from a client that she doesn’t use the split view because she thinks it’s buggy, but she still experiences the problem described in this issue.

@mlewis-everley
Copy link

I just did a test on a sandbox install I have and wasn't able to replicate the issue at all. What version of SS4 are you using? Also, what modules do you have installed?

@jenniferaslan
Copy link
Author

My composer show results are attached at the bottom of this issue. I have SS 4.2 installed on this site. We are going to test on a simple SS4 install that doesn't have certain modules installed on it to see if this still happens to us.

@mlewis-everley
Copy link

You have got quite a few modules installed, my first guess would be something to do with that.

Have you tried making sure everything is up to date? Might be a version issue.

@micahsheets
Copy link

I have narrowed the issue down to using SilverStripe\Versioned\Versioned: use_session: true in the yml config. If that setting is there this error is reproduce able. If it is removed, it does not happen. I do not know of the side affects of not having use_session: true.

@kinglozzer
Copy link
Member

If Versioned.use_session is the cause, then perhaps this issue is another symptom of silverstripe/silverstripe-admin#607?

@chillu
Copy link
Member

chillu commented Oct 3, 2018

Yeah this might be fixed by silverstripe/silverstripe-asset-admin#824?

@bergice
Copy link
Contributor

bergice commented Oct 10, 2018

Confirmed that this is fixed by silverstripe/silverstripe-asset-admin#824

Will review again as soon as that has been merged and closed.

@bergice
Copy link
Contributor

bergice commented Oct 10, 2018

Resolved with silverstripe/silverstripe-versioned#173

@bergice bergice closed this as completed Oct 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants