recaptime.dev / phorge
@recaptime-dev's working patches + fork for Phorge, a community fork of Phabricator. (Upstream dev and stable branches are at upstream/main and upstream/stable respectively.) hq.recaptime.dev/wiki/Phorge
phorge phabricator
1
fork

Configure Feed

Select the types of activity you want to include in your feed.

Fix searching legalpad documents by contributors

Summary:
Found in a change submitted to Phorge (https://we.phorge.it/D25018#inline-231), this fixes a typo in populating search the `contributorPHIDs` constraint in the Legalpad search engine.

Currently when trying to search legalpad documents by contributor an error is encountered:
```lang=console
Array for %Ls conversion is empty. Query: contributor.dst IN (%Ls)
```

Test Plan: I searched for legalpad documents based on a contributor and got back correct results.

Reviewers: #blessed_reviewers, epriestley

Reviewed By: #blessed_reviewers, epriestley

Subscribers: Korvin, epriestley

Differential Revision: https://secure.phabricator.com/D21719

Christopher Speck 3b2868e1 09c3c7d8

+1 -1
+1 -1
src/applications/legalpad/query/LegalpadDocumentSearchEngine.php
··· 53 53 } 54 54 55 55 if ($map['contributorPHIDs']) { 56 - $query->withContributorPHIDs($map['creatorPHIDs']); 56 + $query->withContributorPHIDs($map['contributorPHIDs']); 57 57 } 58 58 59 59 if ($map['creatorPHIDs']) {