Changeset 681 for trunk/grails-app

Timestamp:

Oct 5, 2010, 2:24:33 AM (14 years ago)

Author:

gav

Message:

Improvements to assetRegister report, adding site and section to page header.

Location:

trunk/grails-app

Files:

• controllers/ReportController.groovy (modified) (1 diff)
• services/AssetReportService.groovy (modified) (2 diffs)
• views/appCore/start.gsp (modified) (1 diff)

trunk/grails-app/controllers/ReportController.groovy

@@ -161 +161 @@
         params.logoUrl = grailsApplication.mainContext.getResource('images/logo.png').getURL()
         params.currentUser = authService.currentUser
-        if(params.section.id == 'all')
-            params.section = "All"
-        else
-            params.section = Section.get(params.section.id.toLong())
 
         def dataModel = assetReportService.getAssetRegister(params, RCU.getLocale(request))

trunk/grails-app/services/AssetReportService.groovy

@@ -24 +24 @@
         def result = [:]
 
+        result.section = Section.get(params.section.id.toLong())
+        result.site = result.section.site
+
         // Inner join used to return only attribTypes that are used by AssetExtendedAttributes.
         // So the result is only asset extendedAttributeTypes.
@@ -44 +47 @@
                     'left join asset.assetExtendedAttributes as attrib',
                     'left join attrib.extendedAttributeType as attribT'
-            if(params.section instanceof Section) {
-                namedParams.section = params.section
-                where 'asset.section = :section'
-            }
+            where 'asset.section = :section'
+                    namedParams.section = result.section
             order 'by asset.name asc, attribT.name asc'
         }

trunk/grails-app/views/appCore/start.gsp

@@ -156 +156 @@
                                                 <g:select optionKey="id"
                                                                     from="${Section.findAllByIsActive(true).sort { p1, p2 -> p1.name.compareToIgnoreCase(p2.name) }}"
-                                                                    name="section.id"
-                                                                    noSelection="['all':/${g.message(code:'default.all.select.text')}/]">
+                                                                    name="section.id">
                                                 </g:select>
                                             </g:jasperReport>