Page MenuHomePhabricator
Files F15451949

D14648.id35433.diff
No OneTemporary
Actions

D14648.id35433.diff
View Options

diff --git a/src/applications/ponder/storage/PonderAnswerTransaction.php b/src/applications/ponder/storage/PonderAnswerTransaction.php
--- a/src/applications/ponder/storage/PonderAnswerTransaction.php
+++ b/src/applications/ponder/storage/PonderAnswerTransaction.php
@@ -36,18 +36,6 @@
return $phids;
}
- public function getRemarkupBlocks() {
- $blocks = parent::getRemarkupBlocks();
-
- switch ($this->getTransactionType()) {
- case self::TYPE_CONTENT:
- $blocks[] = $this->getNewValue();
- break;
- }
-
- return $blocks;
- }
-
public function shouldHide() {
switch ($this->getTransactionType()) {
case self::TYPE_QUESTION_ID:
@@ -134,20 +122,14 @@
return parent::getTitleForFeed();
}
- public function getBodyForFeed(PhabricatorFeedStory $story) {
- $new = $this->getNewValue();
-
- $body = null;
-
+ public function getRemarkupBodyForFeed(PhabricatorFeedStory $story) {
+ $text = null;
switch ($this->getTransactionType()) {
case self::TYPE_CONTENT:
- return phutil_escape_html_newlines(
- id(new PhutilUTF8StringTruncator())
- ->setMaximumGlyphs(128)
- ->truncateString($new));
+ $text = $this->getNewValue();
break;
}
- return parent::getBodyForFeed($story);
+ return $text;
}
diff --git a/src/applications/ponder/storage/PonderQuestionTransaction.php b/src/applications/ponder/storage/PonderQuestionTransaction.php
--- a/src/applications/ponder/storage/PonderQuestionTransaction.php
+++ b/src/applications/ponder/storage/PonderQuestionTransaction.php
@@ -43,18 +43,6 @@
return $phids;
}
- public function getRemarkupBlocks() {
- $blocks = parent::getRemarkupBlocks();
-
- switch ($this->getTransactionType()) {
- case self::TYPE_CONTENT:
- $blocks[] = $this->getNewValue();
- break;
- }
-
- return $blocks;
- }
-
public function getTitle() {
$author_phid = $this->getAuthorPHID();
$object_phid = $this->getObjectPHID();
@@ -222,20 +210,6 @@
return parent::getActionName();
}
- public function shouldHide() {
- switch ($this->getTransactionType()) {
- case self::TYPE_CONTENT:
- if ($this->getOldValue() === null) {
- return true;
- } else {
- return false;
- }
- break;
- }
-
- return parent::shouldHide();
- }
-
public function getTitleForFeed() {
$author_phid = $this->getAuthorPHID();
$object_phid = $this->getObjectPHID();
@@ -302,34 +276,14 @@
return parent::getTitleForFeed();
}
- public function getBodyForFeed(PhabricatorFeedStory $story) {
- $new = $this->getNewValue();
- $old = $this->getOldValue();
-
- $body = null;
-
+ public function getRemarkupBodyForFeed(PhabricatorFeedStory $story) {
+ $text = null;
switch ($this->getTransactionType()) {
- case self::TYPE_TITLE:
- if ($old === null) {
- $question = $story->getObject($this->getObjectPHID());
- return phutil_escape_html_newlines(
- id(new PhutilUTF8StringTruncator())
- ->setMaximumGlyphs(128)
- ->truncateString($question->getContent()));
- }
- break;
- case self::TYPE_ANSWERS:
- $answer = $this->getNewAnswerObject($story);
- if ($answer) {
- return phutil_escape_html_newlines(
- id(new PhutilUTF8StringTruncator())
- ->setMaximumGlyphs(128)
- ->truncateString($answer->getContent()));
- }
+ case self::TYPE_CONTENT:
+ $text = $this->getNewValue();
break;
}
-
- return parent::getBodyForFeed($story);
+ return $text;
}
/**

File Metadata

Mime Type
text/plain
Expires
Sun, Mar 30, 12:47 AM (6 d, 9 h ago)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
7726577
Default Alt Text
D14648.id35433.diff (3 KB)

Event Timeline

Phabricator · Built by Phacility