Changeset 327 for trunk/grails-app/services

Timestamp:

Feb 8, 2010, 9:56:44 PM (15 years ago)

Author:

gav

Message:

Small fix to asset tree export, turns out row.last() does not handle blank strings in the list.

File:

• trunk/grails-app/services/CsvService.groovy (modified) (4 diffs)

trunk/grails-app/services/CsvService.groovy

@@ -354 +354 @@
                 assetSubItem.subItems.sort { p1, p2 -> p1.name.compareToIgnoreCase(p2.name) }.each() { assetSubItem4 ->
                     writeAssetSubItem4(assetSubItem4)
-                    row.remove(row.last())
-                    row.remove(row.last())
+                    row.removeRange(row.size()-2, row.size())
                 }
             }
@@ -371 +370 @@
                 assetSubItem.subItems.sort { p1, p2 -> p1.name.compareToIgnoreCase(p2.name) }.each() { assetSubItem3 ->
                     writeAssetSubItem3(assetSubItem3)
-                    row.remove(row.last())
-                    row.remove(row.last())
+                    row.removeRange(row.size()-2, row.size())
                 }
             }
@@ -388 +386 @@
                 assetSubItem.subItems.sort { p1, p2 -> p1.name.compareToIgnoreCase(p2.name) }.each() { assetSubItem2 ->
                     writeAssetSubItem2(assetSubItem2)
-                    row.remove(row.last())
-                    row.remove(row.last())
+                    row.removeRange(row.size()-2, row.size())
                 }
             }
@@ -413 +410 @@
                 asset.assetSubItems.each() { assetSubItem1 ->
                     writeAssetSubItem1(assetSubItem1)
-                    row.remove(row.last())
-                    row.remove(row.last())
+                    row.removeRange(row.size()-2, row.size())
                 }
             }