Update Localization.php in Middleware with "no" tag for estimate.

[bookstack] / app / Settings / SettingService.php

diff --git a/app/Settings/SettingService.php b/app/Settings/SettingService.php
index 489963aad7cde7b361ffdecfcb4e7d7edb4e3301..1c053b3848ea779d480adea01834dce0059d629d 100644 (file)
--- a/app/Settings/SettingService.php
+++ b/app/Settings/SettingService.php
@@ -4,10 +4,8 @@ use Illuminate\Contracts\Cache\Repository as Cache;
 
 /**
  * Class SettingService
 
 /**
  * Class SettingService

- *

  * The settings are a simple key-value database store.
  * The settings are a simple key-value database store.

- *
- * @package BookStack\Services
+ * For non-authenticated users, user settings are stored via the session instead.

  */
 class SettingService
 {
  */
 class SettingService
 {


@@ -52,6 +50,19 @@ class SettingService
         return $formatted;
     }
 
         return $formatted;
     }
 

+    /**
+     * Get a value from the session instead of the main store option.
+     * @param $key
+     * @param bool $default
+     * @return mixed
+     */
+    protected function getFromSession($key, $default = false)
+    {
+        $value = session()->get($key, $default);
+        $formatted = $this->formatValue($value, $default);
+        return $formatted;
+    }
+

     /**
      * Get a user-specific setting from the database or cache.
      * @param \BookStack\Auth\User $user
     /**
      * Get a user-specific setting from the database or cache.
      * @param \BookStack\Auth\User $user


@@ -62,7 +73,7 @@ class SettingService
     public function getUser($user, $key, $default = false)
     {
         if ($user->isDefault()) {
     public function getUser($user, $key, $default = false)
     {
         if ($user->isDefault()) {

-            return session()->get($key, $default);
+            return $this->getFromSession($key, $default);

         }
         return $this->get($this->userKey($user->id, $key), $default);
     }
         }
         return $this->get($this->userKey($user->id, $key), $default);
     }


@@ -87,12 +98,6 @@ class SettingService
      */
     protected function getValueFromStore($key, $default)
     {
      */
     protected function getValueFromStore($key, $default)
     {

-        // Check for an overriding value
-        $overrideValue = $this->getOverrideValue($key);
-        if ($overrideValue !== null) {
-            return $overrideValue;
-        }
-

         // Check the cache
         $cacheKey = $this->cachePrefix . $key;
         $cacheVal = $this->cache->get($cacheKey, null);
         // Check the cache
         $cacheKey = $this->cachePrefix . $key;
         $cacheVal = $this->cache->get($cacheKey, null);


@@ -244,20 +249,4 @@ class SettingService
     {
         return $this->setting->where('setting_key', '=', $key)->first();
     }
     {
         return $this->setting->where('setting_key', '=', $key)->first();
     }

-
-
-    /**
-     * Returns an override value for a setting based on certain app conditions.
-     * Used where certain configuration options overrule others.
-     * Returns null if no override value is available.
-     * @param $key
-     * @return bool|null
-     */
-    protected function getOverrideValue($key)
-    {
-        if ($key === 'registration-enabled' && config('auth.method') === 'ldap') {
-            return false;
-        }
-        return null;
-    }

 }
 }