X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~yarrgweb/git?p=ypp-sc-tools.db-live.git;a=blobdiff_plain;f=yarrg%2Fweb%2Ftabsort;h=45d5e69e3763f56cbd2a1e1b888ad26261094116;hp=ad25f86b9ab981d58ed2e740cd91cc9d7d6ba23e;hb=872e647cac241531e4599c8b8b330d8772020253;hpb=2b6a88303ee81419e9583c4fc34e71f4e692a2e0

diff --git a/yarrg/web/tabsort b/yarrg/web/tabsort
index ad25f86..45d5e69 100644
--- a/yarrg/web/tabsort
+++ b/yarrg/web/tabsort
@@ -39,17 +39,24 @@
 $table => 'ts_table'
 $sortkeys => 'ts_sortkeys'
 $throw => undef
+$tbrow => undef
+$rowclass => undef
 $cols
 </%args>
 
 <%doc>
+	Numeric
+	SortKey
+	MapFn
 	NoSort
 	DoReverse
 </%doc>
 
 <&| script &>
 
-%	my $sortfn= "ts_sort__$table";
+% print $m->content();
+
+%	my $sortfn= "ts_s_$table";
 function <% $sortfn %>(compar) {
   debug('sorting compar='+compar);
   var table= document.getElementById('<% $table %>');
@@ -57,17 +64,44 @@ function <% $sortfn %>(compar) {
   var tbody= firstrow.parentNode
   var rows= tbody.childNodes
   var newrows= new Array;
-  for (var rowix=0; rowix < rows.length; rowix++) {
-    var row= rows.item(rowix);
-    debug('process row '+rowix+' [[ '+row+' ]] id='+row.id)
-    if (!row.id) continue;
+  var finalrows= new Array;
+  var rowix= 0;
 %	if (defined $throw) {
-    if (row.id == '<% $throw %>') continue;
+  for (; rowix < rows.length; rowix++) {
+    var row= rows.item(rowix);
+    if (row.id == '<% $throw %>') break;
+    debug('skip row '+rowix+' [[ '+row+' ]] id='+row.id);
+  }
+  rowix++;
 %	}
-    if (row.tagName != 'TR') continue;
+  for (; rowix < rows.length; rowix++) {
+    var row= rows.item(rowix);
+%	if (defined $tbrow) {
+    if (row.id == '<% $tbrow %>') break;
+%	}
+    if (!row.id) { debug('noid row '+rowix+' [[ '+row+' ]]'); continue; }
+    if (row.tagName != 'TR') {
+      debug('no-tr row '+rowix+' tagName='+row.tagName+' [[ '+row+' ]]');
+      continue;
+    }
+    debug('process row '+rowix+' [[ '+row+' ]] id='+row.id);
     newrows.push(row);
   }
+  for (; rowix < rows.length; rowix++) {
+    var row= rows.item(rowix);
+    finalrows.push(row);
+    debug('final row '+rowix+' [[ '+row+' ]]');
+  }
   newrows.sort(compar);
+%	if (defined $rowclass) {
+  for (var rowix=0; rowix < newrows.length; rowix++) {
+    var row= newrows[rowix];
+    var classname= '<% $rowclass %>'+(rowix % 2);
+    debug('fix row '+rowix+' class '+classname);
+    row.className= classname;
+  }
+%	}
+  newrows= newrows.concat(finalrows);
   for (var rowix=0; rowix < newrows.length; rowix++) {
     var row= newrows[rowix];
     debug('add row '+rowix+' [[ '+row+' ]]');
@@ -81,9 +115,9 @@ function <% $sortfn %>(compar) {
 %	my $thhtml= '';
 %	next if $col->{NoSort};
 
-%	my $mapfn= "ts_compar${cix}_map__$table";
-function <% $mapfn %>(rowelement) {
-  var rowid = rowelement.id;
+%	my $mapfn= "ts_${cix}m_$table";
+function <% $mapfn %>(re) {
+  var rowid = re.id;
 %	if ($col->{SortKey}) {
   return <% $col->{SortKey} %>;
 %	} else {
@@ -96,24 +130,25 @@ function <% $mapfn %>(rowelement) {
 %	}
 }
 
-%	my $comparefn= "ts_compar${cix}_cmp0__$table";
+%	my $comparefn= "ts_${cix}c0_$table";
 function <% $comparefn %>(a,b) {
-  var a_key = <% $mapfn %>(a);
-  var b_key = <% $mapfn %>(b);
+  var ak = <% $mapfn %>(a);
+  var bk = <% $mapfn %>(b);
 %	if ($col->{Numeric}) {
-  return a_key - b_key
+  return ak - bk
 %	} else {
-  if (a_key < b_key) return -1;
-  if (a_key > b_key) return +1;
+  if (ak < bk) return -1;
+  if (ak > bk) return +1;
   return 0;
 %	}
 }
 
-%	foreach my $reverse (qw(0 1)) {
-%		my $tcomparefn= "ts_compar${cix}_cmp${reverse}__$table";
+%	foreach my $reverse (qw(1 0)) {
+%		my $tcomparefn= "ts_${cix}c${reverse}_$table";
 %		if ($reverse) {
 %			next unless $col->{DoReverse};
 function <% $tcomparefn %>(a,b) { return -<% $comparefn %>(a,b); }
+
 %		}
 %		$thhtml .= "<a href=\"javascript:$sortfn($tcomparefn)\">".
 %				($reverse ? '&or;' : '&and;'). '</a>';
@@ -136,4 +171,6 @@ function ts_onload__<% $table %>() {
     thlist.item(cix).innerHTML += ah;
   }
 }
+
+register_onload(ts_onload__<% $table %>);
 </&>