[Plugin: WP SES] major issue

Viewing 2 replies - 1 through 2 (of 2 total)
  • Thread Starter cogmios

    (@cogmios)

    11 years, 9 months ago

    So this is about:

    $this->to = array_merge($this->to, $to);

    Apparently even though $this->to has been defined as an array() it is not recognized as an array.

    On array_merge (http://php.net/manual/en/function.array-merge.php) I read why: this behavior has been changed since php version 5, so i changed it to:

    $this->to = array_merge((array)$this->to, $to);

    and now it works 🙂

    POSSIBLY this should be changed in the original class

    p.s. I also looked through that code somewhat more and there are some more changes you would want to do e.g. do not use curl but instead use WP build-in get request, als use the WP options, maybe use classes instead… etc… I think im going to fork this one.

    Plugin Contributor Sylvain Deaure

    (@sylvaindeaure)

    11 years, 2 months ago

    Thanks for the Help, Cogmios.

    I’m back online and will release an update with the last version of the php-aws-ses class.
    Agree with the use of the build in WP request instead of curl.
    Was just easier to include the class without rewriting lot of it’s code.

    Any suggest is welcome, thanks !

Viewing 2 replies - 1 through 2 (of 2 total)
  • The topic ‘[Plugin: WP SES] major issue’ is closed to new replies.