Skip to content
Snippets Groups Projects
Commit f9648cee authored by uhensler's avatar uhensler
Browse files

OO-4186: Consistent method names

parent 67b734e1
No related branches found
No related tags found
No related merge requests found
Showing
with 25 additions and 25 deletions
......@@ -56,7 +56,7 @@ public interface CourseAssessmentService {
/**
* Returns the persisted AssessmentEntry of the user. Check
* AssessmentConfig.isScoreEvaluationPersisted() before invoking this method.
* AssessmentConfig.isEvaluationPersisted() before invoking this method.
*
* @param courseNode
* @param userCourseEnvironment
......@@ -120,7 +120,7 @@ public interface CourseAssessmentService {
/**
* Returns the ScoreCalculator if the course nod can calculate its score. Check
* AssessmentConfig.isScoreEvaluationCalculated() before invoking this method.
* AssessmentConfig.isEvaluationCalculated() before invoking this method.
*
* @param courseNode
* @return
......
......@@ -35,14 +35,14 @@ public interface AssessmentConfig {
public boolean isAssessable();
/**
* @return true if the score evaluation is persisted
* @return true if the assessment evaluation is persisted
*/
public boolean isScoreEvaluationPersisted();
public boolean isEvaluationPersisted();
/**
* @return true if the score evaluation is calculated e.g as sum of other assessments.
*/
public boolean isScoreEvaluationCalculated();
public boolean isEvaluationCalculated();
/**
* @return True if this course node produces a score variable for the learner
......
......@@ -51,7 +51,7 @@ public interface AssessmentHandler {
public AssessmentConfig getAssessmentConfig(CourseNode courseNode);
/**
* This method has to be implemented if the AssessmentConfig.isScoreEvaluationPersisted() return true.
* This method has to be implemented if the AssessmentConfig.isEvaluationPersisted() return true.
*
* @param courseNode
* @param userCourseEnvironment
......@@ -60,7 +60,7 @@ public interface AssessmentHandler {
public AssessmentEntry getAssessmentEntry(CourseNode courseNode, UserCourseEnvironment userCourseEnvironment);
/**
* This method has to be implemented if the AssessmentConfig.isScoreEvaluationCalculated() return true.
* This method has to be implemented if the AssessmentConfig.isEvaluationCalculated() return true.
*
* @param courseNode
* @param userCourseEnvironment
......@@ -69,7 +69,7 @@ public interface AssessmentHandler {
public AssessmentEvaluation getCalculatedScoreEvaluation(CourseNode courseNode, UserCourseEnvironment userCourseEnvironment);
/**
* This method has to be implemented if the AssessmentConfig.isScoreEvaluationCalculated() return true.
* This method has to be implemented if the AssessmentConfig.isEvaluationCalculated() return true.
*
* @param courseNode
* @param userCourseEnvironment
......
......@@ -43,12 +43,12 @@ public abstract class ModuleAssessmentConfig implements AssessmentConfig {
}
@Override
public boolean isScoreEvaluationPersisted() {
public boolean isEvaluationPersisted() {
return true;
}
@Override
public boolean isScoreEvaluationCalculated() {
public boolean isEvaluationCalculated() {
return false;
}
......
......@@ -39,12 +39,12 @@ public class NonAssessmentConfig implements AssessmentConfig {
}
@Override
public boolean isScoreEvaluationPersisted() {
public boolean isEvaluationPersisted() {
return false;
}
@Override
public boolean isScoreEvaluationCalculated() {
public boolean isEvaluationCalculated() {
return false;
}
......
......@@ -122,9 +122,9 @@ public class CourseAssessmentServiceImpl implements CourseAssessmentService {
AssessmentHandler assessmentHandler = getAssessmentHandler(courseNode);
AssessmentEvaluation assessmentEvaluation = AssessmentEvaluation.EMPTY_EVAL;
if (assessmentConfig.isScoreEvaluationCalculated()) {
if (assessmentConfig.isEvaluationCalculated()) {
assessmentEvaluation = assessmentHandler.getCalculatedScoreEvaluation(courseNode, userCourseEnvironment);
} else if (assessmentConfig.isScoreEvaluationPersisted()) {
} else if (assessmentConfig.isEvaluationPersisted()) {
assessmentEvaluation = getPersistedAssessmentEvaluation(courseNode, userCourseEnvironment);
}
// Other handlers should be able to have other implementations
......
......@@ -358,7 +358,7 @@ public class IdentityListCourseNodeController extends FormBasicController
if(assessmentConfig.hasAttempts()) {
columnsModel.addFlexiColumnModel(new DefaultFlexiColumnModel(IdentityCourseElementCols.attempts));
}
if(!assessmentConfig.isScoreEvaluationCalculated()) {
if(!assessmentConfig.isEvaluationCalculated()) {
columnsModel.addFlexiColumnModel(new DefaultFlexiColumnModel(IdentityCourseElementCols.userVisibility,
new UserVisibilityCellRenderer(getTranslator())));
}
......
......@@ -45,12 +45,12 @@ public class LTIAssessmentConfig implements AssessmentConfig {
}
@Override
public boolean isScoreEvaluationPersisted() {
public boolean isEvaluationPersisted() {
return true;
}
@Override
public boolean isScoreEvaluationCalculated() {
public boolean isEvaluationCalculated() {
return false;
}
......
......@@ -47,12 +47,12 @@ public class IQTESTAssessmentConfig implements AssessmentConfig {
}
@Override
public boolean isScoreEvaluationPersisted() {
public boolean isEvaluationPersisted() {
return true;
}
@Override
public boolean isScoreEvaluationCalculated() {
public boolean isEvaluationCalculated() {
return false;
}
......
......@@ -42,12 +42,12 @@ public class ScormAssessmentConfig implements AssessmentConfig {
}
@Override
public boolean isScoreEvaluationPersisted() {
public boolean isEvaluationPersisted() {
return true;
}
@Override
public boolean isScoreEvaluationCalculated() {
public boolean isEvaluationCalculated() {
return false;
}
......
......@@ -44,12 +44,12 @@ public class STAssessmentConfig implements AssessmentConfig {
}
@Override
public boolean isScoreEvaluationPersisted() {
public boolean isEvaluationPersisted() {
return true;
}
@Override
public boolean isScoreEvaluationCalculated() {
public boolean isEvaluationCalculated() {
return true;
}
......
......@@ -153,14 +153,14 @@ public class ScoreAccounting {
AssessmentEvaluation se = null;
if (recursionLevel <= 15) {
AssessmentConfig assessmentConfig = courseAssessmentService.getAssessmentConfig(cn);
if (update && assessmentConfig.isScoreEvaluationCalculated()) {
if (update && assessmentConfig.isEvaluationCalculated()) {
AssessmentEntry entry = identToEntries.get(cn.getIdent());
se = calculateScoreEvaluation(entry, cn, assessmentConfig);
cachedScoreEvals.put(cn, se);
} else {
se = cachedScoreEvals.get(cn);
if (se == null) { // result of this node has not been cached yet, do it
if(assessmentConfig.isScoreEvaluationPersisted()) {
if(assessmentConfig.isEvaluationPersisted()) {
AssessmentEntry entry = identToEntries.get(cn.getIdent());
se = courseAssessmentService.toAssessmentEvaluation(entry, assessmentConfig);
} else {
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment