Toggle catalog mode in Prestashop based on user login status: logged in or unlogged

Question

Toggle catalog mode in Prestashop based on user login status: logged in or unlogged

I am currently developing a PrestaShop module that allows users to switch the catalog mode on or off depending on whether they are logged in or not.

Everything is working well, but I have encountered an issue.

I want unlogged users to not be able to see prices or make orders. However, with the current solution I have implemented, when an unlogged user first loads the page (with the catalog mode off), the catalog mode turns on and they can see prices (they have to reload the page to hide the prices). So, the first load sets the catalog mode on and the second load displays the real catalog mode.

I have found a JavaScript script that auto-reloads the page to apply the new mode, but this results in the page loading time being twice as long.

Here is the function:

  public function hookHeader()
    {
      $logged = $this->context->customer->isLogged();
      if (!$logged) {
        Configuration::updateValue('PS_CATALOG_MODE', true);
      } else {
        Configuration::updateValue('PS_CATALOG_MODE', false);
      }
      // reload the page once more
      echo '
        <script type="text/javascript">
          (function() {
            if( window.localStorage ) {
              if( !localStorage.getItem( "firstLoad" ) ) {
                localStorage[ "firstLoad" ] = true;
                window.location.reload();
              } else {
                localStorage.removeItem( "firstLoad" );
              }
            }
          })();
        </script>
      ';
    }

I hope someone can assist me with resolving this issue. Thank you.

javascript module prestashop prestashop-1.6

Answer 1

Answer №1

There seems to be an issue with your solution. By updating the value in the database directly, it can cause instability when multiple users are accessing the site simultaneously. The value may flip-flop between on and off states, creating an inconsistent experience for users.

A better approach would be to toggle the value on a per-customer basis. A suggested override for the Configuration class is provided, which checks if the customer is logged in and returns the appropriate value of 0 or 1. It's important to cache this value using static variables to avoid repetitive checks.

However, this solution also has its drawback as it checks the key of the request configuration variable each time.

An improved method would involve changing the value during the session itself. Configuration variables are stored in a PHP array during the session, allowing for direct manipulation at that level.

To implement this change, consider modifying the code at:

https://github.com/PrestaShop/PrestaShop/blob/1.6.1.x/classes/Configuration.php#L203

Possibly by overriding:

https://github.com/PrestaShop/PrestaShop/blob/1.6.1.x/classes/Configuration.php#L140

My suggestion for overriding loadConfiguration:

<?php

// placed in /override/classes/Configuration.php

class Configuration extends ConfigurationCore
{
    public static function loadConfiguration()
    {
        parent::loadConfiguration();
        // 'global' assumed since not running multishop
        self::$_cache[self::$definition['table']][0]['global']['PS_CATALOG_MODE'] = !Context::getContext()->customer->isLogged();
    }
}

Note that this was written from memory, so verify names and details. This is tailored for PS1.6 and later versions.

Answer 2

There seems to be an issue with your solution. By updating the value in the database directly, it can cause instability when multiple users are accessing the site simultaneously. The value may flip-flop between on and off states, creating an inconsistent experience for users.

A better approach would be to toggle the value on a per-customer basis. A suggested override for the Configuration class is provided, which checks if the customer is logged in and returns the appropriate value of 0 or 1. It's important to cache this value using static variables to avoid repetitive checks.

However, this solution also has its drawback as it checks the key of the request configuration variable each time.

An improved method would involve changing the value during the session itself. Configuration variables are stored in a PHP array during the session, allowing for direct manipulation at that level.

To implement this change, consider modifying the code at:

https://github.com/PrestaShop/PrestaShop/blob/1.6.1.x/classes/Configuration.php#L203

Possibly by overriding:

https://github.com/PrestaShop/PrestaShop/blob/1.6.1.x/classes/Configuration.php#L140

My suggestion for overriding loadConfiguration:

<?php

// placed in /override/classes/Configuration.php

