Attachment 95537 Details for Bug 22784 – Bug 22784: Do not display an empty table

[patch] Bug 22784: Do not display an empty table

Bug-22784-Do-not-display-an-empty-table.patch (text/plain), 1.24 KB, created by Séverine Queune on 2019-11-19 10:36:19 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Séverine Queune

Created: 2019-11-19 10:36:19 UTC

Size: 1.24 KB

patch

obsolete

>From a9f2c6dee4f30f8c9aac10a6cd942760d26181ac Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>Date: Mon, 18 Nov 2019 15:16:24 +0100
>Subject: [PATCH] Bug 22784: Do not display an empty table
>MIME-Version: 1.0
>Content-Type: text/plain; charset=UTF-8
>Content-Transfer-Encoding: 8bit
>
>This is terrible and highlight that the whole script must be rewrite.
>GetDistinctValues does not handle the "archived" flag (and we do not
>want to put our hands there), so let's hack that and plan to rewrite the
>whole script.
>
>Signed-off-by: SÃ©verine QUEUNE <severine.queune@bulac.fr>
>---
> suggestion/suggestion.pl | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/suggestion/suggestion.pl b/suggestion/suggestion.pl
>index 4c16f87..4d2b7d7 100755
>--- a/suggestion/suggestion.pl
>+++ b/suggestion/suggestion.pl
>@@ -372,7 +372,7 @@ if ($op=~/else/) {
>                             "suggestionscount"=>scalar(@$suggestions),             
>                             'suggestions_loop'=>$suggestions,
>                             'reasonsloop'     => $reasonsloop,
>-                            };
>+                            } if @$suggestions;
> 
>         delete $$suggestion_ref{$displayby} unless $definedvalue;
>     }
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 22784: 95499 | 95500 | 95501 | 95502 | 95510 | 95511 | 95512 | 95513 | 95533 | 95534 | 95535 | 95536 | 95537 | 95538 | 96900 | 96901 | 96902 | 96903 | 96904 | 96905 | 100806 | 100807 | 100808 | 100809 | 100810 | 100811 | 100854 | 100855 | 100856 | 100857 | 100858 | 100859 | 101948 | 102779 | 102780 | 102781 | 102782 | 102783 | 102784 | 102785 | 102786 | 102944