Skip to content

RFC: make retrieveExtensionMetadata public, accept QVariantMap?

Isaiah requested to merge github/fork/ihnorton/update_extsmgr_and_dmriinstallutil into master

Follow-up to https://github.com/Slicer/Slicer/pull/863#issuecomment-358000228 before I forget:

I noticed that qRestAPI::Parameters is just a QMap<string,string> typedef, so #883 could be reworked for more general use. Then the parameter is automatically wrapped for auto-conversion from a Python dict, and we could use it from Python to query arbitrary metadata combinations instead of only extension-by-name (for example to simplify the Midas query implementation in ExtensionStats).

One question though: should we add an API to return an array if there are multiple results? Right now there are internal checks whether more than one result was returned from the query, in which case the function just returns an empty dict.

Merge request reports