From f1c2f6718d189026f00a741d906298bc4a6e880d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?St=C3=A9phane=20Gigandet?= <stephane@openfoodfacts.org>
Date: Tue, 25 Jan 2022 16:47:32 +0000
Subject: [PATCH] fix: show 'we need your help' message for ingredients
 analysis only when needed - fixes #6341

---
 lib/ProductOpener/Display.pm                                  | 4 ++--
 .../product/includes/ingredients_analysis_details.tt.html     | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/lib/ProductOpener/Display.pm b/lib/ProductOpener/Display.pm
index 3f7743ac7b079..6d734bc42bda3 100644
--- a/lib/ProductOpener/Display.pm
+++ b/lib/ProductOpener/Display.pm
@@ -10996,8 +10996,8 @@ sub display_ingredients_analysis_details($) {
 	my $unknown_ingredients_html = '';
 	my $unknown_ingredients_help_html = '';
 
-	if ($ingredients_text . $specific_ingredients =~ /unknown_ingredient/) {
-		$template_data_ref->{ingredients_text_comp} = 'unknown_ingredient';
+	if (($ingredients_text . $specific_ingredients) =~ /unknown_ingredient/) {
+		$template_data_ref->{unknown_ingredients} = 1;
 
 		$styles .= <<CSS
 .unknown_ingredient {
diff --git a/templates/web/pages/product/includes/ingredients_analysis_details.tt.html b/templates/web/pages/product/includes/ingredients_analysis_details.tt.html
index 3895c3278be98..08cec27447ef9 100755
--- a/templates/web/pages/product/includes/ingredients_analysis_details.tt.html
+++ b/templates/web/pages/product/includes/ingredients_analysis_details.tt.html
@@ -4,12 +4,12 @@
     <a id="ingredients_analysis_link" data-dropdown="ingredient_analysis_drop" aria-controls="ingredient_analysis_drop" aria-expanded="false">
         [% lang("ingredients_analysis_details") %] &raquo;
     </a>
-    [% IF ingredients_text_comp == 'unknown_ingredient' %]
+    [% IF unknown_ingredients %]
         <strong>[% lang("we_need_your_help") %]</strong>
     [% END %]
 </p>
 <div id="ingredient_analysis_drop" data-dropdown-content class="f-dropdown content large" aria-hidden="true" tabindex="-1">
-    [% IF ingredients_text_comp == 'unknown_ingredient' %]
+    [% IF unknown_ingredients %]
         <p class="unknown_ingredient">[% lang("some_unknown_ingredients") %]</p>
         <div class="callout panel">
             <h3>[% lang("we_need_your_help") %]</h3>