From 40fe062851704b8018b36f0d65a52b264cc8d974 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Mike=20F=C3=A4hrmann?= <mike_faehrmann@web.de>
Date: Sat, 1 Feb 2020 01:44:21 +0100
Subject: [PATCH] [pixiv] fix user id for bookmarks API calls (closes #596)

---
 gallery_dl/extractor/pixiv.py | 8 ++++++--
 test/test_results.py          | 2 +-
 2 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/gallery_dl/extractor/pixiv.py b/gallery_dl/extractor/pixiv.py
index 2aa0684c..78b59d1c 100644
--- a/gallery_dl/extractor/pixiv.py
+++ b/gallery_dl/extractor/pixiv.py
@@ -27,7 +27,7 @@ class PixivExtractor(Extractor):
     def __init__(self, match):
         Extractor.__init__(self, match)
         self.api = PixivAppAPI(self)
-        self.user_id = -1
+        self.user_id = None
         self.load_ugoira = self.config("ugoira", True)
 
     def items(self):
@@ -241,6 +241,10 @@ class PixivFavoriteExtractor(PixivExtractor):
         ("https://www.pixiv.net/bookmark.php", {
             "url": "90c1715b07b0d1aad300bce256a0bc71f42540ba",
         }),
+        # own bookmarks with tag (#596)
+        ("https://www.pixiv.net/bookmark.php?tag=foobar", {
+            "count": 0,
+        }),
         # followed users (#515)
         ("https://www.pixiv.net/en/users/173530/following", {
             "pattern": PixivUserExtractor.pattern,
@@ -261,7 +265,7 @@ class PixivFavoriteExtractor(PixivExtractor):
 
         if query:
             self.query = text.parse_query(query)
-            uid = self.query.get("id", -1)
+            uid = self.query.get("id")
             if not uid:
                 self.subcategory = "bookmark"
             elif self.query.get("type") == "user":
diff --git a/test/test_results.py b/test/test_results.py
index 87720882..8d1a1e56 100644
--- a/test/test_results.py
+++ b/test/test_results.py
@@ -21,7 +21,7 @@ TRAVIS_SKIP = {
     "exhentai", "kissmanga", "mangafox", "dynastyscans", "nijie", "bobx",
     "archivedmoe", "archiveofsins", "thebarchive", "fireden", "4plebs",
     "sankaku", "idolcomplex", "mangahere", "readcomiconline", "mangadex",
-    "sankakucomplex", "warosu", "fuskator", "patreon",
+    "sankakucomplex", "warosu", "fuskator", "patreon", "komikcast",
 }
 
 # temporary issues, etc.