=== modified file 'dhis-2/dhis-api/src/main/java/org/hisp/dhis/dataset/DataSet.java' --- dhis-2/dhis-api/src/main/java/org/hisp/dhis/dataset/DataSet.java 2010-12-22 09:16:14 +0000 +++ dhis-2/dhis-api/src/main/java/org/hisp/dhis/dataset/DataSet.java 2011-01-06 13:41:19 +0000 @@ -148,6 +148,11 @@ return sections != null && sections.size() > 0; } + public boolean isMobile() + { + return mobile != null && mobile; + } + // ------------------------------------------------------------------------- // hashCode and equals // ------------------------------------------------------------------------- === modified file 'dhis-2/dhis-web/dhis-web-maintenance/dhis-web-maintenance-dataset/src/main/java/org/hisp/dhis/dataset/action/UpdateDataSetAction.java' --- dhis-2/dhis-web/dhis-web-maintenance/dhis-web-maintenance-dataset/src/main/java/org/hisp/dhis/dataset/action/UpdateDataSetAction.java 2010-12-22 04:05:46 +0000 +++ dhis-2/dhis-web/dhis-web-maintenance/dhis-web-maintenance-dataset/src/main/java/org/hisp/dhis/dataset/action/UpdateDataSetAction.java 2011-01-06 13:41:19 +0000 @@ -158,10 +158,12 @@ dataSet.setPeriodType( periodService.getPeriodTypeByClass( periodType.getClass() ) ); dataSet.setDataElements( dataElements ); dataSet.setMobile( mobile ); - if ( dataSet.getMobile() != null && dataSet.getMobile() ) + + if ( dataSet.isMobile() ) { - dataSet.setVersion( dataSet.getVersion() + 1 ); + dataSet.setVersion( dataSet.getVersion() + 1 ); // TODO we should check if anything is actually updated before bumping version and push this to service layer } + dataSetService.updateDataSet( dataSet ); return SUCCESS;