Page MenuHomePhabricator
Create Task
Maniphest T9234

Ponder query by "Answered by" gives unexpected results
Closed, ResolvedPublic
Actions

Description

I saw that Ponder was moved out of beta just a few hours ago so thought it would be OK to make a task for this.

  1. Navigate to Ponder
  2. Select Edit Query
  3. Update query to only have the "Answered by" filled out with a user

The results appear to not be filtered to show only questions that have been answered by the selected user. Example:
https://secure.phabricator.com/ponder/query/_ya_4qHDHeyJ/#R

Additionally I wasn't sure if intended or not, but it appears as though not-logged-in-users can view the list of questions, but are prompted to login when viewing a specific question.

Revisions and Commits

rP Phabricator
D13953rPad93af83891d Fix Ponder Answer query joins, builtins
D13952rP22dc3c1be2e0 Allow public on Ponder Questions, History pages

Related Objects

Event Timeline

cspeckmim created this task.Aug 21 2015, 3:56 AM
cspeckmim updated the task description. (Show Details)
cspeckmim added a project: Ponder.
cspeckmim added a subscriber: cspeckmim.
chad added a revision: D13952: Allow public on Ponder Questions, History pages.Aug 21 2015, 4:29 AM
chad added a revision: D13953: Fix Ponder Answer query joins, builtins.Aug 21 2015, 4:47 AM
chad added a subscriber: chad.Aug 21 2015, 5:01 AM

Macro suchwow: such bugs much reporting

cspeckmim added a comment.Aug 21 2015, 6:02 AM

I had just been giving Ponder a run and noticing some things. The "Answered" query also returns results with "0 Answers". I can move this into a new task if you'd rather it not reported in a comment.

chad added a comment.Aug 21 2015, 6:13 AM

It's fixed in the attached diffs

cspeckmim added a comment.EditedAug 21 2015, 6:13 AM

Oh that last comment made the assumption that it's supposed to show all questions that have been answered by anyone - I just realized that it might be referring to a query that fills in the "Answered by" field with the active user. If the latter then it's probably the same bug.

chad added a commit: rP22dc3c1be2e0: Allow public on Ponder Questions, History pages.Aug 21 2015, 1:49 PM
chad merged a task: T9240: Ponder Questions on this install should probably be open to the public and googlebot.Aug 21 2015, 1:51 PM
chad added subscribers: devurandom, cburroughs.
chad closed this task as Resolved by committing rPad93af83891d: Fix Ponder Answer query joins, builtins.Aug 21 2015, 1:58 PM
chad added a commit: rPad93af83891d: Fix Ponder Answer query joins, builtins.
urzds added a subscriber: urzds.Jul 12 2017, 11:08 AM
Herald added a subscriber: eadler. · View Herald TranscriptJul 12 2017, 11:08 AM
Phabricator · Built by Phacility