Page MenuHomePhabricator

Fix searching legalpad documents by contributors
ClosedPublic

Authored by cspeckmim on Sat, Sep 4, 5:48 PM.

Details

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:

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.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

cspeckmim edited the summary of this revision. (Show Details)
cspeckmim edited the test plan for this revision. (Show Details)

I didn't make a task for this as it's a fairly small change and the reproduction/test case is also fairly simple.

cspeckmim retitled this revision from Fix typo in legalpad search engine to Fix searching legalpad documents by contributors.Sat, Sep 4, 11:51 PM
This revision is now accepted and ready to land.Sun, Sep 5, 12:17 AM