Okay
  Public Ticket #3522495
preg_replace(): Passing null to parameter #3 ($subject) of type array|string is deprecated
Closed

Comments

  •  8
    benoitfouc33 started the conversation

    Hello there, i've this error using PHP 8.1 and last version of Wordpress and Salient : 

    preg_replace(): Passing null to parameter #3 ($subject) of type array|string is deprecated-

    there is more information, it seems to be caused by the function wp_kses_post used on theme file "wp-content/themes/salient/includes/partials/header/header-menu.php"

    0ec66c08aa000f94a5ae3f3672729fc9.png

  •  987
    ThemeNectar replied

    Hey benoitfouc33,

    Thanks for reporting that. I'll be including the fix in the upcoming minor release due out this weekend.

    Kind regards

  •  8
    benoitfouc33 replied

    Hello here ! yes i confirm that the last update fix this issue BUT that create a new issue, only with the last plugin version, we can't dismiss the banner. sad error . I attach a video


    The link you followed has expired.

    Please try again.


    regards

    Attached files:  5063467bd3c20824da1fa15720efe4d6.mp4

  •  987
    ThemeNectar replied

    Hey benoitfouc33,

    I just tried to replicate that, and it seems that it only will occur when you try and dismiss the notice while in the WooCommerce settings admin area. It doesn't seem to be related to anything in the recent release of Salient. 

    Have you tried to dismiss the notice from the main dashboard in your WP admin yet?

    Kind regards

  •  8
    benoitfouc33 replied

    Hey, exatcly this error is when we use WooCommerce AND HPOS activated.

    I try on website who don't use WooCommerce, that wark working fine.

    I disable HPOS on my main website and i can dismiss the banner. So it's definitly a compatibility issue between Salient and WooCommerce's HPOS feature.

    regards

  •  8
    benoitfouc33 replied

    Hello there, how are you ? The issue still here when we use HPOS :

    c78c0b80d678a14e1bc514e3bd661325.gif

  •  8
    benoitfouc33 replied

    I'm able to replicate this issue on my stagging website !

    c64e7e1195294afabea872b0702b5c55.png

  •  8,427
    Tahir replied

    Adding more priority to this Ticket.

    Best 


    ThemeNectar Support Team 

  •  987
    ThemeNectar replied

    Hey benoitfouc33,

    I have also been able to replicate the conflict and will be including the fix for it in the next minor release.

    Kind regards

  •  8
    benoitfouc33 replied

    Hello there, thank you for replying :)

    can i try the beta of the next update of Salient please ? you made a lot of fix following my tickets

    regards

  •   ThemeNectar replied privately
  •   benoitfouc33 replied privately
  •  987
    ThemeNectar replied

    Hey  benoitfouc33, after installing the pre-release version, you would need to delete and reinstall your Salient Core plugin to get access to the new screen reader text option within the Single Image: 

    1248697764.png

    Kind regards

  •  8
    benoitfouc33 replied

    Hello thank you for replying, i do the manipulation but still not appear.

    I'm using the 2.0.6 version of salient core, maybe it's not the good version ? 

    regards

  •   ThemeNectar replied privately
  •  8
    benoitfouc33 replied

    Hello there, it's perfeeeeeeeeeect thank you very much !

    Do you have an idea about the date of release of this next version of salient ?

  •  987
    ThemeNectar replied

    Hey benoitfouc33,

    There's no exact date yet, but likely sometime later this month we'll release the minor update which contains those changes.

    Kind regards