Index: trunk/grails-app/controllers/ReportController.groovy
===================================================================
--- trunk/grails-app/controllers/ReportController.groovy	(revision 732)
+++ trunk/grails-app/controllers/ReportController.groovy	(revision 733)
@@ -191,13 +191,12 @@
     def equipmentRegisterOhs = {
 
+        params.calculateRegulatoryTaskCompletion = true
+        def result = assetReportService.getEquipmentRegister(params, RCU.getLocale(request))
+
         params.reportTitle = "Equipment Register OH&S"
         params.logoUrl = grailsApplication.mainContext.getResource('images/logo.png').getURL()
         params.currentUser = authService.currentUser
-
-        params.calculateRegulatoryTaskCompletion = true
-        params.startDateString = g.formatDate(format: "EEE, dd-MMM-yyyy", date: params.startDate)
-        params.endDateString = g.formatDate(format: "EEE, dd-MMM-yyyy", date: params.endDate)
-
-        def result = assetReportService.getEquipmentRegister(params, RCU.getLocale(request))
+        params.startDateString = result.startDateString
+        params.endDateString = result.endDateString
 
         if(!result.error) {
Index: trunk/grails-app/services/AssetReportService.groovy
===================================================================
--- trunk/grails-app/services/AssetReportService.groovy	(revision 732)
+++ trunk/grails-app/services/AssetReportService.groovy	(revision 733)
@@ -142,22 +142,15 @@
         result.site = result.section.site
 
-        // Start date.
-        if(params.startDate)
-            params.startDate = dateUtilService.makeDate(params.startDate_year, params.startDate_month, params.startDate_day)
-        else
-            params.startDate = dateUtilService.oneWeekAgo
-
-        // End date.
-        if(params.endDate)
-            params.endDate = dateUtilService.makeDate(params.endDate_year, params.endDate_month, params.endDate_day)
-        else
-            params.endDate = dateUtilService.today
-
+        result.startDate = params.startDate ?: dateUtilService.oneWeekAgo
+        result.endDate = params.endDate ?: dateUtilService.today
         // Auto swap date range.
-        if(params.startDate > params.endDate) {
-            def tempStartDate = params.startDate
-            params.startDate = params.endDate
-            params.endDate = tempStartDate
-        }
+        if(result.startDate > result.endDate) {
+            def tempStartDate = result.startDate
+            result.startDate = result.endDate
+            result.endDate = tempStartDate
+        }
+
+        result.startDateString = g.formatDate(format: "EEE, dd-MMM-yyyy", date: result.startDate)
+        result.endDateString = g.formatDate(format: "EEE, dd-MMM-yyyy", date: result.endDate)
 
         // Inner join used to return only attribTypes that are used by AssetSubItemExtendedAttributes.
@@ -234,6 +227,6 @@
 
             // Subquery namedParams.
-            namedParams.startDate = params.startDate
-            namedParams.endDate = params.endDate+1
+            namedParams.startDate = result.startDate
+            namedParams.endDate = result.endDate+1
 
             select 'new map(asset.name as assetName',