class Configuration extends ConfigurationCore
{
    public static function loadConfiguration()
    {
        parent::loadConfiguration();
        // 'global' assumed since not running multishop
        self::$_cache[self::$definition['table']][0]['global']['PS_CATALOG_MODE'] = !Context::getContext()->customer->isLogged();
    }
}

Note that this was written from memory, so verify names and details. This is tailored for PS1.6 and later versions.

Answer 3

Answer №2

Have you considered utilizing the group settings feature? With customer group settings, you have the ability to adjust the visibility of prices by setting it to "false" for visitors and "true" for customers.

Answer 4

Have you considered utilizing the group settings feature? With customer group settings, you have the ability to adjust the visibility of prices by setting it to "false" for visitors and "true" for customers.

Answer 5

Answer №3

To address the issue at hand, gskema proposes a solution that involves overriding the get() function within the Configuration class:

<?php

// This code should be placed in /override/classes/Configuration.php

class Configuration extends ConfigurationCore
{
  public static function get($key, $id_lang = null, $id_shop_group = null, $id_shop = null)
  {
      if (defined('_PS_DO_NOT_LOAD_CONFIGURATION_') && _PS_DO_NOT_LOAD_CONFIGURATION_) {
          return false;
      }

      // Check if configuration is initialized, if not try manual query
      if (!isset(self::$_cache[self::$definition['table']])) {
          Configuration::loadConfiguration();
          if (!self::$_cache[self::$definition['table']]) {
              return Db::getInstance()->getValue('SELECT `value` FROM `'._DB_PREFIX_.bqSQL(self::$definition['table']).'` WHERE `name` = "'.pSQL($key).'"');
          }
      }
      $id_lang = (int)$id_lang;
      if ($id_shop === null || !Shop::isFeatureActive()) {
          $id_shop = Shop::getContextShopID(true);
      }
      if ($id_shop_group === null || !Shop::isFeatureActive()) {
          $id_shop_group = Shop::getContextShopGroupID(true);
      }

      if (!isset(self::$_cache[self::$definition['table']][$id_lang])) {
          $id_lang = 0;
      }

      if ($id_shop && Configuration::hasKey($key, $id_lang, null, $id_shop)) {
          if($key == 'PS_CATALOG_MODE' && Context::getContext()->controller->controller_type == 'front')
          {
              return !Context::getContext()->customer->isLogged() || self::$_cache[self::$definition['table']][$id_lang]['shop'][$id_shop][$key];
          } else {
              return self::$_cache[self::$definition['table']][$id_lang]['shop'][$id_shop][$key];
          }
      } elseif ($id_shop_group && Configuration::hasKey($key, $id_lang, $id_shop_group)) {
          if($key == 'PS_CATALOG_MODE' && Context::getContext()->controller->controller_type == 'front')
          {
              return !Context::getContext()->customer->isLogged() || self::$_cache[self::$definition['table']][$id_lang]['group'][$id_shop_group][$key];
          } else {
              return self::$_cache[self::$definition['table']][$id_lang]['group'][$id_shop_group][$key];
          }
      } elseif (Configuration::hasKey($key, $id_lang)) {
          if($key == 'PS_CATALOG_MODE' && Context::getContext()->controller->controller_type == 'front')
          {
              return !Context::getContext()->customer->isLogged() || self::$_cache[self::$definition['table']][$id_lang]['global'][$key];
          } else {
              return self::$_cache[self::$definition['table']][$id_lang]['global'][$key];
          }
      }
      return false;
  }
}

/!\ Note: The comparison of key values every time someone tries to retrieve a config variable may slightly slow down the shop.

EDIT

A condition has been added for when the Context is in the front office to resolve the back office issue regarding 'Call isLogged on NULL'.

Answer 6

To address the issue at hand, gskema proposes a solution that involves overriding the get() function within the Configuration class:

<?php

// This code should be placed in /override/classes/Configuration.php

