Avoid double calls to get_logger

Roberto Alsina · Roberto Alsina · commit 5e6e2cd661bd · 2018-03-08T17:08:59.000-03:00
diff --git a/nikola/plugins/command/auto/__init__.py b/nikola/plugins/command/auto/__init__.py
@@ -55,7 +55,7 @@
 import pkg_resources
 
 from nikola.plugin_categories import Command
-from nikola.utils import dns_sd, req_missing, get_logger, get_theme_path
+from nikola.utils import dns_sd, req_missing, get_theme_path
 LRJS_PATH = os.path.join(os.path.dirname(__file__), 'livereload.js')
 
 if sys.platform == 'win32':
@@ -66,7 +66,6 @@ class CommandAuto(Command):
     """Automatic rebuilds for Nikola."""
 
     name = "auto"
-    logger = None
     has_server = True
     doc_purpose = "builds and serves a site; automatically detects site changes, rebuilds, and optionally refreshes a browser"
     dns_sd = None
@@ -116,7 +115,6 @@ class CommandAuto(Command):
 
     def _execute(self, options, args):
         """Start the watcher."""
-        self.logger = get_logger('auto')
         self.sockets = []
         self.rebuild_queue = asyncio.Queue()
         self.last_rebuild = datetime.datetime.now()
diff --git a/nikola/plugins/command/check.py b/nikola/plugins/command/check.py
@@ -43,7 +43,6 @@
 import requests
 
 from nikola.plugin_categories import Command
-from nikola.utils import get_logger
 
 
 def _call_nikola_list(site, cache=None):
@@ -103,7 +102,6 @@ class CommandCheck(Command):
     """Check the generated site."""
 
     name = "check"
-    logger = None
 
     doc_usage = "[-v] (-l [--find-sources] [-r] | -f [--clean-files])"
     doc_purpose = "check links and files in the generated site"
@@ -158,7 +156,6 @@ class CommandCheck(Command):
 
     def _execute(self, options, args):
         """Check the generated site."""
-        self.logger = get_logger('check')
 
         if not options['links'] and not options['files'] and not options['clean']:
             print(self.help())
diff --git a/nikola/plugins/command/deploy.py b/nikola/plugins/command/deploy.py
@@ -37,7 +37,7 @@
 from blinker import signal
 
 from nikola.plugin_categories import Command
-from nikola.utils import get_logger, clean_before_deployment
+from nikola.utils import clean_before_deployment
 
 
 class CommandDeploy(Command):
@@ -48,11 +48,9 @@ class CommandDeploy(Command):
     doc_usage = "[preset [preset...]]"
     doc_purpose = "deploy the site"
     doc_description = "Deploy the site by executing deploy commands from the presets listed on the command line.  If no presets are specified, `default` is executed."
-    logger = None
 
     def _execute(self, command, args):
         """Execute the deploy command."""
-        self.logger = get_logger('deploy')
         # Get last successful deploy date
         timestamp_path = os.path.join(self.site.config['CACHE_FOLDER'], 'lastdeploy')
 
diff --git a/nikola/plugins/command/github_deploy.py b/nikola/plugins/command/github_deploy.py
@@ -32,7 +32,7 @@
 
 from nikola.plugin_categories import Command
 from nikola.plugins.command.check import real_scan_files
-from nikola.utils import get_logger, req_missing, clean_before_deployment
+from nikola.utils import req_missing, clean_before_deployment
 from nikola.__main__ import main
 from nikola import __version__
 
@@ -76,11 +76,9 @@ class CommandGitHubDeploy(Command):
             'help': 'Commit message (default: Nikola auto commit.)',
         },
     ]
-    logger = None
 
     def _execute(self, options, args):
         """Run the deployment."""
-        self.logger = get_logger(CommandGitHubDeploy.name)
 
         # Check if ghp-import is installed
         check_ghp_import_installed()
diff --git a/nikola/plugins/command/serve.py b/nikola/plugins/command/serve.py
@@ -46,7 +46,7 @@
 
 
 from nikola.plugin_categories import Command
