{"action":"created","issue":{"url":"https://api.github.com/repos/spree/spree/issues/5992","labels_url":"https://api.github.com/repos/spree/spree/issues/5992/labels{/name}","comments_url":"https://api.github.com/repos/spree/spree/issues/5992/comments","events_url":"https://api.github.com/repos/spree/spree/issues/5992/events","html_url":"https://github.com/spree/spree/pull/5992","id":56194648,"number":5992,"title":"Use the core File class instead of Spree::File","user":{"login":"sevenseacat","id":543859,"avatar_url":"https://avatars.githubusercontent.com/u/543859?v=3","gravatar_id":"","url":"https://api.github.com/users/sevenseacat","html_url":"https://github.com/sevenseacat","followers_url":"https://api.github.com/users/sevenseacat/followers","following_url":"https://api.github.com/users/sevenseacat/following{/other_user}","gists_url":"https://api.github.com/users/sevenseacat/gists{/gist_id}","starred_url":"https://api.github.com/users/sevenseacat/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sevenseacat/subscriptions","organizations_url":"https://api.github.com/users/sevenseacat/orgs","repos_url":"https://api.github.com/users/sevenseacat/repos","events_url":"https://api.github.com/users/sevenseacat/events{/privacy}","received_events_url":"https://api.github.com/users/sevenseacat/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-02-02T04:22:22Z","updated_at":"2015-02-02T14:56:44Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/spree/spree/pulls/5992","html_url":"https://github.com/spree/spree/pull/5992","diff_url":"https://github.com/spree/spree/pull/5992.diff","patch_url":"https://github.com/spree/spree/pull/5992.patch"},"body":"When eager loading classes in production, this File constant is a different constant than in development"},"comment":{"url":"https://api.github.com/repos/spree/spree/issues/comments/72470175","html_url":"https://github.com/spree/spree/pull/5992#issuecomment-72470175","issue_url":"https://api.github.com/repos/spree/spree/issues/5992","id":72470175,"user":{"login":"gmacdougall","id":764390,"avatar_url":"https://avatars.githubusercontent.com/u/764390?v=3","gravatar_id":"","url":"https://api.github.com/users/gmacdougall","html_url":"https://github.com/gmacdougall","followers_url":"https://api.github.com/users/gmacdougall/followers","following_url":"https://api.github.com/users/gmacdougall/following{/other_user}","gists_url":"https://api.github.com/users/gmacdougall/gists{/gist_id}","starred_url":"https://api.github.com/users/gmacdougall/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gmacdougall/subscriptions","organizations_url":"https://api.github.com/users/gmacdougall/orgs","repos_url":"https://api.github.com/users/gmacdougall/repos","events_url":"https://api.github.com/users/gmacdougall/events{/privacy}","received_events_url":"https://api.github.com/users/gmacdougall/received_events","type":"User","site_admin":false},"created_at":"2015-02-02T14:56:44Z","updated_at":"2015-02-02T14:56:44Z","body":"To be clear, I don't have any problem with explicitly declaring the global namespace for clarity (it's probably the cleanest code). I just want to make sure that we understand what's going on, and why this fixes something.\r\n\r\nIf it's not a fix for something specific, this commit should only be applied to master, and not 2-4-stable."}}