Error Exception

  • Resolved devdavi

    (@devdavi)


    7 months, 4 weeks ago

    Line 340 of /includes/admin.php could use an error check if the value is null. Currently if the plugin cannot fetch the session var, it throws an error. Suggested modification, change this:

    if ($_SESSION['settings_turned_of'] === false && ($apiActiveNotConfigured || $smtpActiveNotConfigured) ) { ?>

    to this:

    if (isset($_SESSION['settings_turned_of']) && $_SESSION['settings_turned_of'] === false && ($apiActiveNotConfigured || $smtpActiveNotConfigured)) { ?>
Viewing 5 replies - 1 through 5 (of 5 total)
  • Moderator Bet Hannon

    (@bethannon1)

    Just seconding the issue. We see this as warnings however:

    Warning: Undefined global variable $_SESSION in?/home/{redacted}.cloudwaysapps.com/khtnaqcrqp/public_html/wp-content/plugins/mailgun/includes/admin.php?on line?340

    Warning: Trying to access array offset on value of type null in?/home/{redacted}.cloudwaysapps.com/khtnaqcrqp/public_html/wp-content/plugins/mailgun/includes/admin.php?on line?340

    @devdavi’s reply fixed it for me.

    Moderator Bet Hannon

    (@bethannon1)

    That’s great, 352digital. I’m sure that will help the Mailgun team get it fixed.

    I’m running this plugin on over 85 sites, however, and making manual edits to the plugin on all those sites just isn’t practical.

    I am also seeing this issue, and with the following error when debug / debug display is set to true.

    Warning: Undefined global variable $_SESSION in /sites/cftetonvalley.org/files/wp-content/plugins/mailgun/includes/admin.php on line 340

    Warning: Trying to access array offset on value of type null in /sites/cftetonvalley.org/files/wp-content/plugins/mailgun/includes/admin.php on line 340

    I’m also seeing this, this was introduced in the latest version of the plugin (1.9.8). Please fix.

    • This reply was modified 7 months, 2 weeks ago by pauldejongnu.
Viewing 5 replies - 1 through 5 (of 5 total)
  • The topic ‘Error Exception’ is closed to new replies.