Details

    • Type: Bug
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 11.1.2
    • Fix Version/s: 11.2.1
    • Component/s: None
    • Labels:
      None

      Description

      When closing Onyx self tests a red screen occurs:

      stack of 1.cause::java.lang.NullPointerException: null -> at de.bps.onyx.plugin.OnyxModule.existsResultSet(OnyxModule.java:375) at de.bps.onyx.plugin.run.OnyxRunController.showView(OnyxRunController.java:276) at de.bps.onyx.plugin.run.OnyxRunController.(OnyxRunController.java:185) at org.olat.course.nodes.IQSELFCourseNode.createNodeRunConstructionResult(IQSELFCourseNode.java:125) at org.olat.course.run.navigation.NavigationHandler.doEvaluateJumpTo(NavigationHandler.java:461) at org.olat.course.run.navigation.NavigationHandler.evaluateJumpToTreeNode(NavigationHandler.java:286) at org.olat.course.run.RunMainController.doNodeClick(RunMainController.java:656) at 
      

      The problem is that the variable assessmentId is null. This patch fixes that:

      diff --git a/src/main/java/de/bps/onyx/plugin/OnyxModule.java b/src/main/java/de/bps/onyx/plugin/OnyxModule.java
      --- a/src/main/java/de/bps/onyx/plugin/OnyxModule.java
      +++ b/src/main/java/de/bps/onyx/plugin/OnyxModule.java
      @@ -372,7 +372,7 @@
       		List<QTIResultSet> resultSets = qrm.getResultSets(courseResourceableId, courseNode.getIdent(), courseNode.getReferencedRepositoryEntry().getKey(),
       				identity);
       		for (QTIResultSet resultSet : resultSets) {
      -			if(resultSet.getAssessmentID() == assessmentId && !resultSet.getSuspended()) {
      +			if(Long.valueOf(resultSet.getAssessmentID()) == assessmentId && !resultSet.getSuspended()) {
       				return true;
       			}
       		}
      

      Another approach can be testing if assessmentId is null.

        Attachments

          Activity

            People

            • Assignee:
              srosse Stéphane Rossé
              Reporter:
              stephan Stephan Clemenz
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0 minutes
                0m
                Logged:
                Time Spent - 1 hour, 35 minutes
                1h 35m