only use media_uniq_path_idx on tube* that need it, more strict otherwise

Author: chapmanjacobd

library/mediadb/db_media.py

@@ -28,11 +28,18 @@ def create(args):
         );
         """
     )
-    try:
-        args.db.execute("CREATE UNIQUE INDEX IF NOT EXISTS media_uniq_path_idx ON media (playlists_id, path);")
-    except sqlite3.IntegrityError:
-        dedupe_rows(args, "media", ["rowid"], ["playlists_id", "path"])
-        args.db.execute("CREATE UNIQUE INDEX IF NOT EXISTS media_uniq_path_idx ON media (playlists_id, path);")
+    if args.action in ("tube-add", "gallery-add", "web-add", "site-add"):
+        try:
+            args.db.execute("CREATE UNIQUE INDEX IF NOT EXISTS media_uniq_path_idx ON media (playlists_id, path);")
+        except sqlite3.IntegrityError:
+            dedupe_rows(args, "media", ["rowid"], ["playlists_id", "path"])
+            args.db.execute("CREATE UNIQUE INDEX IF NOT EXISTS media_uniq_path_idx ON media (playlists_id, path);")
+    else:
+        try:
+            args.db.execute("CREATE UNIQUE INDEX IF NOT EXISTS idx_media_path ON media (path);")
+        except sqlite3.IntegrityError:
+            dedupe_rows(args, "media", ["rowid"], ["path"])
+            args.db.execute("CREATE UNIQUE INDEX IF NOT EXISTS idx_media_path ON media (path);")
 
 
 def exists(args, path) -> bool:

library/utils/arggroups.py

@@ -1078,7 +1078,12 @@ def extractor(parent_parser):
         help="Convert UTF-8 IRIs (RFC 3987) to ASCII percent-encoded URLs (RFC 1738)",
     )
     parser.add_argument(
-        "--no-extract", "--skip-extract", "--insert-only", action="store_true", help="Insert paths into media table"
+        "--no-extract",
+        "--skip-extract",
+        "--insert-only",
+        "-n",
+        action="store_true",
+        help="Insert paths into media table",
     )  # TODO: move to its own subcommand
     parser.add_argument(
         "--no-extract-playlists",

tests/playback/test_links_open.py

@@ -14,7 +14,7 @@ def test_links_open(mock_souffle):
     media = mock_souffle.call_args[0][1]
     assert len(media) == 3
     assert media == [
-        {"path": "https://site0", "category": "p1"},
-        {"path": "https://site1", "category": "p1"},
-        {"path": "https://site2", "category": "p1"},
+        {"path": "https://site0", "hostname": "site0", "category": "p1"},
+        {"path": "https://site1", "hostname": "site1", "category": "p1"},
+        {"path": "https://site2", "hostname": "site2", "category": "p1"},
     ]