From 521320508659d3749647fe8266b5493fc3ce560f Mon Sep 17 00:00:00 2001
From: William Joye <wjoye@cfa.harvard.edu>
Date: Sat, 22 Feb 2020 14:44:03 -0500
Subject: add footprint filter / sort support

---
 ds9/library/fp.tcl | 42 ++++++++++++++++++++++--------------------
 1 file changed, 22 insertions(+), 20 deletions(-)

diff --git a/ds9/library/fp.tcl b/ds9/library/fp.tcl
index 219bc2d..51254a4 100644
--- a/ds9/library/fp.tcl
+++ b/ds9/library/fp.tcl
@@ -77,9 +77,22 @@ proc FPProcess {varname} {
 	puts stderr "FPProcess $varname"
     }
 
-    VOTParse $var(catdb) $var(token)
+    global tmpdb
+    set tmpdb ${varname}tmpdb
+
+    VOTParse $tmpdb $var(token)
     ARDone $varname
 
+    # concat regions
+    if {![TBLValidDB $tmpdb]} {
+	return
+    }
+    global $var(catdb)
+    if {![eval $var(proc,reg) $varname $tmpdb $var(catdb)]} {
+	Error [msgcat::mc {Internal Parse Error}]
+	return
+    }
+
     TBLSortMenu $varname
     FPTable $varname
     FPDialogUpdate $varname
@@ -117,29 +130,18 @@ proc FPTable {varname} {
 	}
     }
 
-    # concat regions
-    set var(tbldb) ${varname}tbldb
-    global $var(tbldb)
-    if {![eval $var(proc,reg) $varname $var(catdb) $var(tbldb)]} {
-	Error [msgcat::mc {Internal Parse Error}]
-	if {[info exists $var(tbldb)]} {
-	    unset $var(tbldb)
-	}
-	set var(tbldb) $var(catdb)
-    }
-
     if {$var(filter) == {} && $var(sort) == {}} {
-	;
+	set var(tbldb) $var(catdb)
     } else {
-	set var(tmpdb) ${varname}tmpdb
-	global $var(tmpdb)
-	if {![TBLFltSort $varname $var(tbldb) $var(tmpdb)]} {
+	set var(tbldb) ${varname}tbldb
+	global $var(tbldb)
+	if {![TBLFltSort $varname $var(catdb) $var(tbldb)]} {
 	    Error "[msgcat::mc {Unable to evaluate filter}] $var(filter)"
-	} else {
-	    unset $var(tbldb)
-	    array set $var(tbldb) [array get $var(tmpdb)]
+	    if {[info exists $var(tbldb)]} {
+		unset $var(tbldb)
+	    }
+	    set var(tbldb) $var(catdb)
 	}
-	unset $var(tmpdb)
     }
 
     global $var(tbldb)
-- 
cgit v0.12