Testing on xampp, v1.0.8.21, php 8.2
Understand that this module might not yet be updated for these versions but hoping you can provide a fix for this notice shown on checkout_confirmation:
Deprecated: mysqli::real_escape_string(): Passing null to parameter #1 ($string) of type string is deprecated
Stripe Payment
Deprecated: mysqli::real_escape_string() - Stripe Payment
-
heatherbell
- Senior Contributor
- Posts: 2540
- Joined: Mon Oct 07, 2019 4:39 am
- Phoenix Version:
- Contact:
Re: Deprecated: mysqli::real_escape_string()
Sorry, but I'll not provide support for PHP 8.1 + until core is updated for them. Hope you understand. It would just be too much of extra work for the over 40 Addons I'm maintaining
Public Phoenix Change Log Cheat Set on Google Sheets
https://docs.google.com/spreadsheets/d/ ... sp=sharing
Need Help?viewtopic.php?f=10&t=27
https://docs.google.com/spreadsheets/d/ ... sp=sharing
Need Help?viewtopic.php?f=10&t=27
-
raiwa
- Certified Developer
- Posts: 1640
- Joined: Sat Dec 21, 2019 8:08 am
- Phoenix Version: 1.1.0.6
- : Buy Me A Beverage
- Contact:
Re: Deprecated: mysqli::real_escape_string()
Hi Robert,
I was curious and had a look on this.
It seems the normalize_ value method where this is used is only called in the perform method.
The stripe module itself doesn't use it only the web hook. The webhook isn't used until card data is submitted and the webhook is triggered from the stripe response.
So I doubt that this error is from the stripe module, but I can be wrong.
Anyway I have no test store on PHP 8.2 and can't get for now to the bottom if it's really stripe.
The error is produced if a null data value is passed to $db->perform. Any values which may get a null value should be checked and set to '' (empty string).
Example:
$value?? '';
Hope this helps.
I was curious and had a look on this.
It seems the normalize_ value method where this is used is only called in the perform method.
The stripe module itself doesn't use it only the web hook. The webhook isn't used until card data is submitted and the webhook is triggered from the stripe response.
So I doubt that this error is from the stripe module, but I can be wrong.
Anyway I have no test store on PHP 8.2 and can't get for now to the bottom if it's really stripe.
The error is produced if a null data value is passed to $db->perform. Any values which may get a null value should be checked and set to '' (empty string).
Example:
$value?? '';
Hope this helps.
Public Phoenix Change Log Cheat Set on Google Sheets
https://docs.google.com/spreadsheets/d/ ... sp=sharing
Need Help?viewtopic.php?f=10&t=27
https://docs.google.com/spreadsheets/d/ ... sp=sharing
Need Help?viewtopic.php?f=10&t=27
-
raiwa
- Certified Developer
- Posts: 1640
- Joined: Sat Dec 21, 2019 8:08 am
- Phoenix Version: 1.1.0.6
- : Buy Me A Beverage
- Contact:
Re: Deprecated: mysqli::real_escape_string()
Many thanks for taking the time to look, that is appreciatedraiwa wrote:I was curious and had a look on this.
Hope this helps.
In process of updating a store to PHP 8.2 on xampp and at the stage of installing our shipping and payment modules.
The notice was associated with Stripe SCA but, of course, it could also be associated with our configurations which are still in progress.
It is only a notice. More important is that Stripe SCA seems to function fine
There is another notice on admin side:
Deprecated: Creation of dynamic property cm_account_stripe_sca_cards::$public_title is deprecated in C:\xampp\htdocs\website\includes\modules\content\account\cm_account_stripe_sca_cards.php on line 31
Many thanks again for your kind help and patience (as always!).
Best wishes to you and yours for 2024!
-
heatherbell
- Senior Contributor
- Posts: 2540
- Joined: Mon Oct 07, 2019 4:39 am
- Phoenix Version:
- Contact: