From: Mikael Nordfeldth <mmn@hethane.se>
Date: Sun, 18 Jan 2015 01:44:55 +0000 (+0100)
Subject: Use HTTPClient to download avatar
X-Git-Url: https://git.mxchange.org/?a=commitdiff_plain;h=010824c4b5d38defede70075d2b54ff54631a17e;p=quix0rs-gnu-social.git

Use HTTPClient to download avatar

also make updateAvatar public so we can call it from update_ostatus_profile.php
---

diff --git a/plugins/OStatus/classes/Ostatus_profile.php b/plugins/OStatus/classes/Ostatus_profile.php
index b94950138f..9e5e7b64c0 100644
--- a/plugins/OStatus/classes/Ostatus_profile.php
+++ b/plugins/OStatus/classes/Ostatus_profile.php
@@ -1233,7 +1233,7 @@ class Ostatus_profile extends Managed_DataObject
      * @param string $url
      * @throws Exception in various failure cases
      */
-    protected function updateAvatar($url)
+    public function updateAvatar($url)
     {
         if ($url == $this->avatar) {
             // We've already got this one.
@@ -1262,10 +1262,15 @@ class Ostatus_profile extends Managed_DataObject
         // ripped from oauthstore.php (for old OMB client)
         $temp_filename = tempnam(sys_get_temp_dir(), 'listener_avatar');
         try {
-            if (!copy($url, $temp_filename)) {
+            $client = new HTTPClient();
+            $response = $client->get($url);
+
+            if (!$response->isOk()) {
                 // TRANS: Server exception. %s is a URL.
-                throw new ServerException(sprintf(_m('Unable to fetch avatar from %s.'), $url));
+                throw new ServerException(sprintf(_m('Unable to fetch avatar from %s.').':%s', $url, $response->getReasonPhrase()));
             }
+            // FIXME: make sure it's an image here instead of _after_ writing to a file?
+            file_put_contents($response->getBody(), $temp_filename);
 
             if ($this->isGroup()) {
                 $id = $this->group_id;