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

Include taglib (http://java.sun.com/jsp/jstl/core) in main.jsp. #74

Open
wants to merge 1 commit into
base: dspace-5_x_x-cris
Choose a base branch
from

Conversation

cmtjk
Copy link

@cmtjk cmtjk commented May 8, 2019

Rational

Taglib (http://java.sun.com/jsp/jstl/core) is not included in main.jsp but used to set dspace.layout.head.last.
This caused dspace.layout.head.last to never be set properly.

This is the source code the browser receives from the server. <c:set> was not processed by the servlet.
Screenshot from 2019-05-08 09:50:02

Fortunately, no problems emerged because the JavaScript within <c:set var="dspace.layout.head.last" scrope="request">...</c:set> could be interpreted without problems by the browser.
But recent changes for our instance, which expanded this JavaScript code, rendered the site unusable, since the servlet consistently chopped off parts of the JavaScript when it became too long before proceeding with the rest of the page.

This PR fixes this issue by importing the respective taglib which was forgotten in eaac488.

'dspace.layout.head.last' properly. This will no longer corrupt main.jsp
when rendered due to lengthy content in <c:set>.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant