Continued:

- check parameters first, then "expensive" function invocation's returned value
- command fetch_instances should always fetch instances from a provided domain
  name
- only when --software=bar is used and no --force parameter was given

diff --git a/fba/boot.py b/fba/boot.py
index 0fb55ec5796751973811f249467add5d5546e00d..f0a109b6cf3a07115009fef6b7d29648584d5555 100644 (file)
--- a/fba/boot.py
+++ b/fba/boot.py
@@ -145,8 +145,9 @@ def init_parser():
     )
     parser.set_defaults(command=commands.fetch_instances)
     parser.add_argument("--domain", help="Instance name (aka. domain) to fetch further instances from. Start with a large instance, e.g. mastodon.social .")
-    parser.add_argument("--software", help="Name of software, e.g. 'lemmy'")
+    parser.add_argument("--force", action="store_true", help="Include also already existing instances, otherwise only new are checked")
     parser.add_argument("--single", action="store_true", help="Only fetch given instance.")
+    parser.add_argument("--software", help="Name of software, e.g. 'lemmy'")
 
     ### Fetch blocks from static text file(s) ###
     parser = subparser_command.add_parser(

diff --git a/fba/commands.py b/fba/commands.py
index 6999e2a5c0ea0b6b52d26c1a20d52ea3ea3fc0d0..9dcbd417e3aea92fbdb2284c1ec76836cb5de556 100644 (file)
--- a/fba/commands.py
+++ b/fba/commands.py
@@ -996,7 +996,7 @@ def fetch_instances(args: argparse.Namespace) -> int:
         elif instances.is_registered(row["domain"]) and software_helper.is_relay(row["software"]):
             logger.warning("row[domain]='%s' is of software type '%s' which is not supported by this command. Please invoke fetch_relays instead - SKIPPED!", row["domain"], row["software"])
             continue
-        elif instances.is_recent(row["domain"]):
+        elif not args.force and not args.software in [None, ""]and instances.is_recent(row["domain"]):
             logger.debug("row[domain]='%s' has been recently crawled - SKIPPED!", row["domain"])
             continue