Fix spacing, have remove() and return JSON for custom-default-avatars

1 files changed, 40 insertions, 54 deletions

diff --git a/lib/avatar.php b/lib/avatar.php
index fa8fece080c..86be0ea2635 100644
--- a/lib/avatar.php
+++ b/lib/avatar.php
@@ -12,35 +12,31 @@
 
 class OC_Avatar {
 	/**
-         * @brief get the users avatar
-         * @param $user string which user to get the avatar for
-         * @param $size integer size in px of the avatar, defaults to 64
-         * @return \OC_Image containing the avatar
-        */
-        public static function get ($user, $size = 64) {
-		if ($user === false) {
-			return self::getDefaultAvatar($user, $size);
-		}
-
-                $view = new \OC\Files\View('/'.$user);
+	 * @brief get the users avatar
+	 * @param $user string which user to get the avatar for
+	 * @param $size integer size in px of the avatar, defaults to 64
+	 * @return mixed \OC_Image containing the avatar or false if there's no image
+	*/
+	public static function get ($user, $size = 64) {
+		$view = new \OC\Files\View('/'.$user);
 
-                if ($view->file_exists('avatar.jpg')) {
-                        $ext = 'jpg';
-                } elseif ($view->file_exists('avatar.png')) {
-                        $ext = 'png';
-                } else {
-                        return self::getDefaultAvatar($user, $size);
+		if ($view->file_exists('avatar.jpg')) {
+			$ext = 'jpg';
+		} elseif ($view->file_exists('avatar.png')) {
+			$ext = 'png';
+		} else {
+			return false;
                 }
 
-                $avatar = new OC_Image($view->file_get_contents('avatar.'.$ext));
-                $avatar->resize($size);
-                return $avatar;
-        }
+		$avatar = new OC_Image($view->file_get_contents('avatar.'.$ext));
+		$avatar->resize($size);
+		return $avatar;
+	}
 
 	/**
 	 * @brief sets the users avatar
 	 * @param $user string user to set the avatar for
-	 * @param $data mixed imagedata or path to set a new avatar, or false to delete the current avatar
+	 * @param $data mixed imagedata or path to set a new avatar
 	 * @throws Exception if the provided file is not a jpg or png image
 	 * @throws Exception if the provided image is not valid, or not a square
 	 * @return true on success
@@ -48,43 +44,33 @@ class OC_Avatar {
 	public static function set ($user, $data) {
 		$view = new \OC\Files\View('/'.$user);
 
-		if ($data === false) {
-			$view->unlink('avatar.jpg');
-			$view->unlink('avatar.png');
-			return true;
-		} else {
-			$img = new OC_Image($data);
-			$type = substr($img->mimeType(), -3);
-			if ($type === 'peg') { $type = 'jpg'; }
-			if ($type !== 'jpg' && $type !== 'png') {
-				$l = \OC_L10N::get('lib');
-				throw new \Exception($l->t("Unknown filetype"));
-			}
-
-			if (!( $img->valid() && ($img->height() === $img->width()) )) {
-				$l = \OC_L10N::get('lib');
-				throw new \Exception($l->t("Invalid image, or the provided image is not square"));
-			}
+		$img = new OC_Image($data);
+		$type = substr($img->mimeType(), -3);
+		if ($type === 'peg') { $type = 'jpg'; }
+		if ($type !== 'jpg' && $type !== 'png') {
+			$l = \OC_L10N::get('lib');
+			throw new \Exception($l->t("Unknown filetype"));
+		}
 
-			$view->unlink('avatar.jpg');
-			$view->unlink('avatar.png');
-			$view->file_put_contents('avatar.'.$type, $data);
-			return true;
+		if (!( $img->valid() && ($img->height() === $img->width()) )) {
+			$l = \OC_L10N::get('lib');
+			throw new \Exception($l->t("Invalid image, or the provided image is not square"));
 		}
+
+		$view->unlink('avatar.jpg');
+		$view->unlink('avatar.png');
+		$view->file_put_contents('avatar.'.$type, $data);
+		return true;
 	}
 
 	/**
-	 * @brief gets the default avatar
-	 * @brief $user string which user to get the avatar for
-	 * @param $size integer size of the avatar in px, defaults to 64
-	 * @return \OC_Image containing the default avatar
-	 * @todo use custom default images, when they arive
+	 * @brief remove the users avatar
+	 * @param $user string user to delete the avatar from
+	 * @return void
 	*/
-	public static function getDefaultAvatar ($user, $size = 64) {
-		// TODO
-		/*$default = new OC_Image(OC::$SERVERROOT."/core/img/defaultavatar.png");
-		$default->resize($size);
-		return $default;*/
-		return;
+	public static function remove ($user) {
+		$view = new \OC\Files\View('/'.$user);
+		$view->unlink('avatar.jpg');
+		$view->unlink('avatar.png');
 	}
 }