class Configuration extends ConfigurationCore
{
  public static function get($key, $id_lang = null, $id_shop_group = null, $id_shop = null)
  {
      if (defined('_PS_DO_NOT_LOAD_CONFIGURATION_') && _PS_DO_NOT_LOAD_CONFIGURATION_) {
          return false;
      }

      // Check if configuration is initialized, if not try manual query
      if (!isset(self::$_cache[self::$definition['table']])) {
          Configuration::loadConfiguration();
          if (!self::$_cache[self::$definition['table']]) {
              return Db::getInstance()->getValue('SELECT `value` FROM `'._DB_PREFIX_.bqSQL(self::$definition['table']).'` WHERE `name` = "'.pSQL($key).'"');
          }
      }
      $id_lang = (int)$id_lang;
      if ($id_shop === null || !Shop::isFeatureActive()) {
          $id_shop = Shop::getContextShopID(true);
      }
      if ($id_shop_group === null || !Shop::isFeatureActive()) {
          $id_shop_group = Shop::getContextShopGroupID(true);
      }

      if (!isset(self::$_cache[self::$definition['table']][$id_lang])) {
          $id_lang = 0;
      }

      if ($id_shop && Configuration::hasKey($key, $id_lang, null, $id_shop)) {
          if($key == 'PS_CATALOG_MODE' && Context::getContext()->controller->controller_type == 'front')
          {
              return !Context::getContext()->customer->isLogged() || self::$_cache[self::$definition['table']][$id_lang]['shop'][$id_shop][$key];
          } else {
              return self::$_cache[self::$definition['table']][$id_lang]['shop'][$id_shop][$key];
          }
      } elseif ($id_shop_group && Configuration::hasKey($key, $id_lang, $id_shop_group)) {
          if($key == 'PS_CATALOG_MODE' && Context::getContext()->controller->controller_type == 'front')
          {
              return !Context::getContext()->customer->isLogged() || self::$_cache[self::$definition['table']][$id_lang]['group'][$id_shop_group][$key];
          } else {
              return self::$_cache[self::$definition['table']][$id_lang]['group'][$id_shop_group][$key];
          }
      } elseif (Configuration::hasKey($key, $id_lang)) {
          if($key == 'PS_CATALOG_MODE' && Context::getContext()->controller->controller_type == 'front')
          {
              return !Context::getContext()->customer->isLogged() || self::$_cache[self::$definition['table']][$id_lang]['global'][$key];
          } else {
              return self::$_cache[self::$definition['table']][$id_lang]['global'][$key];
          }
      }
      return false;
  }
}

/!\ Note: The comparison of key values every time someone tries to retrieve a config variable may slightly slow down the shop.

EDIT

A condition has been added for when the Context is in the front office to resolve the back office issue regarding 'Call isLogged on NULL'.

Toggle catalog mode in Prestashop based on user login status: logged in or unlogged

Answer №1

Answer №2

Answer №3

Similar questions

How can I invoke a personalized function in angularjs from HTML?

Using PHP to identify the origin of a JavaScript file on a webpage

The alignment of elements in the div seems to be slightly skewed

Setting the maximum width in TinyMce

Jstree's select_node function is failing to trigger

Shifting Angular Component Declarations to a New Location

Retrieve the value from every dynamically generated text box by its corresponding number

Toggle Jquery menu with a click of a button

Using Selenium with C# to find elements within a chart

Load JSON data in Javascript with the condition where the 'id' is equal to 'x'

Using AngularJS to filter JSON data

The toLowerCase method seems to be malfunctioning along with several other functions

Look for a regular expression that matches numbers ranging from 1 to 31, either with or without a leading

Adding an interactive element to a predetermined collection

The successful execution of $.ajax does not occur

Incorporate additional text to the downloads hyperlink using jquery

Using jQuery: in the event that $(this) or a different element loses focus, then

Images in Next.js are appearing as broken images on certain pages

I am getting NaN as the output from my function, but I am unsure of the reason behind

Add elements to an array following an AJAX request