• Resolved David July

    (@mountsutro)


    Two errors have been appearing daily in my server error log since updating to the latest version of SendPress. I am not having any problems and a test email sent without issue, but I wanted to bring it to your attention.

    [12-Nov-2013 02:53:03 UTC] PHP Warning:  Missing argument 1 for SendPress_Notifications_Manager::maybe_send_notification() in ../wp-content/plugins/sendpress/classes/class-sendpress-notifications-manager.php on line 32
    
    [12-Nov-2013 02:53:03 UTC] PHP Warning:  Missing argument 2 for SendPress_Notifications_Manager::maybe_send_notification() in ../wp-content/plugins/sendpress/classes/class-sendpress-notifications-manager.php on line 32

    Please let me know if you have any questions. Thank you!

    http://wordpress.org/plugins/sendpress/

Viewing 9 replies - 1 through 9 (of 9 total)
  • Plugin Author itdoug

    (@itdoug)

    We think that problem has been resolved. Let us know if you still see them in your logs. It’s possible we may have introduced something else as we work on adding post notifications.

    Thread Starter David July

    (@mountsutro)

    Thank you for your reply. The missing argument PHP warnings are no longer appearing in the account’s error log.

    [05-Jan-2014 19:50:21 UTC] PHP Warning:  Missing argument 2 for SendPress_Notifications_Manager::send_notification(), called in /wp-content/plugins/sendpress/classes/class-sendpress-notifications-manager.php on line 43 and defined in /wp-content/plugins/sendpress/classes/class-sendpress-notifications-manager.php on line 172
    
    [05-Jan-2014 19:50:21 UTC] PHP Warning:  Invalid argument supplied for foreach() in /wp-content/plugins/sendpress/classes/class-sendpress-notifications-manager.php on line 188

    This is being thrown about once a day for us. Any insight as to why (Or what I can do)?

    Running 0.9.7.1

    Plugin Author itdoug

    (@itdoug)

    I’m assuming that’s in your error log?

    We have a release going out this week that I think should fix it, but if not we can gather some more info and trace it down.

    I’m assuming that’s in your error log?

    We have a release going out this week that I think should fix it, but if not we can gather some more info and trace it down.

    Yessir.
    I took a quick look over at GitHub to see if you had pushed any changes, but it looked like the previous fix was the newest; and I didn’t see a beta branch or the like to comb through.

    So I figured I’d post.
    Thanks for the update. 🙂

    Plugin Author itdoug

    (@itdoug)

    We think we found the issue. We are going to release an update in the next week or so that should clear it up. It’s probably related to your version of PHP, but we will take care of it.

    We’re running PHP 5.4. Is there a setting optimization that’d help?

    Plugin Author itdoug

    (@itdoug)

    I spoke to the developers and they think they should be able to fix this to be more compatible in a future release. Are you still getting errors?

    Bleh, I thought I responded to this.

    I think I got it to stop by filling in the notification field.
    I had previously had it unchecked and not filled in.

    Up until that point it was still throwing errors about 2 weeks ago, yea.

Viewing 9 replies - 1 through 9 (of 9 total)
  • The topic ‘Errors Appearing in Log’ is closed to new replies.