Page MenuHomePhabricator
Differential D13666

Modernize Slowvote Search
ClosedPublic
Actions

Authored by chad on Jul 21 2015, 9:02 PM.
Tags
None
Referenced Files
F18846247: D13666.id33082.diff
Wed, Oct 29, 5:58 PM
F18804260: D13666.diff
Sat, Oct 18, 7:04 AM
F18777475: D13666.id.diff
Sat, Oct 11, 3:11 AM
F18668954: D13666.id33040.diff
Sep 24 2025, 5:18 PM
F18668953: D13666.id33082.diff
Sep 24 2025, 5:18 PM
F18616634: D13666.id.diff
Sep 14 2025, 6:23 PM
F18593866: D13666.diff
Sep 12 2025, 4:13 PM
F18591794: D13666.diff
Sep 12 2025, 10:08 AM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
btrahan
Commits
Restricted Diffusion Commit
rP3bfb43629602: Modernize Slowvote Search
Summary

Updates slowvote search forms and results. Note I can't seem to get the Joins clause to fire anymore, is there a modern equiv?

Test Plan

Search on polls by state, votes, author, projects.

Diff Detail

Repository
rP Phabricator
Branch
slowvote-search
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 7358
Build 7756: [Placeholder Plan] Wait for 30 Seconds
Build 7755: arc lint + arc unit

Event Timeline

chad updated this revision to Diff 33040.Jul 21 2015, 9:02 PM
chad retitled this revision from to Modernize Slowvote Search.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added reviewers: epriestley, btrahan.
Herald added subscribers: Korvin, epriestley. · View Herald TranscriptJul 21 2015, 9:02 PM
epriestley accepted this revision.Jul 22 2015, 8:25 PM
epriestley edited edge metadata.
epriestley added inline comments.
src/applications/slowvote/query/PhabricatorSlowvoteSearchEngine.php
16

I think this is wrong / unnecessary? It will make every query only show stuff you've voted in.

50–54

Or ThreeStateField with "don't care, voted, not voted", although this is a larger change.

This revision is now accepted and ready to land.Jul 22 2015, 8:25 PM
chad planned changes to this revision.Jul 22 2015, 8:34 PM

I'll take a deeper look.

chad added a comment.Jul 22 2015, 8:34 PM

Also, I never got the JOIN clauses to trigger, did I miss something simple?

epriestley added a comment.Jul 22 2015, 9:40 PM

Oh, try implementing buildJoinClauseParts instead of buildJoinsClause?

chad added a comment.Jul 22 2015, 9:41 PM

I tried, but I'll try again. Maybe the other bug was overshadowing my results.

chad added a comment.Jul 22 2015, 11:12 PM

derp, it was Join vs Joins in the method name that sunk me.

chad updated this revision to Diff 33082.Jul 22 2015, 11:13 PM
chad edited edge metadata.
  • Fix Joins
This revision is now accepted and ready to land.Jul 22 2015, 11:13 PM
Closed by commit rP3bfb43629602: Modernize Slowvote Search (authored by chad, committed by chad). · Explain WhyJul 22 2015, 11:27 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
slowvote/
query/
51 lines
91 lines

Diff 33082

View Options

src/applications/slowvote/query/PhabricatorSlowvoteQuery.php

Loading...
View Options

src/applications/slowvote/query/PhabricatorSlowvoteSearchEngine.php

Loading...
Phabricator · Built by Phacility