From: Bradley Taylor <hello@brad-taylor.co.uk>
Date: Wed, 10 Nov 2021 19:23:40 +0000 (+0000)
Subject: Flip logic for determining if $section variable is set.
X-Git-Url: https://vcs.fsf.org/?a=commitdiff_plain;h=64dc59a66da9a33055bd6bef4a9c2a032433d6b5;p=civicrm-core.git

Flip logic for determining if $section variable is set.

Previously empty() function calls were added in order to provide PHP8 compatiability,
however, the logic commited was back-to-front.

This broke the ability to display pie and bar charts for those reports which support them.
---

diff --git a/templates/CRM/Report/Form/Layout/Graph.tpl b/templates/CRM/Report/Form/Layout/Graph.tpl
index d3de9439ed..137753d1bd 100644
--- a/templates/CRM/Report/Form/Layout/Graph.tpl
+++ b/templates/CRM/Report/Form/Layout/Graph.tpl
@@ -20,7 +20,7 @@
 {/if}
 
 {if empty($printOnly)} {* NO print section starts *}
-  {if !empty($section)}
+  {if empty($section)}
     {include file="CRM/common/chart.tpl" divId="chart_$uniqueId"}
   {/if}
   {if !empty($chartData)}