Changeset 733

Timestamp:

Jul 27, 2010, 4:08:14 PM (12 years ago)

Author:

duh

Message:

• resolved issue #153: sample page eventhandlers are bound in the wrong order

Location:

trunk

Files:

• grails-app/taglib/dbnp/studycapturing/WizardTagLib.groovy (modified) (1 diff)
• grails-app/views/wizard/pages/_samples.gsp (modified) (3 diffs)
• web-app/js/table-editor.js (modified) (3 diffs)

trunk/grails-app/taglib/dbnp/studycapturing/WizardTagLib.groovy

@@ -140 +140 @@
                                 "\$('" + attrs.get('form') + "')"
                         )
+                }
+
+                // change 'this' if a this attribute is preset
+                if (attrs.get('this')) {
+                        button = button.replace('this', attrs.get('this'))
                 }
 

trunk/grails-app/views/wizard/pages/_samples.gsp

@@ -29 +29 @@
         </span>
 
+        <script type="text/javascript">
+                function switchTemplate( element ) {
+                        <wizard:ajaxSubmitJs functionName="switchTemplate" this="element" url="[controller:'wizard',action:'pages']" update="[success:'wizardPage',failure:'wizardError']" afterSuccess="onWizardPage()"/>
+                }
+        </script>
+
         <g:if test="${samples}">
                 <g:if test="${samples.size() > samplesWithTemplate}">
@@ -46 +52 @@
                                                 <div class="firstColumn">${s+1}</div>
                                                 <div class="column">
-                                                        <wizard:templateSelect name="template_${s}" entity="${dbnp.studycapturing.Sample}" value="${sampleData['sample'].template}" addDummy="true" ajaxOnChange="switchTemplate" url="[controller:'wizard',action:'pages']" update="[success:'wizardPage',failure:'wizardError']" afterSuccess="onWizardPage()" />
+                                                        <wizard:templateSelect name="template_${s}" entity="${dbnp.studycapturing.Sample}" value="${sampleData['sample'].template}" addDummy="true" tableEditorChangeEvent="switchTemplate(element);" />
                                                 </div>
                                                 <wizard:templateColumns name="sample_${s}" class="column" id="1" entity="${sampleData.sample}"/>
@@ -75 +81 @@
                                                 <div class="firstColumn">${s+1}</div>
                                                 <div class="column">
-                                                        <wizard:templateSelect name="template_${s}" entity="${dbnp.studycapturing.Sample}" value="${sampleData['sample'].template}" addDummy="true" ajaxOnChange="switchTemplate" url="[controller:'wizard',action:'pages']" update="[success:'wizardPage',failure:'wizardError']" afterSuccess="onWizardPage()" />
+                                                        <wizard:templateSelect name="template_${s}" entity="${dbnp.studycapturing.Sample}" value="${sampleData['sample'].template}" addDummy="true" tableEditorChangeEvent="switchTemplate(element);" />
                                                 </div>
                                                 <wizard:templateColumns name="sample_${s}" class="column" id="1" entity="${sampleData.sample}"/>

trunk/web-app/js/table-editor.js

@@ -97 +97 @@
             // does this column contain an input field
             if (input && type) {
+                // check field type
                 switch (type) {
                     case 'text':
@@ -123 +124 @@
                         inputElement.bind('change.tableEditor', function() {
                             that.updateSingleInputElements(input, columnNumber, 'select');
+
+                            // probably we want to bind these extra event handlers
+                            // separately, but for now this will suffice :)
+                            that.handleExtraEvents( inputElement );
                         });
                         break;
@@ -211 +216 @@
                 break;
         }
+    },
+
+    /**
+     * execute extra functions when binding to a particular event. The extra change
+     * handlers are called after replicating template fields
+     * example: <select ... tableEditorChangeEvent="console.log(element);" ... />
+     * @param element
+     */
+    handleExtraEvents: function( element ) {
+        // define parameters
+        var events = ['change'];
+
+        // check if we need to execute some more event handlers
+        for ( var i=0; i < events.length; i++ ) {
+            var call = element.attr('tableEditor' + events[ i ].substr(0, 1).toUpperCase() + events[ i ].substr(1).toLowerCase() + 'Event');
+            if ( call ) {
+                // yes, execute!
+                eval( call );
+            }
+        }
     }
 }