Changeset 457 for trunk/grails-app/taglib

Timestamp:

Mar 26, 2010, 2:22:04 AM (15 years ago)

Author:

gav

Message:

Refactor asset tree generation to improve speed by around 4 fold.

File:

• trunk/grails-app/taglib/AssetTreeTagLib.groovy (modified) (3 diffs)

trunk/grails-app/taglib/AssetTreeTagLib.groovy

@@ -7 +7 @@
 
     def js = new JsUtilService()
-    def ts = new AssetTreeService()
+    def assetTreeService
 
     /**
@@ -20 +20 @@
     */
     def assetTreeButton = { attrs ->
-        def mkp = new groovy.xml.MarkupBuilder(out) //this line will be unnecessary in versions of Grails after version 1.2
-
-        mkp.div( class: ts.buttonHtmlClass() ) {
-            a( href: ts.hrefShowPane() ) {
-                img(src: ts.treeRootImg())
-            }
-        } // mkp
+        out << assetTreeService.buildAssetTreeButton(attrs)
     } // assetTreeButton
 
@@ -33 +27 @@
     */
     def assetTreePane = { attrs ->
-        def mkp = new groovy.xml.MarkupBuilder(out) //this line will be unnecessary in versions of Grails after version 1.2
-
-        mkp.div(class: ts.paneHtmlClass(), id: ts.paneHtmlId(), style: 'display:none;') {
-            div(class: ts.paneCloseHtmlClass()) {
-                a( href: js.toggle(ts.paneHtmlId()) ) {
-                    img(src: ts.closeImg())
-                }
-            }
-
-            div(class: ts.tableDivHtmlClass()) {
-                table(id: ts.tableHtmlId()) {
-                    tr() {
-                        td(valign: 'top', class: 'value') {
-                            ul() {
-                                img(src: ts.treeRootImg(), id: ts.tableLoadingImgId(), alt: 'TreeRoot')
-                                li() {
-                                } // li
-                            } // ul
-                        } // td
-                    } // tr
-                } // table
-            } // div
-        } // mkp
-    } // assetTreePane
+        out << assetTreeService.buildAssetTreePane(attrs)
+    }
 
 } // end class