Changeset View
Changeset View
Standalone View
Standalone View
src/applications/phame/storage/PhameBlogTransaction.php
<?php | <?php | ||||
final class PhameBlogTransaction | final class PhameBlogTransaction | ||||
extends PhabricatorApplicationTransaction { | extends PhabricatorApplicationTransaction { | ||||
const TYPE_NAME = 'phame.blog.name'; | const TYPE_NAME = 'phame.blog.name'; | ||||
const TYPE_SUBTITLE = 'phame.blog.subtitle'; | const TYPE_SUBTITLE = 'phame.blog.subtitle'; | ||||
const TYPE_DESCRIPTION = 'phame.blog.description'; | const TYPE_DESCRIPTION = 'phame.blog.description'; | ||||
const TYPE_FULLDOMAIN = 'phame.blog.full.domain'; | const TYPE_FULLDOMAIN = 'phame.blog.full.domain'; | ||||
const TYPE_STATUS = 'phame.blog.status'; | const TYPE_STATUS = 'phame.blog.status'; | ||||
const TYPE_PARENTSITE = 'phame.blog.parent.site'; | const TYPE_PARENTSITE = 'phame.blog.parent.site'; | ||||
const TYPE_PARENTDOMAIN = 'phame.blog.parent.domain'; | const TYPE_PARENTDOMAIN = 'phame.blog.parent.domain'; | ||||
const TYPE_PROFILEIMAGE = 'phame.blog.header.image'; | |||||
const TYPE_HEADERIMAGE = 'phame.blog.profile.image'; | |||||
const MAILTAG_DETAILS = 'phame-blog-details'; | const MAILTAG_DETAILS = 'phame-blog-details'; | ||||
const MAILTAG_SUBSCRIBERS = 'phame-blog-subscribers'; | const MAILTAG_SUBSCRIBERS = 'phame-blog-subscribers'; | ||||
const MAILTAG_OTHER = 'phame-blog-other'; | const MAILTAG_OTHER = 'phame-blog-other'; | ||||
public function getApplicationName() { | public function getApplicationName() { | ||||
return 'phame'; | return 'phame'; | ||||
} | } | ||||
public function getApplicationTransactionType() { | public function getApplicationTransactionType() { | ||||
return PhabricatorPhameBlogPHIDType::TYPECONST; | return PhabricatorPhameBlogPHIDType::TYPECONST; | ||||
} | } | ||||
public function shouldHide() { | public function shouldHide() { | ||||
$old = $this->getOldValue(); | $old = $this->getOldValue(); | ||||
switch ($this->getTransactionType()) { | switch ($this->getTransactionType()) { | ||||
case self::TYPE_DESCRIPTION: | case self::TYPE_DESCRIPTION: | ||||
if ($old === null) { | if ($old === null) { | ||||
return true; | return true; | ||||
} | } | ||||
} | } | ||||
return parent::shouldHide(); | return parent::shouldHide(); | ||||
} | } | ||||
public function getRequiredHandlePHIDs() { | |||||
$old = $this->getOldValue(); | |||||
$new = $this->getNewValue(); | |||||
$req_phids = array(); | |||||
switch ($this->getTransactionType()) { | |||||
case self::TYPE_PROFILEIMAGE: | |||||
case self::TYPE_HEADERIMAGE: | |||||
$req_phids[] = $old; | |||||
$req_phids[] = $new; | |||||
break; | |||||
} | |||||
return array_merge($req_phids, parent::getRequiredHandlePHIDs()); | |||||
} | |||||
public function getIcon() { | public function getIcon() { | ||||
$old = $this->getOldValue(); | $old = $this->getOldValue(); | ||||
$new = $this->getNewValue(); | $new = $this->getNewValue(); | ||||
switch ($this->getTransactionType()) { | switch ($this->getTransactionType()) { | ||||
case self::TYPE_NAME: | case self::TYPE_NAME: | ||||
if ($old === null) { | if ($old === null) { | ||||
return 'fa-plus'; | return 'fa-plus'; | ||||
} else { | } else { | ||||
return 'fa-pencil'; | return 'fa-pencil'; | ||||
} | } | ||||
break; | break; | ||||
case self::TYPE_DESCRIPTION: | case self::TYPE_DESCRIPTION: | ||||
case self::TYPE_FULLDOMAIN: | case self::TYPE_FULLDOMAIN: | ||||
return 'fa-pencil'; | return 'fa-pencil'; | ||||
case self::TYPE_HEADERIMAGE: | |||||
return 'fa-image'; | |||||
case self::TYPE_PROFILEIMAGE: | |||||
return 'fa-star'; | |||||
case self::TYPE_STATUS: | case self::TYPE_STATUS: | ||||
if ($new == PhameBlog::STATUS_ARCHIVED) { | if ($new == PhameBlog::STATUS_ARCHIVED) { | ||||
return 'fa-ban'; | return 'fa-ban'; | ||||
} else { | } else { | ||||
return 'fa-check'; | return 'fa-check'; | ||||
} | } | ||||
break; | break; | ||||
} | } | ||||
Show All 24 Lines | switch ($this->getTransactionType()) { | ||||
$tags[] = self::MAILTAG_SUBSCRIBERS; | $tags[] = self::MAILTAG_SUBSCRIBERS; | ||||
break; | break; | ||||
case self::TYPE_NAME: | case self::TYPE_NAME: | ||||
case self::TYPE_SUBTITLE: | case self::TYPE_SUBTITLE: | ||||
case self::TYPE_DESCRIPTION: | case self::TYPE_DESCRIPTION: | ||||
case self::TYPE_FULLDOMAIN: | case self::TYPE_FULLDOMAIN: | ||||
case self::TYPE_PARENTSITE: | case self::TYPE_PARENTSITE: | ||||
case self::TYPE_PARENTDOMAIN: | case self::TYPE_PARENTDOMAIN: | ||||
case self::TYPE_PROFILEIMAGE: | |||||
case self::TYPE_HEADERIMAGE: | |||||
$tags[] = self::MAILTAG_DETAILS; | $tags[] = self::MAILTAG_DETAILS; | ||||
break; | break; | ||||
default: | default: | ||||
$tags[] = self::MAILTAG_OTHER; | $tags[] = self::MAILTAG_OTHER; | ||||
break; | break; | ||||
} | } | ||||
return $tags; | return $tags; | ||||
} | } | ||||
▲ Show 20 Lines • Show All 68 Lines • ▼ Show 20 Lines | switch ($type) { | ||||
$new); | $new); | ||||
} else { | } else { | ||||
return pht( | return pht( | ||||
'%s updated the blog\'s parent domain to "%s".', | '%s updated the blog\'s parent domain to "%s".', | ||||
$this->renderHandleLink($author_phid), | $this->renderHandleLink($author_phid), | ||||
$new); | $new); | ||||
} | } | ||||
break; | break; | ||||
case self::TYPE_HEADERIMAGE: | |||||
if (!$old) { | |||||
return pht( | |||||
"%s set this blog's header image to %s.", | |||||
$this->renderHandleLink($author_phid), | |||||
epriestley: Should these be renderHandleLink()? | |||||
Not Done Inline ActionsI'm breaking something, can't figure it out.
chad: I'm breaking something, can't figure it out.
> Transaction ("PHID-XACT-BLOG-dtq3uozqaazuqdx"… | |||||
$this->renderHandleLink($new)); | |||||
} else if (!$new) { | |||||
return pht( | |||||
"%s removed this blog's header image.", | |||||
$this->renderHandleLink($author_phid)); | |||||
} else { | |||||
return pht( | |||||
"%s updated this blog's header image from %s to %s.", | |||||
$this->renderHandleLink($author_phid), | |||||
$this->renderHandleLink($old), | |||||
$this->renderHandleLink($new)); | |||||
} | |||||
break; | |||||
case self::TYPE_PROFILEIMAGE: | |||||
if (!$old) { | |||||
return pht( | |||||
"%s set this blog's profile image to %s.", | |||||
$this->renderHandleLink($author_phid), | |||||
$this->renderHandleLink($new)); | |||||
} else if (!$new) { | |||||
return pht( | |||||
"%s removed this blog's profile image.", | |||||
$this->renderHandleLink($author_phid)); | |||||
} else { | |||||
return pht( | |||||
"%s updated this blog's profile image from %s to %s.", | |||||
$this->renderHandleLink($author_phid), | |||||
$this->renderHandleLink($old), | |||||
$this->renderHandleLink($new)); | |||||
} | |||||
break; | |||||
case self::TYPE_STATUS: | case self::TYPE_STATUS: | ||||
switch ($new) { | switch ($new) { | ||||
case PhameBlog::STATUS_ACTIVE: | case PhameBlog::STATUS_ACTIVE: | ||||
return pht( | return pht( | ||||
'%s published this blog.', | '%s published this blog.', | ||||
$this->renderHandleLink($author_phid)); | $this->renderHandleLink($author_phid)); | ||||
case PhameBlog::STATUS_ARCHIVED: | case PhameBlog::STATUS_ARCHIVED: | ||||
return pht( | return pht( | ||||
▲ Show 20 Lines • Show All 60 Lines • ▼ Show 20 Lines | switch ($type) { | ||||
$this->renderHandleLink($object_phid)); | $this->renderHandleLink($object_phid)); | ||||
break; | break; | ||||
case self::TYPE_PARENTDOMAIN: | case self::TYPE_PARENTDOMAIN: | ||||
return pht( | return pht( | ||||
'%s updated the parent domain for %s.', | '%s updated the parent domain for %s.', | ||||
$this->renderHandleLink($author_phid), | $this->renderHandleLink($author_phid), | ||||
$this->renderHandleLink($object_phid)); | $this->renderHandleLink($object_phid)); | ||||
break; | break; | ||||
case self::TYPE_HEADERIMAGE: | |||||
return pht( | |||||
'%s updated the header image for %s.', | |||||
$this->renderHandleLink($author_phid), | |||||
$this->renderHandleLink($object_phid)); | |||||
break; | |||||
case self::TYPE_PROFILEIMAGE: | |||||
return pht( | |||||
'%s updated the profile image for %s.', | |||||
$this->renderHandleLink($author_phid), | |||||
$this->renderHandleLink($object_phid)); | |||||
break; | |||||
case self::TYPE_STATUS: | case self::TYPE_STATUS: | ||||
switch ($new) { | switch ($new) { | ||||
case PhameBlog::STATUS_ACTIVE: | case PhameBlog::STATUS_ACTIVE: | ||||
return pht( | return pht( | ||||
'%s published the blog %s.', | '%s published the blog %s.', | ||||
$this->renderHandleLink($author_phid), | $this->renderHandleLink($author_phid), | ||||
$this->renderHandleLink($object_phid)); | $this->renderHandleLink($object_phid)); | ||||
case PhameBlog::STATUS_ARCHIVED: | case PhameBlog::STATUS_ARCHIVED: | ||||
Show All 37 Lines |
Should these be renderHandleLink()?