[Koha-patches] [PATCH] [SIGNED-OFF] Fix for Bug 6602 - Reports dictionary doesn't properly recognize text columns

Katrin Fischer Katrin.Fischer.83 at web.de
Sun Aug 14 16:04:21 CEST 2011


From: Owen Leonard <oleonard at myacpl.org>

Reports dictionary won't let you set a comparison for MEDIUMTEXT columns. To
test, choose a MEDIUMTEXT column like borrowers.surname in Step 3 of adding
a new dictionary definition. Before the patch, in Step 4 you would not see
any kind of form field for entering a comparison. After patching Step 4 will
offer you a field labeled "Search string matches."

Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
Fixes described problem with mediumtext columns.
---
 reports/dictionary.pl |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/reports/dictionary.pl b/reports/dictionary.pl
index 393c917..4f94d69 100755
--- a/reports/dictionary.pl
+++ b/reports/dictionary.pl
@@ -115,7 +115,7 @@ elsif ($phase eq 'New Term step 4'){
 		if ($type eq 'DATE' || $type eq 'DATETIME'){
 			$tmp_hash{'date'}=1;
 		}
-		if ($type eq 'TEXT'){
+		if ($type eq 'TEXT' || $type eq 'MEDIUMTEXT'){
 			$tmp_hash{'text'}=1;
 		}
 #		else {
-- 
1.7.4.1



More information about the Koha-patches mailing list