X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=src%2FModel%2FItemURI.php;h=1755552299741b6884060ea8e4452c75b4aac13d;hb=70674f2ecc9d34ba7eabbd15c06f85b469b27d0e;hp=020c468d23a81700bdd2c31e6db56ec2548bd3f6;hpb=8b7cb5d9efeab580c2592e0fbe301f7142b73a3d;p=friendica.git

diff --git a/src/Model/ItemURI.php b/src/Model/ItemURI.php
index 020c468d23..1755552299 100644
--- a/src/Model/ItemURI.php
+++ b/src/Model/ItemURI.php
@@ -1,6 +1,6 @@
 <?php
 /**
- * @copyright Copyright (C) 2010-2022, the Friendica project
+ * @copyright Copyright (C) 2010-2023, the Friendica project
  *
  * @license GNU AGPL version 3 or any later version
  *
@@ -21,8 +21,10 @@
 
 namespace Friendica\Model;
 
+use Friendica\Core\Logger;
 use Friendica\Database\Database;
 use Friendica\Database\DBA;
+use Friendica\DI;
 
 class ItemURI
 {
@@ -35,17 +37,16 @@ class ItemURI
 	 */
 	public static function insert(array $fields)
 	{
-		// If the URI gets too long we only take the first parts and hope for best
-		$uri = substr($fields['uri'], 0, 255);
+		$fields = DI::dbaDefinition()->truncateFieldsForTable('item-uri', $fields);
 
-		if (!DBA::exists('item-uri', ['uri' => $uri])) {
-			DBA::insert('item-uri', $fields, Database::INSERT_UPDATE);
+		if (!DBA::exists('item-uri', ['uri' => $fields['uri']])) {
+			DBA::insert('item-uri', $fields, Database::INSERT_IGNORE);
 		}
 
-		$itemuri = DBA::selectFirst('item-uri', ['id', 'guid'], ['uri' => $uri]);
-
+		$itemuri = DBA::selectFirst('item-uri', ['id', 'guid'], ['uri' => $fields['uri']]);
 		if (!DBA::isResult($itemuri)) {
 			// This shouldn't happen
+			Logger::warning('Item-uri not found', $fields);
 			return null;
 		}
 
@@ -60,44 +61,34 @@ class ItemURI
 	 * Searched for an id of a given uri. Adds it, if not existing yet.
 	 *
 	 * @param string $uri
+	 * @param bool   $insert
+	 *
 	 * @return integer item-uri id
+	 *
 	 * @throws \Exception
 	 */
-	public static function getIdByURI(string $uri): int
+	public static function getIdByURI(string $uri, bool $insert = true): int
 	{
 		if (empty($uri)) {
 			return 0;
 		}
 
-		// If the URI gets too long we only take the first parts and hope for best
-		$uri = substr($uri, 0, 255);
-
 		$itemuri = DBA::selectFirst('item-uri', ['id'], ['uri' => $uri]);
 
-		if (!DBA::isResult($itemuri)) {
+		if (!DBA::isResult($itemuri) && $insert) {
 			return self::insert(['uri' => $uri]);
 		}
 
-		return $itemuri['id'];
+		return $itemuri['id'] ?? 0;
 	}
+
 	/**
-	 * Searched for an id of a given guid.
-	 *
-	 * @param string $guid
-	 * @return integer item-uri id
+	 * @param int $uriId
+	 * @return bool
 	 * @throws \Exception
 	 */
-	public static function getIdByGUID(string $guid): int
+	public static function exists(int $uriId): bool
 	{
-		// If the GUID gets too long we only take the first parts and hope for best
-		$guid = substr($guid, 0, 255);
-
-		$itemuri = DBA::selectFirst('item-uri', ['id'], ['guid' => $guid]);
-
-		if (!DBA::isResult($itemuri)) {
-			return 0;
-		}
-
-		return $itemuri['id'];
+		return DBA::exists('item-uri', ['id' => $uriId]);
 	}
 }