Attempt to read property “ID” on null in Shortcode

  • Resolved mattf10

    (@mattf10)


    4 months, 3 weeks ago

    Plugin is filling my logs with errors:

    Attempt to read property "ID" on null in /wp-content/plugins/optinmonster/OMAPI/Shortcodes/Shortcode.php on line 243" while reading response header from upstream, client: ###.###.###.###, server: www.berkeleyfloristsupply.com, request: "POST /aboutus/directions/ HTTP/1.0", host: "www.berkeleyfloristsupply.com", referrer: "https://www.berkeleyfloristsupply.com/aboutus/directions/"

    The code referred to in the error is this:

    OMAPI_Rules::check_shortcode( $this->campaign, $this->post->ID )

    The problem is with the second parameter ‘$this->post->ID’, when the post doesn’t exist.

    This should be fixed by adding an ‘isset’ check for ‘$this->post’ before trying to access the ‘ID’ property.

    Thanks

    The page I need help with: [log in to see the link]

Viewing 3 replies - 1 through 3 (of 3 total)
  • PC

    (@pranayachaudhary)

    Hello @mattf10

    Thanks for reaching out.

    I hope you are doing great.

    I checked your account and see that you are on one of our paid plans. The forum support is only for our free plan users.

    We request you to kindly reach out to our support team directly on support [at] optinmonster [dot] com and they’d be able to assist you with the matter.

    Thread Starter mattf10

    (@mattf10)

    Thanks for the reply. I’m the webmaster and I don’t have anything to do with the subscription. In any case, I don’t need help with my site specifically since this is just a small issue with the code. It can be fixed with a one line code change by your developers.

    It may help your developers to know that the page with the problem has a WPForms contact form on it, as well as the subscription pop-up. Regardless, it’s good programming practice to check that a variable is set before trying to access it. Since that’s not being done here, it’s causing the error I reported.

    PC

    (@pranayachaudhary)

    Thanks for brining that up @mattf10 We really appreciate it.

    I will pass the information on to our developers.

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