Page MenuHomePhabricator

D8352.diff
No OneTemporary

D8352.diff

Index: src/applications/differential/controller/DifferentialRevisionViewController.php
===================================================================
--- src/applications/differential/controller/DifferentialRevisionViewController.php
+++ src/applications/differential/controller/DifferentialRevisionViewController.php
@@ -903,9 +903,6 @@
->needComments(true)
->execute();
- $engine = id(new PhabricatorMarkupEngine())
- ->setViewer($viewer);
-
$timeline = id(new DifferentialTransactionView())
->setUser($viewer)
->setObjectPHID($revision->getPHID())
Index: src/applications/transactions/view/PhabricatorApplicationTransactionView.php
===================================================================
--- src/applications/transactions/view/PhabricatorApplicationTransactionView.php
+++ src/applications/transactions/view/PhabricatorApplicationTransactionView.php
@@ -149,23 +149,23 @@
}
protected function getOrBuildEngine() {
- if ($this->engine) {
- return $this->engine;
- }
-
- $field = PhabricatorApplicationTransactionComment::MARKUP_FIELD_COMMENT;
-
- $engine = id(new PhabricatorMarkupEngine())
- ->setViewer($this->getUser());
- foreach ($this->transactions as $xaction) {
- if (!$xaction->hasComment()) {
- continue;
+ if (!$this->engine) {
+ $field = PhabricatorApplicationTransactionComment::MARKUP_FIELD_COMMENT;
+
+ $engine = id(new PhabricatorMarkupEngine())
+ ->setViewer($this->getUser());
+ foreach ($this->transactions as $xaction) {
+ if (!$xaction->hasComment()) {
+ continue;
+ }
+ $engine->addObject($xaction->getComment(), $field);
}
- $engine->addObject($xaction->getComment(), $field);
+ $engine->process();
+
+ $this->engine = $engine;
}
- $engine->process();
- return $engine;
+ return $this->engine;
}
private function buildChangeDetailsLink(

File Metadata

Mime Type
text/plain
Expires
Wed, Nov 20, 10:11 AM (16 h, 26 m)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
6749075
Default Alt Text
D8352.diff (1 KB)

Event Timeline