#922 Add doc string for virtual methods
Closed by tkopecek. Opened by till.
till/koji virtualmethod-dosctring  into  master

Download 922.patch

This makes the api documentation available via introspection and therefore the development of scripts easier.

Thanks! Awesome idea :smile:

The only thing that concerns me is that, while we currently only use VirtualMethod in ClientSession, there isn't really any part of it that (before this change) that explicitly ties it to that usage. Caching the _listapi result in the class var doesn't make sense if the class could be used elsewhere.

Actually.... even as is we have a problem if we have two ClientSession instances talking to two different servers with different apis.

So perhaps a slight rework of this. Maybe subclass ClientSession, add a more explicit link back to the controlling session, and cache the api data there instead.

What do you think?

So perhaps a slight rework of this. Maybe subclass ClientSession, add a more explicit link back to the controlling session, and cache the api data there instead.

err, I think I meant to subclass VirtualMethod

Issue #1533

I've extended it in #1987

Metadata Update from @jcupova:
- Pull-request tagged with: testing-ready

Metadata Update from @jcupova:
- Pull-request tagged with: testing-done

Pull-Request has been closed by tkopecek

Metadata