Copy-sftp module bug

[jaimeozaez]

When module copy-sftp is run by itself (I mean, outside of finish function), an error is raised (even though all the files seem to have been copied correctly).

This is an example of the error:

╭───────────────────────────────────────────── Traceback (most recent call last) ──────────────────────────────────────────────╮
│ /data/ucct/bi/pipelines/micromamba/envs/buisciii-tools_2.2.5/bin/bu-isciii:8 in <module>                                     │
│                                                                                                                              │
│   7 │   sys.argv[0] = re.sub(r'(-script\.pyw|\.exe)?$', '', sys.argv[0])                                                     │
│ ❱ 8 │   sys.exit(run_bu_isciii())                                                                                            │
│   9                                                                                                                          │
│                                                                                                                              │
│ /data/ucct/bi/pipelines/micromamba/envs/buisciii-tools_2.2.5/lib/python3.13/site-packages/bu_isciii/__main__.py:66 in        │
│ run_bu_isciii                                                                                                                │
│                                                                                                                              │
│    65 │   # Lanch the click cli                                                                                              │
│ ❱  66 │   bu_isciii_cli()                                                                                                    │
│    67                                                                                                                        │
│                                                                                                                              │
│ /data/ucct/bi/pipelines/micromamba/envs/buisciii-tools_2.2.5/lib/python3.13/site-packages/click/core.py:1161 in __call__     │
│                                                                                                                              │
│   1160 │   │   """Alias for :meth:`main`."""                                                                                 │
│ ❱ 1161 │   │   return self.main(*args, **kwargs)                                                                             │
│   1162                                                                                                                       │
│                                                                                                                              │
│ /data/ucct/bi/pipelines/micromamba/envs/buisciii-tools_2.2.5/lib/python3.13/site-packages/click/core.py:1082 in main         │
│                                                                                                                              │
│   1081 │   │   │   │   with self.make_context(prog_name, args, **extra) as ctx:                                              │
│ ❱ 1082 │   │   │   │   │   rv = self.invoke(ctx)                                                                             │
│   1083 │   │   │   │   │   if not standalone_mode:                                                                           │
│                                                                                                                              │
│ /data/ucct/bi/pipelines/micromamba/envs/buisciii-tools_2.2.5/lib/python3.13/site-packages/click/core.py:1697 in invoke       │
│                                                                                                                              │
│   1696 │   │   │   │   with sub_ctx:                                                                                         │
│ ❱ 1697 │   │   │   │   │   return _process_result(sub_ctx.command.invoke(sub_ctx))                                           │
│   1698                                                                                                                       │
│                                                                                                                              │
│ /data/ucct/bi/pipelines/micromamba/envs/buisciii-tools_2.2.5/lib/python3.13/site-packages/click/core.py:1443 in invoke       │
│                                                                                                                              │
│   1442 │   │   if self.callback is not None:                                                                                 │
│ ❱ 1443 │   │   │   return ctx.invoke(self.callback, **ctx.params)                                                            │
│   1444                                                                                                                       │
│                                                                                                                              │
│ /data/ucct/bi/pipelines/micromamba/envs/buisciii-tools_2.2.5/lib/python3.13/site-packages/click/core.py:788 in invoke        │
│                                                                                                                              │
│    787 │   │   │   with ctx:                                                                                                 │
│ ❱  788 │   │   │   │   return __callback(*args, **kwargs)                                                                    │
│    789                                                                                                                       │
│                                                                                                                              │
│ /data/ucct/bi/pipelines/micromamba/envs/buisciii-tools_2.2.5/lib/python3.13/site-packages/click/decorators.py:33 in new_func │
│                                                                                                                              │
│    32 │   def new_func(*args: "P.args", **kwargs: "P.kwargs") -> "R":                                                        │
│ ❱  33 │   │   return f(get_current_context(), *args, **kwargs)                                                               │
│    34                                                                                                                        │
│                                                                                                                              │
│ /data/ucct/bi/pipelines/micromamba/envs/buisciii-tools_2.2.5/lib/python3.13/site-packages/bu_isciii/__main__.py:383 in       │
│ copy_sftp                                                                                                                    │
│                                                                                                                              │
│   382 │   )                                                                                                                  │
│ ❱ 383 │   new_del.copy_sftp()                                                                                                │
│   384                                                                                                                        │
│                                                                                                                              │
│ /data/ucct/bi/pipelines/micromamba/envs/buisciii-tools_2.2.5/lib/python3.13/site-packages/bu_isciii/copy_sftp.py:167 in      │
│ copy_sftp                                                                                                                    │
│                                                                                                                              │
│   166 │   │   │   │   for folders_list in self.last_folders:                                                                 │
│ ❱ 167 │   │   │   │   │   final_folder = os.path.join(                                                                       │
│   168 │   │   │   │   │   │   self.sftp_folder, self.service_folder, folders_list                                            │
│ in join:90                                                                                                                   │
│ in _check_arg_types:188                                                                                                      │
╰──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯
TypeError: join() argument must be str, bytes, or os.PathLike object, not 'NoneType'

[victor5lm]

This issue arises exclusively when selecting "all" when running the copy-sftp module, because the variable "folders_list" is None:

buisciii-tools/bu_isciii/copy_sftp.py

Lines 165 to 173 in c22c847

	finally:
	for folders_list in self.last_folders:
	final_folder = os.path.join(
	self.sftp_folder, self.service_folder, folders_list
	)
	stderr.print(
	"Listing the content of the final folder %s" % folders_list
	)
	self.listdirs(final_folder)