{"comment":{"url":"https://api.github.com/repos/openmrs/openmrs-module-legacyui/comments/14319236","html_url":"https://github.com/openmrs/openmrs-module-legacyui/commit/1ae160c3863a37830c333d967978356c08fe5f70#commitcomment-14319236","id":14319236,"user":{"login":"wluyima","id":1395907,"avatar_url":"https://avatars.githubusercontent.com/u/1395907?v=3","gravatar_id":"","url":"https://api.github.com/users/wluyima","html_url":"https://github.com/wluyima","followers_url":"https://api.github.com/users/wluyima/followers","following_url":"https://api.github.com/users/wluyima/following{/other_user}","gists_url":"https://api.github.com/users/wluyima/gists{/gist_id}","starred_url":"https://api.github.com/users/wluyima/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wluyima/subscriptions","organizations_url":"https://api.github.com/users/wluyima/orgs","repos_url":"https://api.github.com/users/wluyima/repos","events_url":"https://api.github.com/users/wluyima/events{/privacy}","received_events_url":"https://api.github.com/users/wluyima/received_events","type":"User","site_admin":false},"position":4,"line":51,"path":"omod/src/main/java/org/openmrs/web/CopyLegacyUiContentToWebInf.java","commit_id":"1ae160c3863a37830c333d967978356c08fe5f70","created_at":"2015-11-11T02:22:23Z","updated_at":"2015-11-11T02:22:23Z","body":"I've actually remembered, you need to revert this, in theory all JSPs are supposed to reside under WEB-INF/view for security purposes so that a user can't access them by entering a URL that points directly to a JSP. If you look at them before the move, they were some JSPs under webapps/admin which were just redirecting to the appropriate URLs for those under WEB-INF/view/admin so that one can enter something like /openmrs/dictionary and they automatically get redirected to the dictionary index page. But we can no longer support this from the legacyui module, see [LUI-35] (https://issues.openmrs.org/browse/LUI-35). And the solution was to stop supporting such URLs and change them as seen at https://github.com/openmrs/openmrs-module-legacyui/commit/901d5ed39e506fce2aa2cfe8c902e3549e1722fd, the other option if we wish to support them is to add URL mappings for them dynamically via the new servlet API methods to the spring DispatcherServlet and then do the redirect from a a controller like we are doing for some pages"}}