Skip to content
Snippets Groups Projects
Verified Commit 282aed13 authored by Dave Long's avatar Dave Long
Browse files

Issue #3445909 by seanB, smustgrave, alexpott, catch: Add static caching to...

Issue #3445909 by seanB, smustgrave, alexpott, catch: Add static caching to LayoutTempstoreRepository

(cherry picked from commit 27c9bcca)
parent f5ecdb8f
No related branches found
No related tags found
10 merge requests!12802Issue #3537193 by opauwlo: Add enable absolute path option for CKEditor5 image uploads,!12745Fixed: Path alias language doesn't changes on changing of node language,!12684Issue #3220784,!12537Add ViewsConfigUpdater deprecation support for default_argument_skip_url,!12523Issue #3493858 by vidorado, xavier.masson, smustgrave: Extend ViewsBlockBase...,!122353526426-warning-for-missing,!11958Issue #3490507 by alexpott, smustgrave: Fix bogus mocking in...,!11769Issue #3517987: Add option to contextual filters to encode slashes in query parameter.,!11185Issue #3477324 by andypost, alexpott: Fix usage of str_getcsv() and fgetcsv() for PHP 8.4,!9944Issue #3483353: Consider making the createCopy config action optionally fail...
Pipeline #194004 canceled
......@@ -16,6 +16,13 @@ class LayoutTempstoreRepository implements LayoutTempstoreRepositoryInterface {
*/
protected $tempStoreFactory;
/**
* The static cache of loaded values.
*
* @var \Drupal\layout_builder\SectionStorageInterface[]
*/
protected array $cache = [];
/**
* LayoutTempstoreRepository constructor.
*
......@@ -31,6 +38,12 @@ public function __construct(SharedTempStoreFactory $temp_store_factory) {
*/
public function get(SectionStorageInterface $section_storage) {
$key = $this->getKey($section_storage);
// Check if the storage is present in the static cache.
if (isset($this->cache[$key])) {
return $this->cache[$key];
}
$tempstore = $this->getTempstore($section_storage)->get($key);
if (!empty($tempstore['section_storage'])) {
$storage_type = $section_storage->getStorageType();
......@@ -39,6 +52,9 @@ public function get(SectionStorageInterface $section_storage) {
if (!($section_storage instanceof SectionStorageInterface)) {
throw new \UnexpectedValueException(sprintf('The entry with storage type "%s" and ID "%s" is invalid', $storage_type, $key));
}
// Set the storage in the static cache.
$this->cache[$key] = $section_storage;
}
return $section_storage;
}
......@@ -48,6 +64,12 @@ public function get(SectionStorageInterface $section_storage) {
*/
public function has(SectionStorageInterface $section_storage) {
$key = $this->getKey($section_storage);
// Check if the storage is present in the static cache.
if (isset($this->cache[$key])) {
return TRUE;
}
$tempstore = $this->getTempstore($section_storage)->get($key);
return !empty($tempstore['section_storage']);
}
......@@ -58,6 +80,8 @@ public function has(SectionStorageInterface $section_storage) {
public function set(SectionStorageInterface $section_storage) {
$key = $this->getKey($section_storage);
$this->getTempstore($section_storage)->set($key, ['section_storage' => $section_storage]);
// Update the storage in the static cache.
$this->cache[$key] = $section_storage;
}
/**
......@@ -66,6 +90,8 @@ public function set(SectionStorageInterface $section_storage) {
public function delete(SectionStorageInterface $section_storage) {
$key = $this->getKey($section_storage);
$this->getTempstore($section_storage)->delete($key);
// Remove the storage from the static cache.
unset($this->cache[$key]);
}
/**
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment