Index: lams_admin/src/java/org/lamsfoundation/lams/admin/web/controller/LtiConsumerManagementController.java =================================================================== diff -u -rf2ad75cef0c507a64877942631fee13efbc6ed50 -rccc788f07597ad3fde52adb6ff13e964b552c35d --- lams_admin/src/java/org/lamsfoundation/lams/admin/web/controller/LtiConsumerManagementController.java (.../LtiConsumerManagementController.java) (revision f2ad75cef0c507a64877942631fee13efbc6ed50) +++ lams_admin/src/java/org/lamsfoundation/lams/admin/web/controller/LtiConsumerManagementController.java (.../LtiConsumerManagementController.java) (revision ccc788f07597ad3fde52adb6ff13e964b552c35d) @@ -44,12 +44,12 @@ * Shows all available LTI tool consumers */ @RequestMapping(path = "/start") - public String unspecified(HttpServletRequest request) { + public String start(HttpServletRequest request) { List ltiConsumers = integrationService.getAllToolConsumers(); Collections.sort(ltiConsumers); request.setAttribute("ltiConsumers", ltiConsumers); - return "lti/ltiConsumerList"; + return "integration/ltiConsumerList"; } /** @@ -63,6 +63,8 @@ if (sid != null) { ExtServer ltiConsumer = integrationService.getExtServer(sid); BeanUtils.copyProperties(ltiConsumerForm, ltiConsumer); + + //display lessonFinishUrl. it's not part of ltiConsumerForm, as long as this property is not going to be edited String lessonFinishUrl = ltiConsumer.getLessonFinishUrl() == null ? "-" : ltiConsumer.getLessonFinishUrl(); request.setAttribute("lessonFinishUrl", lessonFinishUrl); @@ -71,7 +73,7 @@ //do nothing } - return "lti/ltiConsumer"; + return "integration/ltiConsumer"; } /** @@ -85,7 +87,7 @@ ltiConsumer.setDisabled(disable); integrationService.saveExtServer(ltiConsumer); - return unspecified(request); + return start(request); } /** @@ -96,7 +98,7 @@ Integer sid = WebUtil.readIntParam(request, "sid", true); userManagementService.deleteById(ExtServer.class, sid); - return unspecified(request); + return start(request); } /** @@ -160,22 +162,24 @@ if (errorMap.isEmpty()) { ExtServer ltiConsumer = null; - if (sid == null) { + if (sid.equals(-1)) { ltiConsumer = new ExtServer(); BeanUtils.copyProperties(ltiConsumer, ltiConsumerForm); ltiConsumer.setSid(null); ltiConsumer.setServerTypeId(ExtServer.LTI_CONSUMER_SERVER_TYPE); ltiConsumer.setUserinfoUrl("blank"); + } else { ltiConsumer = integrationService.getExtServer(sid); BeanUtils.copyProperties(ltiConsumer, ltiConsumerForm); } + ltiConsumer.setTimeToLiveLoginRequestEnabled(false); integrationService.saveExtServer(ltiConsumer); - return unspecified(request); + return start(request); } else { request.setAttribute("errorMap", errorMap); - return "lti/ltiConsumer"; + return "integration/ltiConsumer"; } } Index: lams_admin/web/integration/extLessonForm.jsp =================================================================== diff -u --- lams_admin/web/integration/extLessonForm.jsp (revision 0) +++ lams_admin/web/integration/extLessonForm.jsp (revision ccc788f07597ad3fde52adb6ff13e964b552c35d) @@ -0,0 +1,38 @@ +<%@ include file="/taglibs.jsp"%> + +

+
+ +
+
+ +
+
+ +
+
+ +
+
+ +
+
+ +
+
+ +
\ No newline at end of file Index: lams_admin/web/integration/ltiConsumer.jsp =================================================================== diff -u --- lams_admin/web/integration/ltiConsumer.jsp (revision 0) +++ lams_admin/web/integration/ltiConsumer.jsp (revision ccc788f07597ad3fde52adb6ff13e964b552c35d) @@ -0,0 +1,126 @@ + +<%@ include file="/taglibs.jsp"%> + + + + + ${title} + + + + + + + + + + + + + + +

+ + + + + + + +

+ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + +
 * + + +
 * + + +
 * + + +
: + +
 * + + +
: + +
: + ${lessonFinishUrl} +
: + +
+ + <%@ include file="extLessonForm.jsp"%> + +
+ + " /> +
+ +
+
+ +
Index: lams_admin/web/integration/ltiConsumerList.jsp =================================================================== diff -u --- lams_admin/web/integration/ltiConsumerList.jsp (revision 0) +++ lams_admin/web/integration/ltiConsumerList.jsp (revision ccc788f07597ad3fde52adb6ff13e964b552c35d) @@ -0,0 +1,88 @@ + + +<%@ include file="/taglibs.jsp"%> + + + + + ${title} + + + + + + + + + +

+ + + +

+ + + + + + + + + + + + + + + + + + + + + + + + +
+ + + + + + + + + + + + +   + + + + + + + + + + + + +   + + + +
+ +

${fn:length(ltiConsumers)} 

+ + + +
+ +
+ + Index: lams_admin/web/integration/serverlist.jsp =================================================================== diff -u --- lams_admin/web/integration/serverlist.jsp (revision 0) +++ lams_admin/web/integration/serverlist.jsp (revision ccc788f07597ad3fde52adb6ff13e964b552c35d) @@ -0,0 +1,72 @@ + + +<%@ include file="/taglibs.jsp"%> + + + + + ${title} + + + + + + + + + +

+ +

+ + + + + + + + + + + + + + + + + + + + + + +
+ + + + + + + + + + +   + + + + + + + + +   + +
+

${fn:length(servers)} 

+ + + +
+ +
Index: lams_admin/web/integration/servermaintain.jsp =================================================================== diff -u --- lams_admin/web/integration/servermaintain.jsp (revision 0) +++ lams_admin/web/integration/servermaintain.jsp (revision ccc788f07597ad3fde52adb6ff13e964b552c35d) @@ -0,0 +1,136 @@ + +<%@ include file="/taglibs.jsp"%> + + + + + + + ${title} + + + + + + + + + + + + + + + + +

+ + + + + + + +

+ + + + + + +
+ + +
+
+ + +
+
+ + +
+
+ + +
+
+ + +
+
+ +
+
+ +
+
style="display:none;" > + + +
+
+ + +
+
+ + +
+
+ + +
+ + <%@ include file="extLessonForm.jsp"%> + +
+ " /> + + " /> +
+
+
+ + +
\ No newline at end of file Fisheye: Tag ccc788f07597ad3fde52adb6ff13e964b552c35d refers to a dead (removed) revision in file `lams_admin/web/lti/ltiConsumer.jsp'. Fisheye: No comparison available. Pass `N' to diff? Fisheye: Tag ccc788f07597ad3fde52adb6ff13e964b552c35d refers to a dead (removed) revision in file `lams_admin/web/lti/ltiConsumerList.jsp'. Fisheye: No comparison available. Pass `N' to diff? Fisheye: Tag ccc788f07597ad3fde52adb6ff13e964b552c35d refers to a dead (removed) revision in file `lams_admin/web/serverlist.jsp'. Fisheye: No comparison available. Pass `N' to diff? Fisheye: Tag ccc788f07597ad3fde52adb6ff13e964b552c35d refers to a dead (removed) revision in file `lams_admin/web/servermaintain.jsp'. Fisheye: No comparison available. Pass `N' to diff?