{"action":"opened","issue":{"url":"https://api.github.com/repos/dotCMS/core/issues/7686","labels_url":"https://api.github.com/repos/dotCMS/core/issues/7686/labels{/name}","comments_url":"https://api.github.com/repos/dotCMS/core/issues/7686/comments","events_url":"https://api.github.com/repos/dotCMS/core/issues/7686/events","html_url":"https://github.com/dotCMS/core/issues/7686","id":73325832,"number":7686,"title":"When Page Cache is On we should NEVER instantiate Velocity  ","user":{"login":"jtesser","id":1486445,"avatar_url":"https://avatars.githubusercontent.com/u/1486445?v=3","gravatar_id":"","url":"https://api.github.com/users/jtesser","html_url":"https://github.com/jtesser","followers_url":"https://api.github.com/users/jtesser/followers","following_url":"https://api.github.com/users/jtesser/following{/other_user}","gists_url":"https://api.github.com/users/jtesser/gists{/gist_id}","starred_url":"https://api.github.com/users/jtesser/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jtesser/subscriptions","organizations_url":"https://api.github.com/users/jtesser/orgs","repos_url":"https://api.github.com/users/jtesser/repos","events_url":"https://api.github.com/users/jtesser/events{/privacy}","received_events_url":"https://api.github.com/users/jtesser/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/dotCMS/core/labels/Type%20:%20bug","name":"Type : bug","color":"006b75"}],"state":"open","locked":false,"assignee":{"login":"oarrietadotcms","id":8225685,"avatar_url":"https://avatars.githubusercontent.com/u/8225685?v=3","gravatar_id":"","url":"https://api.github.com/users/oarrietadotcms","html_url":"https://github.com/oarrietadotcms","followers_url":"https://api.github.com/users/oarrietadotcms/followers","following_url":"https://api.github.com/users/oarrietadotcms/following{/other_user}","gists_url":"https://api.github.com/users/oarrietadotcms/gists{/gist_id}","starred_url":"https://api.github.com/users/oarrietadotcms/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/oarrietadotcms/subscriptions","organizations_url":"https://api.github.com/users/oarrietadotcms/orgs","repos_url":"https://api.github.com/users/oarrietadotcms/repos","events_url":"https://api.github.com/users/oarrietadotcms/events{/privacy}","received_events_url":"https://api.github.com/users/oarrietadotcms/received_events","type":"User","site_admin":false},"milestone":{"url":"https://api.github.com/repos/dotCMS/core/milestones/28","html_url":"https://github.com/dotCMS/core/milestones/3.2.1","labels_url":"https://api.github.com/repos/dotCMS/core/milestones/28/labels","id":1067014,"number":28,"title":"3.2.1","description":"","creator":{"login":"jtesser","id":1486445,"avatar_url":"https://avatars.githubusercontent.com/u/1486445?v=3","gravatar_id":"","url":"https://api.github.com/users/jtesser","html_url":"https://github.com/jtesser","followers_url":"https://api.github.com/users/jtesser/followers","following_url":"https://api.github.com/users/jtesser/following{/other_user}","gists_url":"https://api.github.com/users/jtesser/gists{/gist_id}","starred_url":"https://api.github.com/users/jtesser/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jtesser/subscriptions","organizations_url":"https://api.github.com/users/jtesser/orgs","repos_url":"https://api.github.com/users/jtesser/repos","events_url":"https://api.github.com/users/jtesser/events{/privacy}","received_events_url":"https://api.github.com/users/jtesser/received_events","type":"User","site_admin":false},"open_issues":16,"closed_issues":5,"state":"open","created_at":"2015-04-14T14:47:32Z","updated_at":"2015-05-05T14:12:39Z","due_on":null,"closed_at":null},"comments":0,"created_at":"2015-05-05T14:12:39Z","updated_at":"2015-05-05T14:12:39Z","closed_at":null,"body":"We do it on lines \r\nhttps://github.com/dotCMS/core/blob/master-3.2/src/com/dotmarketing/velocity/VelocityServlet.java#L534\r\nhttps://github.com/dotCMS/core/blob/master-3.2/src/com/dotmarketing/velocity/VelocityServlet.java#L469\r\n\r\nthese lines eventually make calls that setup the engine. \r\n\r\nWe could get a lot smarter in this way. In the CMSFIlter we get the iHTMLPage. When if we stick this in the request and use throughout the the request processing.  Much smarter :-p "}}