[BUG] Deprecated: sha1(): Passing null to parameter #1 ($string) of type string is deprecated in psgdpr/psgdpr.php on line 701

[Progi1984]

Prerequisites

• I understand and accept the project's code of conduct.
• I have already searched in existing issues and found no previous report of this bug.

Describe the bug and add attachments

There is a deprecated message

Expected behavior

There is not deprecated message

Steps to reproduce

1. Go to BO > Modules
2. Filter on the module psgdpr
3. Click on the "Configure" module
4. Click on the "Manage hooks" button
5. Click on the "Hook a module" button
6. Save
7. Go to FO
8. Go on the footer (

PrestaShop version(s) where the bug happened

9.1.0

PHP version(s) where the bug happened

8.1

Your company or customer's name goes here (if applicable).

@PrestaShopCorp

[gross-nvs]

Seems like it was fixed a while ago: 5a0915f

But the version 2.0.3 was never release somehow? Is is just a tag to make?

[lmeyer1]

Could you release a new version with this fix, please?
Thank you!

[Maofree]

Hi
now I get the same problem on prestashop 9.1.2

another type of bugfix is

'psgdpr_customer_token' => sha1((string) Context::getContext()->customer->secure_key),

[HartLarsson]

i did a small fix while i wait the new version out:

    /**
     * Allow to return the checkbox to display in modules
     *
     * @param array $params
     *
     * @return string html content to display
     */
    public function hookDisplayGDPRConsent($params)
    {
        // get id_lang
        $id_lang = Context::getContext()->language->id;

        if (!isset($params['id_module'])) {
            return '';
        }

        $id_module = (int) $params['id_module'];

        $active = GDPRConsent::getConsentActive($id_module);
        if ($active === false) {
            return '';
        }
        $message = GDPRConsent::getConsentMessage($id_module, $id_lang);

        $url = Context::getContext()->link->getModuleLink($this->name, 'FrontAjaxGdpr', [], true);

        $id_customer = Context::getContext()->customer->id;
        $id_guest = 0;
        if ($id_customer == null) {
            $id_guest = Context::getContext()->cart->id_guest;
            $id_customer = 0;
        }
        
        // FIX: if secure_key is null
        $customer_secure_key = Context::getContext()->customer->secure_key ?? '';
        
        $this->context->smarty->assign([
            'ps_version' => $this->ps_version,
            'psgdpr_id_guest' => $id_guest,
            'psgdpr_id_customer' => $id_customer,
            'psgdpr_customer_token' => sha1($customer_secure_key),
            'psgdpr_guest_token' => sha1('psgdpr' . $id_guest . $_SERVER['REMOTE_ADDR'] . date('Y-m-d')),
            'psgdpr_id_module' => $id_module,
            'psgdpr_consent_message' => $message,
            'psgdpr_front_controller' => $url,
        ]);

        return $this->fetch('module:' . $this->name . '/views/templates/hook/displayGDPRConsent.tpl');
    }