From: fabrixxm <fabrix.xm@gmail.com>
Date: Tue, 11 Dec 2018 19:01:54 +0000 (+0100)
Subject: Photo model: set edited and updated on update, fix exists(), docs
X-Git-Url: https://git.mxchange.org/?a=commitdiff_plain;h=15a3fc05d35ce957bc9663c8ed9819f1cf5c22c6;p=friendica.git

Photo model: set edited and updated on update, fix exists(), docs
---

diff --git a/src/Model/Photo.php b/src/Model/Photo.php
index 4942381a4c..1b30879456 100644
--- a/src/Model/Photo.php
+++ b/src/Model/Photo.php
@@ -148,22 +148,31 @@ class Photo extends BaseObject
 	 * @brief Check if photo with given resource id exists
 	 *
 	 * @param string  $resourceid  Resource ID of the photo
+	 * @param array   $conditions  Array of extra conditions. Optional
 	 *
 	 * @return boolean
 	 */
-	public static function exists($resourceid)
+	public static function exists($resourceid, array $conditions = [])
 	{
-		return DBA::count("photo", ["resource-id" => $resourceid]) > 0;
+		if (!is_null($resourceid)) {
+			$conditions["resource-id"] = $resourceid;
+		}
+		if (count($conditions) == 0) {
+			// no conditions defined. return false
+			return false;
+		}
+		return DBA::count("photo", $conditions) > 0;
 	}
 
+
 	/**
 	 * @brief Get Image object for given row id. null if row id does not exist
 	 *
-	 * @param integer  $id  Row id
+	 * @param array  $photo  Photo data. Needs at least 'id', 'type', 'backend-class', 'backend-ref'
 	 *
 	 * @return \Friendica\Object\Image
 	 */
-	public static function getImageForPhoto($photo)
+	public static function getImageForPhoto(array $photo)
 	{
 		$data = "";
 		if ($photo["backend-class"] == "") {
@@ -307,11 +316,21 @@ class Photo extends BaseObject
 		return $r;
 	}
 
-	public static function delete(array $conditions, $options = [])
+	/**
+	 * @brief Delete info from table and data from storage
+	 *
+	 * @param array  $conditions  Field condition(s)
+	 * @param array  $options     Options array, Optional
+	 *
+	 * @return boolean
+	 *
+	 * @see \Friendica\Database\DBA::delete
+	 */
+	public static function delete(array $conditions, array $options = [])
 	{
 		// get photo to delete data info
 		$photos = self::select(["backend-class","backend-ref"], $conditions);
-		
+
 		foreach($photos as $photo) {
 			$backend_class = (string)$photo["backend-class"];
 			if ($backend_class !== "") {
@@ -348,9 +367,10 @@ class Photo extends BaseObject
 					$fields["data"] = $img->asString();
 				}
 			}
+			$fields['updated'] = DateTimeFormat::utcNow();
 		}
 
-		$fields['updated'] = DateTimeFormat::utcNow();
+		$fields['edited'] = DateTimeFormat::utcNow();
 
 		return DBA::update("photo", $fields, $conditions);
 	}