Index: lams_central/src/java/org/lamsfoundation/lams/authoring/ObjectExtractor.java =================================================================== diff -u -rc09edcc3e54b8a8324169e0877b057b3abb8b483 -r7ab40577a703a54fa2c1b20fe5d050656ec70795 --- lams_central/src/java/org/lamsfoundation/lams/authoring/ObjectExtractor.java (.../ObjectExtractor.java) (revision c09edcc3e54b8a8324169e0877b057b3abb8b483) +++ lams_central/src/java/org/lamsfoundation/lams/authoring/ObjectExtractor.java (.../ObjectExtractor.java) (revision 7ab40577a703a54fa2c1b20fe5d050656ec70795) @@ -596,7 +596,7 @@ } Long groupID = WDDXProcessor.convertToLong(groupDetails,WDDXTAGS.GROUP_ID); - // does it exist already? Check on the ID field as the UIID field may not have + // does it exist already? Check on the ID field first as the UIID field may not have // been set if this grouping was created on the back end (based on a runtime copy of the sequence) if ( groupID != null && grouping.getGroups() != null && grouping.getGroups().size() > 0 ) { Iterator iter = grouping.getGroups().iterator(); @@ -616,7 +616,12 @@ group.setGroupName(WDDXProcessor.convertToString(groupDetails,WDDXTAGS.GROUP_NAME)); group.setGrouping(grouping); group.setGroupUIID(groupUIID); - // group.setOrderId(WDDXProcessor.convertToInteger(groupDetails,WDDXTAGS.ORDER_ID)); + + if ( keyExists(groupDetails,WDDXTAGS.ORDER_ID) ) + group.setOrderId(WDDXProcessor.convertToInteger(groupDetails,WDDXTAGS.ORDER_ID)); + else + group.setOrderId(0); + return group; }