TypeError and Notice

  • Resolved Josh Journey

    (@ljosh)


    2 months ago

    Howdy, hoping all is well. Better Messages is fantastic and am thankful for the development! ??
    I’m using the latest version of Better Messages, latest version of WordPress, PHP 8.1.31. Read another support thread mentioning uninstalling and re-installing Better Messages which I did just in case before posting. I’m experiencing two errors. One is mildly annoying and the other is preventing messages from being sent.

    A PHP notice is visible on the following view:
    Better Messages > Settings > Integrations > Emojis > Emojis for Selector
    PHP Deprecated: uksort(): Returning bool from comparison function is deprecated, return an integer less than, equal to, or greater than zero in /home/site/adventuringtimes/wp-content/plugins/bp-better-messages/views/layout-settings.php on line 3415
    Screenshot: https://photos.app.goo.gl/qMBPnXj7HnUBQfw79

    When composing a message via site.com/pm#/new-conversation followed by typing a name it shows:
    TypeError: can’t assign to property “thread_id” on “
    PHP Deprecated: rtrim(): Passing null to parameter #1 ($string) of type string is deprecated in /home/site/adventuringtimes/wp-content/plugins/bp-better-messages/inc/functions.php on line 1089{\”result\”:\”new_thread\”}”: not an object
    Screenshot: https://photos.app.goo.gl/ThxREg6mNLvCGx919

    When clicking an existing conversation the message works just fine. This prevents all users from being able to create a new conversation from message view. When sending a message through the user profile it works just fine. It doesn’t matter if the user is friended or not, the same result occurs even with the same user (tested both scenarios). This error also shows in the error log.

    Warm regards,
    Josh

Viewing 4 replies - 1 through 4 (of 4 total)
  • Plugin Author wordplus

    (@wordplus)

    Hi there!

    I will check this php deprecations notices, but they should not break the plugin flow.

    The issue is you have deprecations configured to be displayed even in api response and that is what you need to reconfigure as that is not good behaviour and even not secure.

    Thanks!

    Thread Starter Josh Journey

    (@ljosh)

    I will check this php deprecations notices, but they should not break the plugin flow.

    Thank you. Indeed I discovered a plugin conflict when disabling all plugins and enabling one by one alongside Better Messages. But I’m happy to report the conflict with s2framework is resolved with the latest release of Better Messages and all errors are resolved! Much appreciated. ??

    he issue is you have deprecations configured to be displayed even in api response and that is what you need to reconfigure as that is not good behaviour and even not secure.

    I only displayed the error notices on my dev site. But I’ll look into showing for admin only and have error display hidden for actual site. Warm regards,
    Josh

    Plugin Author wordplus

    (@wordplus)

    Hi there!

    PHP Deprecated is clearly lower level then PHP errors or even warnings or notices, so I can only recommend to hide that from displaying at your website.

    Thanks!

    Thread Starter Josh Journey

    (@ljosh)

    I set errors to hidden. Thank you. ??

Viewing 4 replies - 1 through 4 (of 4 total)
  • You must be logged in to reply to this topic.