From a2e618de7688e0d8f4672fc801e95e31682355e1 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Sim=C3=A3o=20Rodrigues?=
 <SARodrigues@users.noreply.github.com>
Date: Tue, 5 Sep 2023 11:12:35 +0100
Subject: [PATCH] Revert "FLAG-900: Undefined year causing an error in the TCL
 due to Fires widget in protected and some custom area dashboards"

---
 .../fires/tree-loss-fires-annual/index.js       | 17 ++++++++++-------
 services/analysis-cached.js                     |  4 +---
 2 files changed, 11 insertions(+), 10 deletions(-)

diff --git a/components/widgets/fires/tree-loss-fires-annual/index.js b/components/widgets/fires/tree-loss-fires-annual/index.js
index a1205fb6a8..fe00d018da 100644
--- a/components/widgets/fires/tree-loss-fires-annual/index.js
+++ b/components/widgets/fires/tree-loss-fires-annual/index.js
@@ -19,6 +19,9 @@ import {
 
 import getWidgetProps from './selectors';
 
+const MAX_YEAR = 2022;
+const MIN_YEAR = 2001;
+
 const getGlobalLocation = (params) => ({
   adm0: params.type === 'global' ? null : params.adm0,
   adm1: params.type === 'global' ? null : params.adm1,
@@ -635,11 +638,9 @@ export default {
   settings: {
     threshold: 30,
     ifl: 2000,
-    startYear: 2001,
-    endYear: 2022,
   },
   getData: (params = {}) => {
-    const { adm0, adm1, adm2, type, startYear, endYear } = params || {};
+    const { adm0, adm1, adm2, type } = params || {};
 
     const globalLocation = {
       adm0: type === 'global' ? null : adm0,
@@ -665,17 +666,19 @@ export default {
         };
       }
 
-      const { range: yearsRange } = getYearsRangeFromMinMax(startYear, endYear);
-
+      const { startYear, endYear, range } = getYearsRangeFromMinMax(
+        MIN_YEAR,
+        MAX_YEAR
+      );
       return {
         ...data,
         settings: {
           startYear,
           endYear,
-          yearsRange,
+          yearsRange: range,
         },
         options: {
-          years: yearsRange,
+          years: range,
         },
       };
     });
diff --git a/services/analysis-cached.js b/services/analysis-cached.js
index 299ee31820..f9bce06026 100644
--- a/services/analysis-cached.js
+++ b/services/analysis-cached.js
@@ -19,7 +19,7 @@ const SQL_QUERIES = {
   lossFires:
     'SELECT {select_location}, umd_tree_cover_loss__year, SUM(umd_tree_cover_loss__ha) AS umd_tree_cover_loss__ha, SUM(umd_tree_cover_loss_from_fires__ha) AS "umd_tree_cover_loss_from_fires__ha" FROM data {WHERE} GROUP BY umd_tree_cover_loss__year, {location} ORDER BY umd_tree_cover_loss__year, {location}',
   lossFiresOTF:
-    'SELECT umd_tree_cover_loss__year, sum(umd_tree_cover_loss__ha), sum(umd_tree_cover_loss_from_fires__ha) FROM data WHERE umd_tree_cover_loss__year >= {startYear} AND umd_tree_cover_loss__year <= {endYear} AND umd_tree_cover_density_2000__threshold >= {threshold} GROUP BY umd_tree_cover_loss__year',
+    'SELECT umd_tree_cover_loss__year, sum(umd_tree_cover_loss__ha), sum(umd_tree_cover_loss_from_fires__ha) FROM data WHERE umd_tree_cover_loss__year >= {startYear} AND umd_tree_cover_loss__year <= {endYear} AND umd_tree_cover_density_2000__threshold >= 30 GROUP BY umd_tree_cover_loss__year',
   emissions:
     'SELECT {select_location}, umd_tree_cover_loss__year, SUM("gfw_gross_emissions_co2e_all_gases__Mg") AS "gfw_gross_emissions_co2e_all_gases__Mg", SUM("gfw_full_extent_gross_emissions_non_CO2__Mg_CO2e") AS "gfw_gross_emissions_co2e_non_co2__Mg", SUM("gfw_full_extent_gross_emissions_CO2_only__Mg_CO2") AS "gfw_gross_emissions_co2e_co2_only__Mg" FROM data {WHERE} GROUP BY umd_tree_cover_loss__year, {location} ORDER BY umd_tree_cover_loss__year, {location}',
   emissionsLossOTF:
@@ -817,7 +817,6 @@ export const getLossFiresOTF = (params) => {
     geostore,
     startYear,
     endYear,
-    threshold,
   } = params || {};
 
   const geostoreId = geostore.id || adm0;
@@ -828,7 +827,6 @@ export const getLossFiresOTF = (params) => {
     `${urlBase + sql}&geostore_id=${geostoreId}`
       .replace('{startYear}', startYear)
       .replace('{endYear}', endYear)
-      .replace('{threshold}', threshold)
   );
 
   if (download) {