← Back to team overview

sts-sponsors team mailing list archive

Re: [Merge] ~lloydwaltersj/maas:fix-broken-oapi-param-names into maas:master

 


Diff comments:

> diff --git a/src/maasserver/api/doc_oapi.py b/src/maasserver/api/doc_oapi.py
> index b7e3a9d..56efa28 100644
> --- a/src/maasserver/api/doc_oapi.py
> +++ b/src/maasserver/api/doc_oapi.py
> @@ -213,8 +213,11 @@ def _oapi_item_from_docstring(
>              oper_obj["deprecated"] = True
>          for param in ap_dict["params"]:
>              description = _prettify(param["description_stripped"])
> -            name = param["name"].strip("}{")
> -            path_var = name in uri_params
> +            # LP 2009140
> +            stripped_name = re.match(r"(?<=\{)\.+(?<=\})", param["name"])

good point, will change

> +            name = (
> +                param["name"] if not stripped_name else stripped_name.group()
> +            )
>              required = (
>                  param["options"]["required"].lower() == "true"
>                  or name != param["name"]


-- 
https://code.launchpad.net/~lloydwaltersj/maas/+git/maas/+merge/438279
Your team MAAS Committers is subscribed to branch maas:master.