From 59980ddbaf67d4b5acce9a13fbea9e6d6b87f803 Mon Sep 17 00:00:00 2001
From: LouisGac <louis.gac@limesurvey.org>
Date: Mon, 14 Aug 2017 12:08:12 +0200
Subject: [PATCH] Dev: added class variable cache for getFilesToLoad

---
 application/models/TemplateConfiguration.php | 34 ++++++++++++--------
 1 file changed, 20 insertions(+), 14 deletions(-)

diff --git a/application/models/TemplateConfiguration.php b/application/models/TemplateConfiguration.php
index dc505a8685f..868aad2fe98 100644
--- a/application/models/TemplateConfiguration.php
+++ b/application/models/TemplateConfiguration.php
@@ -59,6 +59,10 @@ class TemplateConfiguration extends TemplateConfig
     /** @var string $sTypeIcon the type of template for icon (core vs user)*/
     private $sTypeIcon;
 
+    /** @var array $aFilesToLoad cache for the method getFilesToLoad()*/
+    private $aFilesToLoad;
+
+
     /**
      * @return string the associated database table name
      */
@@ -365,28 +369,30 @@ public function getOptionPage()
      */
     protected function getFilesToLoad($oTemplate, $sType)
     {
+        if (empty($this->aFilesToLoad)){
+            $sField = 'files_'.$sType;
+            $jFiles = $oTemplate->$sField;
 
-        $sField = 'files_'.$sType;
-        $jFiles = $oTemplate->$sField;
+            $this->aFilesToLoad = array();
 
-        $aFiles = array();
+            if(!empty($jFiles)){
+                $oFiles = json_decode($jFiles);
+                foreach($oFiles as $action => $aFileList){
+                    if ($action == "add" || $action == "replace"){
 
-        if(!empty($jFiles)){
-            $oFiles = json_decode($jFiles);
-            foreach($oFiles as $action => $aFileList){
-                if ($action == "add" || $action == "replace"){
+                        // Specific inheritance of one of the value of the json array
+                        if ($aFileList[0] == 'inherit'){
+                            $aParentjFiles = (array) json_decode($oTemplate->getParentConfiguration->$sField);
+                            $aFileList = $aParentjFiles[$action];
+                        }
 
-                    // Specific inheritance of one of the value of the json array
-                    if ($aFileList[0] == 'inherit'){
-                        $aParentjFiles = (array) json_decode($oTemplate->getParentConfiguration->$sField);
-                        $aFileList = $aParentjFiles[$action];
+                        $this->aFilesToLoad = array_merge($this->aFilesToLoad, $aFileList);
                     }
-
-                    $aFiles = array_merge($aFiles, $aFileList);
                 }
             }
         }
-        return $aFiles;
+
+        return $this->aFilesToLoad;
     }
 
     /**