Fix #437: add 10 editsite flags for tabcmd Classic parity

tabcmd/commands/site/edit_site_command.py

@@ -8,6 +8,17 @@
 from tabcmd.execution.logger_config import log
 
 
+def _str_to_bool(value):
+    """Coerce a 'true'/'false' string (from argparse choices) to a real bool.
+
+    TSC SiteItem boolean setters are guarded by @property_is_boolean and reject
+    strings, so we must convert before assignment.
+    """
+    if value is None:
+        return None
+    return str(value).lower() == "true"
+
+
 class EditSiteCommand(Server):
     """
     Command to change the name of a site or its web folder name. Users can also use this command to allow or deny
@@ -24,6 +35,7 @@ def define_args(edit_site_parser):
         args_group.add_argument("--site-name", default=None, dest="new_site_name", help=_("editsite.options.site-name"))
         set_common_site_args(args_group)
         set_site_status_arg(args_group)
+        set_edit_site_only_args(args_group)
 
     @classmethod
     def run_command(cls, args):
@@ -41,6 +53,50 @@ def run_command(cls, args):
             site_item.storage_quota = args.storage_quota
         if args.status:
             site_item.state = args.status
+
+        # New flags for W-tabcmd-437. Every attribute is only touched when the
+        # corresponding argparse dest is not None, so an unpassed flag leaves
+        # the server-side setting unchanged.
+        if getattr(args, "guest_access_enabled", None) is not None:
+            site_item.guest_access_enabled = _str_to_bool(args.guest_access_enabled)
+
+        if getattr(args, "cache_warmup_enabled", None) is not None:
+            site_item.cache_warmup_enabled = args.cache_warmup_enabled
+
+        if getattr(args, "subscription_email", None) is not None:
+            # Matches Classic: set the value AND toggle the *_enabled flag.
+            # An empty string clears the value and disables the feature.
+            # NOTE: TSC RequestFactory currently lowercases this value (bug to
+            # file upstream); user@Domain.com will be sent as user@domain.com.
+            site_item.custom_subscription_email = args.subscription_email
+            site_item.custom_subscription_email_enabled = bool(args.subscription_email)
+
+        if getattr(args, "subscription_footer", None) is not None:
+            # Same two-attribute wiring as subscription_email. TSC also
+            # lowercases this string; tracked as an upstream TSC bug.
+            site_item.custom_subscription_footer = args.subscription_footer
+            site_item.custom_subscription_footer_enabled = bool(args.subscription_footer)
+
+        if getattr(args, "web_extraction_enabled", None) is not None:
+            site_item.web_extraction_enabled = _str_to_bool(args.web_extraction_enabled)
+
+        if getattr(args, "allow_subscriptions", None) is not None:
+            # Inverted: REST API exposes this as disableSubscriptions.
+            site_item.disable_subscriptions = not args.allow_subscriptions
+
+        if getattr(args, "allow_web_authoring", None) is not None:
+            site_item.authoring_enabled = args.allow_web_authoring
+
+        if getattr(args, "allow_mobile_snapshots", None) is not None:
+            site_item.sheet_image_enabled = args.allow_mobile_snapshots
+
+        if getattr(args, "use_default_time_zone", None):
+            # Only set use_default_time_zone; do NOT set time_zone -- argparse
+            # already enforces mutual exclusion between the two flags.
+            site_item.use_default_time_zone = True
+        elif getattr(args, "time_zone", None) is not None:
+            site_item.time_zone = args.time_zone
+
         try:
             logger.info(_("editsite.status").format(site_item.name))
             server.sites.update(site_item)

tabcmd/execution/global_options.py

@@ -181,6 +181,128 @@ def set_site_status_arg(parser):
     return parser
 
 
+# only in edit-site: flags that map onto TSC SiteItem attributes but are not
+# currently wired in create_site_command. Kept out of set_common_site_args so
+# createsite --help does not list options its run_command ignores. See #437.
+def set_edit_site_only_args(parser):
+    # guest access: Classic passes true/false as a string value
+    parser.add_argument(
+        "-g",
+        "--guest-access-enabled",
+        choices=["true", "false"],
+        help=_("editsite.options.guest_access_enabled"),
+    )
+
+    # cache warmup: paired positive/negative boolean, default None so an
+    # unpassed flag is a no-op (leaves the server setting unchanged)
+    cache_warmup_group = parser.add_mutually_exclusive_group()
+    cache_warmup_group.add_argument(
+        "--cache-warmup",
+        dest="cache_warmup_enabled",
+        action="store_true",
+        default=None,
+        help=_("editsite.options.cache_warmup"),
+    )
+    cache_warmup_group.add_argument(
+        "--no-cache-warmup",
+        dest="cache_warmup_enabled",
+        action="store_false",
+        default=None,
+        help=_("editsite.options.no_cache_warmup"),
+    )
+
+    # subscription email: also flips custom_subscription_email_enabled
+    parser.add_argument(
+        "-e",
+        "--subscription-email",
+        default=None,
+        help=_("editsite.options.subscription_email"),
+    )
+
+    # subscription footer: also flips custom_subscription_footer_enabled.
+    # -f matches Classic tabcmd editsite parity; editsite does not use -f for --filename.
+    parser.add_argument(
+        "-f",
+        "--subscription-footer",
+        default=None,
+        help=_("editsite.options.subscription_footer"),
+    )
+
+    # web extraction: string value, coerced to bool at wire time
+    parser.add_argument(
+        "--web-extraction-enabled",
+        choices=["true", "false"],
+        help=_("editsite.options.web_extraction_enabled"),
+    )
+
+    # allow subscriptions: inverted onto disable_subscriptions in run_command
+    allow_subs_group = parser.add_mutually_exclusive_group()
+    allow_subs_group.add_argument(
+        "--allow-subscriptions",
+        dest="allow_subscriptions",
+        action="store_true",
+        default=None,
+        help=_("editsite.options.allow_subscriptions"),
+    )
+    allow_subs_group.add_argument(
+        "--no-allow-subscriptions",
+        dest="allow_subscriptions",
+        action="store_false",
+        default=None,
+        help=_("editsite.options.no_allow_subscriptions"),
+    )
+
+    # allow web authoring: direct mapping onto authoring_enabled
+    authoring_group = parser.add_mutually_exclusive_group()
+    authoring_group.add_argument(
+        "--allow-web-authoring",
+        dest="allow_web_authoring",
+        action="store_true",
+        default=None,
+        help=_("editsite.options.allow_web_authoring"),
+    )
+    authoring_group.add_argument(
+        "--no-allow-web-authoring",
+        dest="allow_web_authoring",
+        action="store_false",
+        default=None,
+        help=_("editsite.options.no_allow_web_authoring"),
+    )
+
+    # allow mobile snapshots: maps onto sheet_image_enabled (NOT
+    # mobile_biometrics_enabled, which is a separate biometric-auth feature)
+    mobile_group = parser.add_mutually_exclusive_group()
+    mobile_group.add_argument(
+        "--allow-mobile-snapshots",
+        dest="allow_mobile_snapshots",
+        action="store_true",
+        default=None,
+        help=_("editsite.options.allow_mobile_snapshots"),
+    )
+    mobile_group.add_argument(
+        "--no-allow-mobile-snapshots",
+        dest="allow_mobile_snapshots",
+        action="store_false",
+        default=None,
+        help=_("editsite.options.no_allow_mobile_snapshots"),
+    )
+
+    # time zone: string value, mutually exclusive with --use-default-time-zone
+    tz_group = parser.add_mutually_exclusive_group()
+    tz_group.add_argument(
+        "--time-zone",
+        default=None,
+        help=_("editsite.options.time_zone"),
+    )
+    tz_group.add_argument(
+        "--use-default-time-zone",
+        action="store_true",
+        default=None,
+        help=_("editsite.options.use_default_time_zone"),
+    )
+    return parser
+
+
 # mismatched arguments: createsite says --url, editsite says --site-id
 # just let both commands use either of them
 def set_site_id_args(parser):

tabcmd/locales/en/tabcmd_messages_en.properties

@@ -44,6 +44,20 @@ editsite.options.site-name=Display name of the site
 editsite.options.status=Change availability of site. Must be either ''{0}'' or ''{1}''
 editsite.options.extract_encryption_mode=Extract encryption mode: disabled, enabled, or enforced. If not set, then it is not changed on the server.
 editsite.options.run_now_enabled=Allow [or deny] Run Now option for this site. Default is set to allow Run Now
+editsite.options.guest_access_enabled=Allow or deny guest access to the site. Value must be ''true'' or ''false''
+editsite.options.cache_warmup=Enable cache warmup for the site
+editsite.options.no_cache_warmup=Disable cache warmup for the site
+editsite.options.subscription_email=Custom "from" email address used for subscription emails. Pass an empty string to clear
+editsite.options.subscription_footer=Custom footer text appended to subscription emails. Pass an empty string to clear
+editsite.options.web_extraction_enabled=Allow or deny web data extraction on the site. Value must be ''true'' or ''false''
+editsite.options.allow_subscriptions=Allow subscriptions on the site
+editsite.options.no_allow_subscriptions=Disable subscriptions on the site
+editsite.options.allow_web_authoring=Allow web authoring on the site
+editsite.options.no_allow_web_authoring=Disable web authoring on the site
+editsite.options.allow_mobile_snapshots=Allow mobile app snapshots (sheet images) on the site
+editsite.options.no_allow_mobile_snapshots=Disable mobile app snapshots (sheet images) on the site
+editsite.options.time_zone=Time zone for the site (IANA identifier, e.g. America/Los_Angeles). Mutually exclusive with --use-default-time-zone
+editsite.options.use_default_time_zone=Use the server default time zone for the site. Mutually exclusive with --time-zone
 editsite.short_description=Edit a site
 editsite.status=Edit site ''{0}'' on the server...
 encryptextracts.short_description=Encrypt extracts on a site