From 890b2cdc4e93ab64d9b8bdcdeaed8b325e48b6ae Mon Sep 17 00:00:00 2001
From: Damien De Paoli <ddp@depaoli.id.au>
Date: Fri, 8 Jan 2021 21:35:38 +1100
Subject: [PATCH] fixed BUG-14 (add/sel buttons not being disabled again when
 no selections)

---
 BUGs                          |  1 -
 templates/books_for_loan.html | 26 +++++++++++++++++++++-----
 2 files changed, 21 insertions(+), 6 deletions(-)

diff --git a/BUGs b/BUGs
index 467e718..b215146 100644
--- a/BUGs
+++ b/BUGs
@@ -1,5 +1,4 @@
 #### BUGS (next-15)
-BUG-14:		Add/Rem Sel buttons don't disable again if all selections are NULL
 
 ### DB/back-end
 BUG-2:		series does not deal with calcd_rating... 
diff --git a/templates/books_for_loan.html b/templates/books_for_loan.html
index 40e43c8..ea2ac78 100644
--- a/templates/books_for_loan.html
+++ b/templates/books_for_loan.html
@@ -1,8 +1,13 @@
     <script>
     function ActivateBFLRemSelButton()
 	{
-		$('#bfl-remsel-btn').removeClass('disabled')
-		$('#bfl-remsel-btn').attr('disabled',false)
+		if( $("#book_table input:checkbox:checked").length > 0 )
+		{
+			$('#bfl-remsel-btn').removeClass('disabled')
+			$('#bfl-remsel-btn').attr('disabled',false)
+		}
+		else
+			DeactivateBFLRemSelButton()
 	}
 	
     function DeactivateBFLRemSelButton()
@@ -52,14 +57,25 @@
 
     function ActivateBFLAddSelButton()
 	{
-		$('#bfl-addsel-btn').removeClass('disabled')
-		$('#bfl-addsel-btn').attr('disabled',false)
+		if( $("#addsel_table input:checkbox:checked").length > 0 )
+		{
+			$('#bfl-addsel-btn').removeClass('disabled')
+			$('#bfl-addsel-btn').attr('disabled',false)
+		}
+		else
+			DeactivateBFLAddSelButton()
+
 	}
 
     function DeactivateBFLAddSelButton()
 	{
 		$('#bfl-addsel-btn').addClass('disabled')
 		$('#bfl-addsel-btn').attr('disabled',true)
+	}
+
+    function RemoveBFLAddSelButton()
+	{
+		DeactivateBFLAddSelButton()
 		$('#bfl-addsel-btn').attr('style','display:none')
 	}
 
@@ -67,7 +83,7 @@
 	{
 		var params=$("#addsel_table :input").serializeArray()
 		$.post( "/add_books_to_loan/" + {{loan_id}}, params, function(data) { 
-			DeactivateBFLAddSelButton()
+			RemoveBFLAddSelButton()
 			DeactivateBFLRemSelButton()
 			$('#bfl-search-term').val('')
 			$('#bfl-new').html('')