Page MenuHomePhabricator
Files F13966520

D7716.id17424.diff
No OneTemporary
Actions

D7716.id17424.diff
View Options

Index: src/applications/maniphest/controller/ManiphestTaskDetailController.php
===================================================================
--- src/applications/maniphest/controller/ManiphestTaskDetailController.php
+++ src/applications/maniphest/controller/ManiphestTaskDetailController.php
@@ -382,10 +382,15 @@
if (!$user->isLoggedIn()) {
// TODO: Eventually, everything should run through this. For now, we're
// only using it to get a consistent "Login to Comment" button.
- $comment_form = id(new PhabricatorApplicationTransactionCommentView())
+ $comment_box = id(new PhabricatorApplicationTransactionCommentView())
->setUser($user)
->setRequestURI($request->getRequestURI());
$preview_panel = null;
+ } else {
+ $comment_box = id(new PHUIObjectBoxView())
+ ->setFlush(true)
+ ->setHeaderText($comment_header)
+ ->appendChild($comment_form);
}
$object_box = id(new PHUIObjectBoxView())
@@ -396,11 +401,6 @@
$object_box->addPropertyList($description);
}
- $comment_box = id(new PHUIObjectBoxView())
- ->setFlush(true)
- ->setHeaderText($comment_header)
- ->appendChild($comment_form);
-
return $this->buildApplicationPage(
array(
$crumbs,
Index: src/applications/transactions/view/PhabricatorApplicationTransactionCommentView.php
===================================================================
--- src/applications/transactions/view/PhabricatorApplicationTransactionCommentView.php
+++ src/applications/transactions/view/PhabricatorApplicationTransactionCommentView.php
@@ -85,6 +85,7 @@
$uri = id(new PhutilURI('/login/'))
->setQueryParam('next', (string) $this->getRequestURI());
return id(new PHUIObjectBoxView())
+ ->setFlush(true)
->setHeaderText(pht('Add Comment'))
->appendChild(
javelin_tag(

File Metadata

Mime Type
text/plain
Expires
Thu, Oct 17, 8:21 AM (3 w, 5 d ago)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
6721130
Default Alt Text
D7716.id17424.diff (1 KB)

Event Timeline

Phabricator · Built by Phacility