Index: lams_common/src/java/org/lamsfoundation/lams/learningdesign/Grouping.java =================================================================== diff -u -rc6274262a61a40b34f77cd8db35f374cfbcc17b4 -r248ac2f8f3f720a0ccf53fbc2e6bf2f0e71e0c2a --- lams_common/src/java/org/lamsfoundation/lams/learningdesign/Grouping.java (.../Grouping.java) (revision c6274262a61a40b34f77cd8db35f374cfbcc17b4) +++ lams_common/src/java/org/lamsfoundation/lams/learningdesign/Grouping.java (.../Grouping.java) (revision 248ac2f8f3f720a0ccf53fbc2e6bf2f0e71e0c2a) @@ -68,9 +68,6 @@ /** nullable persistent field */ private Integer groupingUIID; - private Integer groupingTypeId; - - /** persistent field */ private Set groups; @@ -139,20 +136,15 @@ */ public Integer getGroupingTypeId() { - /* if(this instanceof LessonClass) + if(this instanceof LessonClass) return CLASS_GROUPING_TYPE; else if(this instanceof ChosenGrouping) return CHOSEN_GROUPING_TYPE; else - return RANDOM_GROUPING_TYPE; */ - return this.groupingTypeId; + return RANDOM_GROUPING_TYPE; } - - public void setGroupingTypeId(Integer groupingTypeID) - { - this.groupingTypeId = groupingTypeID; - } + /** * @@ -349,16 +341,16 @@ public boolean isRandomGrouping() { - return getGroupingTypeId().intValue() == RANDOM_GROUPING_TYPE.intValue(); + return getGroupingTypeId() == RANDOM_GROUPING_TYPE; } public boolean isChosenGrouping() { - return getGroupingTypeId().intValue() == CHOSEN_GROUPING_TYPE.intValue(); + return getGroupingTypeId() == CHOSEN_GROUPING_TYPE; } public boolean isClassGrouping() { - return getGroupingTypeId().intValue() == CLASS_GROUPING_TYPE.intValue(); + return getGroupingTypeId() == CLASS_GROUPING_TYPE; } }