Page MenuHomePhabricator

D13958.id33861.diff
No OneTemporary

D13958.id33861.diff

diff --git a/src/applications/ponder/controller/PonderQuestionViewController.php b/src/applications/ponder/controller/PonderQuestionViewController.php
--- a/src/applications/ponder/controller/PonderQuestionViewController.php
+++ b/src/applications/ponder/controller/PonderQuestionViewController.php
@@ -20,7 +20,7 @@
return new Aphront404Response();
}
- $answers = $this->buildAnswers($question->getAnswers());
+ $answers = $this->buildAnswers($question);
$answer_add_panel = id(new PonderAddAnswerView())
->setQuestion($question)
@@ -81,6 +81,20 @@
->addPropertyList($properties)
->appendChild($footer);
+ if ($viewer->getPHID() == $question->getAuthorPHID()) {
+ $status = $question->getStatus();
+ $answers_list = $question->getAnswers();
+ if ($answers_list && ($status == PonderQuestionStatus::STATUS_OPEN)) {
+ $info_view = id(new PHUIInfoView())
+ ->setSeverity(PHUIInfoView::SEVERITY_WARNING)
+ ->appendChild(
+ pht(
+ 'If this question has been resolved, please consider closing
+ the question and marking the answer as helpful.'));
+ $object_box->setInfoView($info_view);
+ }
+ }
+
$crumbs = $this->buildApplicationCrumbs($this->buildSideNavView());
$crumbs->addTextCrumb('Q'.$id, '/Q'.$id);
@@ -206,8 +220,9 @@
* TODO - re-factor this to ajax in one answer panel at a time in a more
* standard fashion. This is necessary to scale this application.
*/
- private function buildAnswers(array $answers) {
+ private function buildAnswers(PonderQuestion $question) {
$viewer = $this->getViewer();
+ $answers = $question->getAnswers();
$author_phids = mpull($answers, 'getAuthorPHID');
$handles = $this->loadViewerHandles($author_phids);

File Metadata

Mime Type
text/plain
Expires
Wed, Mar 26, 8:15 AM (3 w, 3 h ago)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
7707790
Default Alt Text
D13958.id33861.diff (1 KB)

Event Timeline