{"action":"created","issue":{"url":"https://api.github.com/repos/codevise/pageflow/issues/1033","repository_url":"https://api.github.com/repos/codevise/pageflow","labels_url":"https://api.github.com/repos/codevise/pageflow/issues/1033/labels{/name}","comments_url":"https://api.github.com/repos/codevise/pageflow/issues/1033/comments","events_url":"https://api.github.com/repos/codevise/pageflow/issues/1033/events","html_url":"https://github.com/codevise/pageflow/issues/1033","id":372631069,"node_id":"MDU6SXNzdWUzNzI2MzEwNjk=","number":1033,"title":"500 error on content save","user":{"login":"tim-wgbh","id":11877425,"node_id":"MDQ6VXNlcjExODc3NDI1","avatar_url":"https://avatars3.githubusercontent.com/u/11877425?v=4","gravatar_id":"","url":"https://api.github.com/users/tim-wgbh","html_url":"https://github.com/tim-wgbh","followers_url":"https://api.github.com/users/tim-wgbh/followers","following_url":"https://api.github.com/users/tim-wgbh/following{/other_user}","gists_url":"https://api.github.com/users/tim-wgbh/gists{/gist_id}","starred_url":"https://api.github.com/users/tim-wgbh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tim-wgbh/subscriptions","organizations_url":"https://api.github.com/users/tim-wgbh/orgs","repos_url":"https://api.github.com/users/tim-wgbh/repos","events_url":"https://api.github.com/users/tim-wgbh/events{/privacy}","received_events_url":"https://api.github.com/users/tim-wgbh/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2018-10-22T18:04:04Z","updated_at":"2018-10-29T12:06:29Z","closed_at":null,"author_association":"NONE","body":"Our editor saw this error when saving a text page:\r\n![screen shot 2018-10-22 at 1 30 27 pm](https://user-images.githubusercontent.com/11877425/47309710-2ba0bb00-d603-11e8-9ab9-1cbef62857a7.jpg)\r\n\r\nI traced it to an unloaded concern method, #verify_edit_lock!:\r\n\r\n![screen shot 2018-10-22 at 1 29 00 pm](https://user-images.githubusercontent.com/11877425/47309745-407d4e80-d603-11e8-84f8-6254d42f3445.jpg)\r\n\r\nDo you know why this might be happening?  Thank you."},"comment":{"url":"https://api.github.com/repos/codevise/pageflow/issues/comments/433885959","html_url":"https://github.com/codevise/pageflow/issues/1033#issuecomment-433885959","issue_url":"https://api.github.com/repos/codevise/pageflow/issues/1033","id":433885959,"node_id":"MDEyOklzc3VlQ29tbWVudDQzMzg4NTk1OQ==","user":{"login":"tf","id":26554,"node_id":"MDQ6VXNlcjI2NTU0","avatar_url":"https://avatars3.githubusercontent.com/u/26554?v=4","gravatar_id":"","url":"https://api.github.com/users/tf","html_url":"https://github.com/tf","followers_url":"https://api.github.com/users/tf/followers","following_url":"https://api.github.com/users/tf/following{/other_user}","gists_url":"https://api.github.com/users/tf/gists{/gist_id}","starred_url":"https://api.github.com/users/tf/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tf/subscriptions","organizations_url":"https://api.github.com/users/tf/orgs","repos_url":"https://api.github.com/users/tf/repos","events_url":"https://api.github.com/users/tf/events{/privacy}","received_events_url":"https://api.github.com/users/tf/received_events","type":"User","site_admin":false},"created_at":"2018-10-29T12:06:29Z","updated_at":"2018-10-29T12:06:29Z","author_association":"MEMBER","body":"In development mode this can definitely cause Pageflow's `ApplicationController` not to be auto loaded since you already defined the constant. In production mode, I would expect eager loading to work anyway, but maybe I am missing something. Generally speaking, I think it is safer practice to `include` concerns in an initializer to patch some library's classes rather than trying to reopen classes and risking to break auto loading. In this case the even simpler approach might be to override `layouts/pageflow/application.html.erb` by creating a template with this name in your `app/views` directory. You are changing the layout for all entries anyway - independent of the theme.\r\n\r\nStill, please note that overriding templates like that basically means you will need to manually port changes on future updates. Pageflow provides [some partials](https://github.com/codevise/pageflow/wiki/Customizing-Views) as extension points which are empty by default and should be safer to override. "}}
{"actor":{"display_login":"tf"}}