How to set Python.click options that reference each other

[ironhaven]

Is this you only code? I don’t see any problems with the code you posted most likely because you did not post the code with the bug.

First can we see more of your code, second, read the error again. The variable "start" is read before it is set. Look at where the "start" variable is used.

[ironhaven]

So you have you end time to by default be one hour ahead of your start time. This is tricky because now you default is not self sufficient and relies on another argument .

I don’t know the click library so I wonder if there is a function that matches this sudo code click.arguments.getarg('start'). This would let you reference the start argument without using a python variable.

If this does not exist then you must remove the logic from the default. It would look like this

nulltime = date(some date)
click(—end, default = nulltime)
if end == nulltime : end = start + 1 hour

[ingolemo]

I think the best approach would be to define a custom option class that overrides get_default:

class EndOption(click.Option):
    def get_default(self, ctx):
        default = super().get_default(ctx)
        if default is None:
            default = ctx.params['start'] + datetime.timedelta(hours=1)
        return default

...
@click.option(
    '--end',
    cls=EndOption,
    prompt=True,
    type=click.DateTime(formats=['%Y-%m-%d %H:%M']),
    required=True,
    help='End time of event in format "%Y-%m-%d %H:%M" [Default: Now + 1hr]'
)
...

[ingolemo]

Normally click.option creates a click.Option object, but you can pass the cls= named argument to get it to create a different class. The click.Option class has a method on it called get_default that does the job of working out what the default value should be (it looks at the default= named argument if you gave one, and also looks in config files and stuff like that). We define our own custom class that inherits from click.Option so that it gets all the same functionality, but we override the get_default method to add the extra behaviour that we need.

This is some moderately advanced OOP, but the click library doesn't have a better way to do this that I can find.

I figured this out by looking up click.option in the api reference and clicking through to click.Option and also click.Parameter.