[TASK] Implement internal cache in PageProvider

glutrot · May 19, 2015 · 2f6d737 · 2f6d737
1 parent ddbab1a
commit 2f6d737
Showing 1 changed file with 25 additions and 14 deletions.
diff --git a/Classes/Provider/PageProvider.php b/Classes/Provider/PageProvider.php
@@ -79,6 +79,11 @@ class PageProvider extends AbstractProvider implements ProviderInterface {
 	 */
 	protected $configurationService;
 
+	/**
+	 * @var array
+	 */
+	private static $cache = array();
+
 	/**
 	 * CONSTRUCTOR
 	 */
@@ -298,22 +303,28 @@ protected function setDefaultValuesInFieldsWithInheritedValues(Form $form, array
 	 * @return array
 	 */
 	protected function getInheritedConfiguration(array $row) {
-		$tree = $this->getInheritanceTree($row);
-		$data = array();
-		foreach ($tree as $branch) {
-			$provider = $this->configurationService->resolvePrimaryConfigurationProvider($this->tableName, self::FIELD_NAME_SUB, $branch);
-			$form = $provider->getForm($branch);
-			if (NULL === $form) {
-				return $data;
-			}
-			$fields = $form->getFields();
-			$values = $provider->getFlexFormValuesSingle($branch);
-			foreach ($fields as $field) {
-				$values = $this->unsetInheritedValues($field, $values);
+		$tableName = $this->getTableName($row);
+		$tableFieldName = $this->getFieldName($row);
+		$cacheKey = $tableName . $tableFieldName . $row['uid'];
+		if (TRUE === empty(self::$cache[$cacheKey])) {
+			$tree = $this->getInheritanceTree($row);
+			$data = array();
+			foreach ($tree as $branch) {
+				$provider = $this->configurationService->resolvePrimaryConfigurationProvider($this->tableName, self::FIELD_NAME_SUB, $branch);
+				$form = $provider->getForm($branch);
+				if (NULL === $form) {
+					break;
+				}
+				$fields = $form->getFields();
+				$values = $provider->getFlexFormValuesSingle($branch);
+				foreach ($fields as $field) {
+					$values = $this->unsetInheritedValues($field, $values);
+				}
+				$data = RecursiveArrayUtility::merge($data, $values);
 			}
-			$data = RecursiveArrayUtility::merge($data, $values);
+			self::$cache[$cacheKey] = $data;
 		}
-		return $data;
+		return self::$cache[$cacheKey];
 	}
 
 	/**