-from nikola.utils import dns_sd, get_logger
+from nikola.utils import dns_sd
 
 
 class IPv6Server(HTTPServer):
@@ -61,7 +61,6 @@ class CommandServe(Command):
     name = "serve"
     doc_usage = "[options]"
     doc_purpose = "start the test webserver"
-    logger = None
     dns_sd = None
 
     cmd_options = (
@@ -120,7 +119,6 @@ def shutdown(self, signum=None, _frame=None):
 
     def _execute(self, options, args):
         """Start test server."""
-        self.logger = get_logger('serve')
         out_dir = self.site.config['OUTPUT_FOLDER']
         if not os.path.isdir(out_dir):
             self.logger.error("Missing '{0}' folder?".format(out_dir))
diff --git a/nikola/plugins/compile/ipynb.py b/nikola/plugins/compile/ipynb.py
@@ -42,7 +42,7 @@
 
 from nikola import shortcodes as sc
 from nikola.plugin_categories import PageCompiler
-from nikola.utils import makedirs, req_missing, get_logger, LocaleBorg
+from nikola.utils import makedirs, req_missing, LocaleBorg
 
 
 class CompileIPynb(PageCompiler):
@@ -54,11 +54,6 @@ class CompileIPynb(PageCompiler):
     default_kernel = 'python3'
     supports_metadata = True
 
-    def set_site(self, site):
-        """Set Nikola site."""
-        self.logger = get_logger('compile_ipynb')
-        super(CompileIPynb, self).set_site(site)
-
     def _compile_string(self, nb_json):
         """Export notebooks as HTML strings."""
         self._req_missing_ipynb()
diff --git a/nikola/plugins/task/bundles.py b/nikola/plugins/task/bundles.py
@@ -45,13 +45,12 @@ class BuildBundles(LateTask):
 
     def set_site(self, site):
         """Set Nikola site."""
-        self.logger = utils.get_logger('bundles')
+        super(BuildBundles, self).set_site(site)
         if webassets is None and site.config['USE_BUNDLES']:
             utils.req_missing(['webassets'], 'USE_BUNDLES', optional=True)
             self.logger.warn('Setting USE_BUNDLES to False.')
             site.config['USE_BUNDLES'] = False
             site._GLOBAL_CONTEXT['use_bundles'] = False
-        super(BuildBundles, self).set_site(site)
 
     def gen_tasks(self):
         """Bundle assets using WebAssets."""
diff --git a/nikola/plugins/task/galleries.py b/nikola/plugins/task/galleries.py
@@ -62,12 +62,11 @@ class Galleries(Task, ImageProcessor):
 
     def set_site(self, site):
         """Set Nikola site."""
+        super(Galleries, self).set_site(site)
         site.register_path_handler('gallery', self.gallery_path)
         site.register_path_handler('gallery_global', self.gallery_global_path)
         site.register_path_handler('gallery_rss', self.gallery_rss_path)
 
-        self.logger = utils.get_logger('render_galleries')
-
         self.kw = {
             'thumbnail_size': site.config['THUMBNAIL_SIZE'],
             'max_image_size': site.config['MAX_IMAGE_SIZE'],
@@ -104,8 +103,6 @@ def set_site(self, site):
         # Create self.gallery_links
         self.create_galleries_paths()
 
-        return super(Galleries, self).set_site(site)
-
     def _find_gallery_path(self, name):
         # The system using self.proper_gallery_links and self.improper_gallery_links
         # is similar as in listings.py.
diff --git a/nikola/plugins/task/scale_images.py b/nikola/plugins/task/scale_images.py
@@ -38,11 +38,6 @@ class ScaleImage(Task, ImageProcessor):
 
     name = "scale_images"
 
-    def set_site(self, site):
-        """Set Nikola site."""
-        self.logger = utils.get_logger('scale_images')
-        return super(ScaleImage, self).set_site(site)
-
     def process_tree(self, src, dst):
         """Process all images in a src tree and put the (possibly) rescaled images in the dst folder."""
         thumb_fmt = self.kw['image_thumbnail_format']
