[Pieforms-commit] SF.net SVN: pieforms: [106] pieforms-php5/trunk/src/pieform/renderers/table .php

[<ora...@us...>]

Revision: 106
          http://svn.sourceforge.net/pieforms/?rev=106&view=rev
Author:   oracleshinoda
Date:     2006-12-22 19:39:29 -0800 (Fri, 22 Dec 2006)

Log Message:
-----------
Made the tablerenderer work again with the changes made recently. It now also correctly sets the error class on elements have an error on them

Modified Paths:
--------------
    pieforms-php5/trunk/src/pieform/renderers/table.php

Modified: pieforms-php5/trunk/src/pieform/renderers/table.php
===================================================================
--- pieforms-php5/trunk/src/pieform/renderers/table.php	2006-12-23 03:36:11 UTC (rev 105)
+++ pieforms-php5/trunk/src/pieform/renderers/table.php	2006-12-23 03:39:29 UTC (rev 106)
@@ -99,7 +99,7 @@
 }
 
 function pieform_renderer_table_header() {
-    return "<table cellspacing=\"0\" border=\"0\"><tbody>\n";
+    return "<table cellspacing=\"0\"><tbody>\n";
 }
 
 function pieform_renderer_table_footer() {
@@ -108,21 +108,13 @@
 
 function pieform_renderer_table_messages_js($id) {
     $result = <<<EOF
-// Given a message and form element name, should set an error on the element
 function {$id}_set_error(message, element) {
-    {$id}_remove_error(element);
-    element += '_container';
-    // @todo set error class on input elements...
-    $(element).parentNode.insertBefore(TR({'id': '{$id}_error_' + element}, TD({'colspan': 2, 'class': 'errmsg'}, message)), $(element).nextSibling);
+    element = '{$id}_' + element + '_container';
+    var container = getFirstElementByTagAndClassName('TD', null, $(element));
+    addElementClass(container, 'error');
+    addElementClass(container.firstChild, 'error');
+    insertSiblingNodesAfter($(element), TR({'id': '{$id}_error_' + element}, TD({'colspan': 2, 'class': 'errmsg'}, message)));
 }
-// Given a form element name, should remove an error associated with it
-function {$id}_remove_error(element) {
-    element += '_container';
-    var elem = $('{$id}_error_' + element);
-    if (elem) {
-        removeElement(elem);
-    }
-}
 function {$id}_remove_all_errors() {
     forEach(getElementsByTagAndClassName('TD', 'errmsg', $('$id')), function(item) {
         removeElement(item.parentNode);
@@ -134,8 +126,8 @@
     if (elem) {
         swapDOM(elem, msg);
     }
-    else {
-        appendChildNodes($('{$id}_' + {$id}_btn + '_container').parentNode, msg);
+    else if ({$id}_btn) {
+        insertSiblingNodesAfter($('{$id}_' + {$id}_btn + '_container'), msg);
     }
 }
 function {$id}_remove_message() {
@@ -144,7 +136,6 @@
         removeElement(elem);
     }
 }
-    
 EOF;
     return $result;
 }


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.