[freeside-commits] freeside/httemplate/elements select-table.html, 1.9, 1.10

Ivan,,, ivan at wavetail.420.am
Sat Apr 12 20:03:02 PDT 2008


Update of /home/cvs/cvsroot/freeside/httemplate/elements
In directory wavetail.420.am:/tmp/cvs-serv32345

Modified Files:
	select-table.html 
Log Message:
some stuff to allow select-table to be used as as edit/elements/edit.html m2* element: add js_only and html_only options, add id option to specify element id, add a kludge to onchange option

Index: select-table.html
===================================================================
RCS file: /home/cvs/cvsroot/freeside/httemplate/elements/select-table.html,v
retrieving revision 1.9
retrieving revision 1.10
diff -u -d -r1.9 -r1.10
--- select-table.html	2 Aug 2007 23:09:43 -0000	1.9
+++ select-table.html	13 Apr 2008 03:03:00 -0000	1.10
@@ -28,12 +28,16 @@
     'disable_empty'  => 0, # bool (implied by multiple)
     'debug'          => 0, #set true to enable
     'label_callback' => sub { my $record = shift; return "label"; },
+    'js_only'      => 0, #set true to return only the JS portions (i.e. nothing)
+    'html_only'    => 0, #set true to return only the HTML portions (no-op, i.e. return everything)
   )
 
 </%doc>
+% unless ( $opt{'js_only'} ) {
 
 <SELECT <% $opt{'multiple'} ? 'MULTIPLE' : '' %>
         NAME = "<% $opt{'element_name'} || $opt{'field'} || $key %>"
+        ID   = "<% $opt{'id'} || $key %>"
         <% $onchange %>
         <% $opt{'element_etc'} %>
 >
@@ -61,6 +65,7 @@
 
 </SELECT>
 
+%}
 <%init>
 
 my( %opt ) = @_;
@@ -68,11 +73,19 @@
 warn "elements/select-table.html: \n". Dumper(%opt)
   if exists $opt{debug} && $opt{debug};
 
-my $onchange = $opt{'onchange'}
-                 ? 'onChange="'. $opt{'onchange'}. '(this)"'
-                 : '';
+my $onchange = '';
+if ( $opt{'onchange'} ) {
+  $onchange = $opt{'onchange'};
+  $onchange .= '(this)' unless $onchange =~ /\(\w*\);?$/;
+  $onchange =~ s/\(what\);/\(this\);/g; #ugh, terrible hack.  all onchange
+                                        #callbacks should act the same
+  $onchange = 'onChange="'. $onchange. '"';
+}
 
-my $key = dbdef->table($opt{'table'})->primary_key; #? $opt{'primary_key'} ||
+my $dbdef_table = dbdef->table($opt{'table'})
+  or die "can't find dbdef for ". $opt{'table'}. " table\n";
+
+my $key = $dbdef_table->primary_key; #? $opt{'primary_key'} ||
 
 my $name_col = $opt{'name_col'};
 



More information about the freeside-commits mailing list