[RTKO] default volatile on setupOpac

library/Class/DigitalResource.php

@@ -20,15 +20,16 @@
  */
 
 
-class Class_DigitalResource extends Class_Entity
+class Class_DigitalResource
 {
   use Trait_StormFileSystem;
 
+  protected $_config_provider = null;
   protected static ?Storm_Collection $_plugins = null;
   protected static $_instance;
   protected ?array $_batches = null;
 
-  public static function getInstance()
+  public static function getInstance(): self
   {
     if ( ! static::$_instance)
       static::$_instance = new static;
@@ -86,6 +87,20 @@ class Class_DigitalResource extends Class_Entity
   }
 
 
+  public function getConfigProvider()
+  {
+    return $this->_config_provider ?? (new Class_DigitalResource_ConfigProvider);
+  }
+
+
+  public function setConfigProvider($provider): self
+  {
+    $this->_config_provider = $provider;
+
+    return $this;
+  }
+
+
   public function bootstrap(Zend_Controller_Front $front_controller): self
   {
     $this->getPlugins()->eachDo(fn($config) => $config->registerFrontController($front_controller));
@@ -94,10 +109,10 @@ class Class_DigitalResource extends Class_Entity
   }
 
 
-  public function enablePluginWith(string $admin_var)
+  public function enablePluginWith(string $admin_var): ?Class_DigitalResource_Config
   {
     if ( ! $config = $this->getConfigWith($admin_var))
-      return $this;
+      return null;
 
     return $config->enablePlugin();
   }
@@ -212,19 +227,13 @@ class Class_DigitalResource extends Class_Entity
   }
 
 
-  public function hasRightAccessPlugin($plugin, $user): bool
+  public function hasRightAccessPlugin(string $plugin, Class_Users $user): bool
   {
-    if ( ! $this->isPluginDocType($plugin))
-      return false;
-
-    if ( ! $this->hasPermissionOn($plugin, $user))
-      return false;
-
-    return true;
+    return $this->isPluginDocType($plugin) && $this->hasPermissionOn($plugin, $user);
   }
 
 
-  public function hasPermissionOn($plugin, $user): bool
+  public function hasPermissionOn(string $plugin, Class_Users $user): bool
   {
     if ( ! $config = $this->configFor($plugin))
       return false;
@@ -422,14 +431,6 @@ class Class_DigitalResource extends Class_Entity
   }
 
 
-  protected function getConfigProvider()
-  {
-    return $this->get('ConfigProvider')
-      ? $this->get('ConfigProvider')
-      : new Class_DigitalResource_ConfigProvider;
-  }
-
-
   public function isAdminVarOn(string $plugin, string $name): bool
   {
     return Class_AdminVar::isModuleEnabled($this->withNamespace($plugin, $name));
@@ -454,12 +455,6 @@ class Class_DigitalResource extends Class_Entity
   }
 
 
-  public function getDocType($plugin)
-  {
-    return $plugin;
-  }
-
-
   protected function getPlugins(): Storm_Collection
   {
     if (static::$_plugins)
@@ -473,11 +468,8 @@ class Class_DigitalResource extends Class_Entity
   }
 
 
-  public function build($class_name, $config)
+  public function build(string $class_name, $config)
   {
-    if ($builder =  $this->get('Builder'))
-      return $builder->newInstance($class_name, $config);
-
     if (class_exists($class_name))
       return new $class_name($config);
 
@@ -503,7 +495,7 @@ class Class_DigitalResource extends Class_Entity
                                       if ($batch = $config->getBatch())
                                         $batches[$this->getBatchName($config->getName())] = $this->build($batch, $config);
 
-                                      foreach($config->getOtherBatches() as $name => $instance)
+                                      foreach ($config->getOtherBatches() as $name => $instance)
                                         $batches[$name] = $instance;
 
                                       return $batches;