From b50b8e6cf427f6dabbfc83779dd860f2a288e3a7 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Mike=20F=C3=A4hrmann?= <mike_faehrmann@web.de>
Date: Thu, 13 May 2021 21:56:34 +0200
Subject: [PATCH] =?UTF-8?q?refactor=20applying=20'parent-=E2=80=A6'=20opti?=
 =?UTF-8?q?ons?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 gallery_dl/job.py | 37 ++++++++++++++++++-------------------
 1 file changed, 18 insertions(+), 19 deletions(-)

diff --git a/gallery_dl/job.py b/gallery_dl/job.py
index 8db6f646..3f93deaa 100644
--- a/gallery_dl/job.py
+++ b/gallery_dl/job.py
@@ -52,9 +52,6 @@ class Job():
                 extr.category = pextr.category
                 extr.subcategory = pextr.subcategory
 
-            # transfer parent directory
-            extr._parentdir = pextr._parentdir
-
             # reuse connection adapters
             extr.session.adapters = pextr.session.adapters
 
@@ -192,7 +189,7 @@ class Job():
 class DownloadJob(Job):
     """Download images into appropriate directory/filename locations"""
 
-    def __init__(self, url, parent=None, kwdict=None):
+    def __init__(self, url, parent=None):
         Job.__init__(self, url, parent)
         self.log = self.get_logger("download")
         self.blacklist = None
@@ -201,21 +198,9 @@ class DownloadJob(Job):
         self.hooks = ()
         self.downloaders = {}
         self.out = output.select()
+        self.visited = parent.visited if parent else set()
         self._skipcnt = 0
 
-        if parent:
-            self.visited = parent.visited
-            pfmt = parent.pathfmt
-            if pfmt and parent.extractor.config("parent-directory"):
-                self.extractor._parentdir = pfmt.directory
-            if parent.extractor.config("parent-metadata"):
-                if parent.kwdict:
-                    self.kwdict.update(parent.kwdict)
-                if kwdict:
-                    self.kwdict.update(kwdict)
-        else:
-            self.visited = set()
-
     def handle_url(self, url, kwdict):
         """Download the resource specified in 'url'"""
         hooks = self.hooks
@@ -307,8 +292,22 @@ class DownloadJob(Job):
                     extr = None
 
         if extr:
-            job = self.__class__(extr, self, kwdict)
-            if extr.config("parent-skip"):
+            job = self.__class__(extr, self)
+            pfmt = self.pathfmt
+            pextr = self.extractor
+
+            if pfmt and pextr.config("parent-directory"):
+                extr._parentdir = pfmt.directory
+            else:
+                extr._parentdir = pextr._parentdir
+
+            if pextr.config("parent-metadata"):
+                if self.kwdict:
+                    job.kwdict.update(self.kwdict)
+                if kwdict:
+                    job.kwdict.update(kwdict)
+
+            if pextr.config("parent-skip"):
                 job._skipcnt = self._skipcnt
                 self.status |= job.run()
                 self._skipcnt = job._skipcnt