Index Operator Interface

[ggpwnkthx]

An OperationIndex that has an ID containing a . (e.g. me.GetProfile) can't be accessed through the sad smiley interface without adding some kind of chained attribute functionality - at least to stay aligned with the spirit of accessors. I wrote something up for that, but it's feels hacky and I don't like it.

Instead, how do you feel about an index operator for the OpenAPI class?

class OpenAPI:
  ...
    def __getitem__(self, item: str) -> OperationIndex:
        """
        index operator interface
        access operations by operationId
        """
        return getattr(self._, item)

api = OpenAPI.loads(...)
api["me.GetProfile"].request()

This could be expanded on to handle the api[path, method] syntax pretty easily, too.

class OpenAPI:
  ...
    def __getitem__(self, item: Union[str, Tuple]) -> OperationIndex:
        """
        index operator interface
        access operations by operationId
        """
        return self.createRequest(item) if isinstance(item, tuple) else getattr(self._, item)

[commonism]

I'd prefer to have this on OperationIndex:

api._["me.GetProfile"]
api._[(path, method)]

As I see this - __call__ can be used instead of request()

api._["me.GetProfile"](parameters=dict(user="…"))

[ggpwnkthx]

Sounds good. I like the __call__ idea as well!
I've wrapped up work for today but I can submit a PR tomorrow.

[commonism]

Request.call already exists, I mentioned this as I saw you do not use it

aiopenapi3/aiopenapi3/request.py

Lines 96 to 106 in 55b200d

	def __call__(self, *args, return_headers: bool = False, **kwargs) -> Union["JSON", Tuple[Dict[str, str], "JSON"]]:
	"""
	:param args:
	:param return_headers: if set return a tuple (header, body)
	:param kwargs:
	:return: body or (header, body)
	"""
	headers, data, result = self.request(*args, **kwargs)
	if return_headers:
	return headers, data
	return data

it's used implicit when using the ssi

api._.getUsers(parameters=dict(groups=["…"]))

[ggpwnkthx]

I see (said the blind man... lol). I can't believe I missed that. Thank you.