# Butler.queryMetadata doc/API/implementation mismatch

XMLWordPrintable

#### Details

• Type: Story
• Status: Invalid
• Resolution: Done
• Fix Version/s: None
• Component/s:
• Labels:
None
• Sprint:
DB_S17_01
• Team:
Data Access and Database

#### Description

key appears to have been removed from the API for Butler.queryMetadata, but it appears in the doc and implementation:

  def queryMetadata(self, datasetType, format=None, dataId={}, **rest):  """Returns the valid values for one or more keys when given a partial  input collection data id.    Parameters  ----------  datasetType - str  The type of dataset to inquire about.  key - str  A key giving the level of granularity of the inquiry.  format - str, tuple  An optional key or tuple of keys to be returned.  dataId - DataId, dict  The partial data id.  **rest -  Keyword arguments for the partial data id.    Returns  -------  A list of valid values or tuples of valid values as specified by the format (defaulting to the same as  the key) at the key's level of granularity.  """ ...  if format is None:  format = (key,)  else:  format = sequencify(format) ... 

#### Activity

Hide
Tim Jenness added a comment - - edited

Interesting because I made exactly this comment in DM-7156 and it was allegedly fixed there. Looking at what was in DM-7156 it seems that only a partial fix was made and I don't think the linter was rerun.

Show
Tim Jenness added a comment - - edited Interesting because I made exactly this comment in DM-7156 and it was allegedly fixed there. Looking at what was in DM-7156 it seems that only a partial fix was made and I don't think the linter was rerun.
Hide
Colin Slater added a comment -

Duplicates DM-7894

Show
Colin Slater added a comment - Duplicates DM-7894
Hide
Tim Jenness added a comment -

As noted by Colin Slater this ticket already exists.

Show
Tim Jenness added a comment - As noted by Colin Slater this ticket already exists.

#### People

Assignee:
Nate Pease [X] (Inactive)
Reporter:
Paul Price
Watchers:
Colin Slater, Kian-Tat Lim, Paul Price, Tim Jenness