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

Open Beginner friendly
#239 5 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
72/100
Issue type
Bug
Clarity
Mostly clear
Activity status
Active
Tech stack
php
Domain
backend

Research direction

Start at psgdpr/psgdpr.php line 701 and inspect the sha1() call reached by the reported hook-management flow. Reproduce it on PrestaShop 9.1.0 with PHP 8.1 using the listed back-office and footer steps. Done means the flow no longer emits the deprecated null-to-string message.

Written by the indexing model from the issue text.

Description

Prerequisites
Describe the bug and add attachments

There is a deprecated message
Image

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

Dominant language
JavaScript
Stars
33
Forks
74
Avg merge
2d 19h
Merged PRs (30d)
1

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from PrestaShop/psgdpr

All issues in PrestaShop/psgdpr

Similar issues

More JavaScript issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.