The method getActiveItems() of FacetApiAdapter class has wrong doc comment for input argument $facet.

For now it is 

   * @param array|string $facet
   *   Either the facet definition as returned by facetapi_facet_load() or the
   *   machine readable name of the facet.

but method accepts only array values.

CommentFileSizeAuthor
#2 documentation_issue-2563117-2.patch466 bytesPravin Ajaaz
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

undertext created an issue. See original summary.

Pravin Ajaaz’s picture

Pravin Ajaaz CreditAttribution: Pravin Ajaaz as a volunteer and at Ameex-Drupal Geeks commented
Status: Active » Needs review
FileSize
documentation_issue-2563117-2.patch466 bytes

A patch to fix it.

Sonal.Sangale’s picture

Sonal.Sangale CreditAttribution: Sonal.Sangale at Blisstering Solutions commented
Assigned: Unassigned » Sonal.Sangale
Status: Needs review » Reviewed & tested by the community
Pravin Ajaaz’s picture

Pravin Ajaaz CreditAttribution: Pravin Ajaaz as a volunteer and at Young Globes commented

  • eugene.ilyin committed 30fb518 on 7.x-1.x 
    Issue #2563117 by undertext: Documentation issue with getActiveItems()...
eugene.ilyin’s picture

eugene.ilyin CreditAttribution: eugene.ilyin as a volunteer and at DrupalJedi commented

Patch is wrong. Applied, but with corrections.

eugene.ilyin’s picture

eugene.ilyin CreditAttribution: eugene.ilyin as a volunteer and at DrupalJedi commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

remydenton’s picture

remydenton CreditAttribution: remydenton at Isovera commented
Related issues: +#2867867: Allow getActiveItems() method to accept argument as either a string or an array

Follow-up issue: https://www.drupal.org/node/2867867