Merge pull request #6222 from annando/long-uri
authorHypolite Petovan <hypolite@mrpetovan.com>
Mon, 3 Dec 2018 00:18:45 +0000 (19:18 -0500)
committerGitHub <noreply@github.com>
Mon, 3 Dec 2018 00:18:45 +0000 (19:18 -0500)
Avoid problems with too long URI (should only happen with some RSS feeds)

src/Model/ItemURI.php

index 559babb..452f7e5 100644 (file)
@@ -22,11 +22,14 @@ class ItemURI extends BaseObject
         */
        public static function insert($fields)
        {
-               if (!DBA::exists('item-uri', ['uri' => $fields['uri']])) {
+               // If the URI gets too long we only take the first parts and hope for best
+               $uri = substr($fields['uri'], 0, 255);
+
+               if (!DBA::exists('item-uri', ['uri' => $uri])) {
                        DBA::insert('item-uri', $fields, true);
                }
 
-               $itemuri = DBA::selectFirst('item-uri', ['id'], ['uri' => $fields['uri']]);
+               $itemuri = DBA::selectFirst('item-uri', ['id'], ['uri' => $uri]);
 
                if (!DBA::isResult($itemuri)) {
                        // This shouldn't happen
@@ -45,6 +48,9 @@ class ItemURI extends BaseObject
         */
        public static function getIdByURI($uri)
        {
+               // 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)) {