add contextvar magic

small js optimisation

core/utils/generic_helpers.py

@@ -1,12 +1,20 @@
 import datetime
+from contextvars import ContextVar
 
 from django.contrib.admin.models import CHANGE, LogEntry
 from django.contrib.contenttypes.models import ContentType
 
 from core.models import FinancialYear
 
 
+_current_financial_year = ContextVar("current_financial_year", default=None)
+
+
 def get_current_financial_year():
+    # FIXME: decide if to include this
+    if var := _current_financial_year.get():
+        return var
+
     y = FinancialYear.objects.filter(current=True)
     if y:
         current_financial_year = y.last().financial_year
@@ -27,6 +35,8 @@ def get_current_financial_year():
             # calendar year
             current_financial_year -= 1
 
+    _current_financial_year.set(current_financial_year)
+
     return current_financial_year
 
 

forecast/views/edit_forecast.py

@@ -471,7 +471,7 @@ class EditForecastView(
     def class_name(self):
         return "wide-table"
 
-    # FIXME: might be risky
+    # FIXME: Triple check this is safe
     @cached_property
     def cost_centre_details(self):
         cost_centre = CostCentre.objects.select_related("directorate__group").get(
@@ -532,7 +532,7 @@ def get_payroll_forecast_report(self):
 
         return data
 
-    @property
+    @cached_property
     def future_year_display(self):
         if self._future_year_display is None:
             current_year = get_current_financial_year()

front_end/src/Util.js

@@ -32,10 +32,11 @@ function getCookie(name) {
     return cookieValue;
 }
 
+const NumberFormat = new Intl.NumberFormat('en-GB'); 
+
 export const formatValue = (value) => {
-    let nfObject = new Intl.NumberFormat('en-GB'); 
     let pounds = Math.round(value)
-    return nfObject.format(pounds); 
+    return NumberFormat.format(pounds); 
 }
 
 /**