bugfix: without this, create_symlink would only be called based on the CLI argument, not the .yml setting (#509)

Author: muffato

shpc/main/modules/__init__.py

@@ -212,8 +212,6 @@ def check_symlink(self, module_dir, symlink=False):
         Given an install command, if --symblink is provided make sure we have
         a symlink_base defined in settings and the directory exists.
         """
-        # Global override to arg
-        symlink = self.settings.symlink_tree is True or symlink
         if not symlink:
             return
 
@@ -379,6 +377,9 @@ def install(self, name, tag=None, symlink=False, **kwargs):
         subfolder = os.path.join(uri, tag.name)
         container_dir = self.container.container_dir(subfolder)
 
+        # Global override to arg
+        symlink = self.settings.symlink_tree is True or symlink
+
         # Cut out early if symlink desired or already exists
         self.check_symlink(module_dir, symlink)
         shpc.utils.mkdirp([module_dir, container_dir])