Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Updated JS Routing file #1

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Updated JS Routing file #1

wants to merge 1 commit into from

Conversation

github-actions[bot]
Copy link

Questions Answers
Branch? develop
Description? Updated JS Routing file
Type? improvement
Category? CO
BC breaks? no
Deprecations? no
Fixed ticket? N/A
How to test? N/A
Possible impacts? N/A

nesrineabdmouleh pushed a commit that referenced this pull request Jun 2, 2023
@uguranium
Fix Null 500 error
0eb6190 
Error fix for stripslashes(): Argument #1 ($string) must be of type string, null given. The parameter is null after migration. Please can we use the if condition and check if it's not null? Because if it is null gives a 500 error.
nesrineabdmouleh pushed a commit that referenced this pull request Jul 30, 2024
@olecorre
Update RequestSql.php
fe8361a 
fix error in PHP 8:  count(): Argument #1 ($value) must be of type Countable|array, string given
nesrineabdmouleh pushed a commit that referenced this pull request Jul 30, 2024
@kpodemski
Merge pull request PrestaShop#36082 from olecorre/patch-1
b06ac78 
fix error in PHP 8:  count(): Argument #1 ($value) must be of type Countable|array, string given
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@prestonBot