Page MenuHomePhorge

D25312.1734662554.diff
No OneTemporary

D25312.1734662554.diff

diff --git a/src/applications/notification/view/PhabricatorNotificationStatusView.php b/src/applications/notification/view/PhabricatorNotificationStatusView.php
--- a/src/applications/notification/view/PhabricatorNotificationStatusView.php
+++ b/src/applications/notification/view/PhabricatorNotificationStatusView.php
@@ -40,34 +40,41 @@
protected function getTagContent() {
$have = PhabricatorEnv::getEnvConfig('notification.servers');
if ($have) {
- $icon = id(new PHUIIconView())
- ->setIcon('fa-circle-o yellow');
- $text = pht('Connecting...');
- return phutil_tag(
- 'span',
- array(
- 'class' => 'connection-status-text '.
- 'aphlict-connection-status-connecting',
- ),
- array(
- $icon,
- $text,
- ));
+ return $this->getIconAndMsg(
+ 'aphlict-connection-status-connecting',
+ 'fa-circle-o yellow',
+ pht('Connecting...'));
} else {
- $text = pht('Notification server not enabled');
- $icon = id(new PHUIIconView())
- ->setIcon('fa-circle-o grey');
- return phutil_tag(
- 'span',
- array(
- 'class' => 'connection-status-text '.
- 'aphlict-connection-status-notenabled',
- ),
- array(
- $icon,
- $text,
- ));
+ return $this->getIconAndMsg(
+ 'aphlict-connection-status-notenabled',
+ 'fa-circle-o grey',
+ pht('Notification server not enabled'));
}
}
+ /**
+ * Create an icon and a text
+ *
+ * @param string $class_name Raw CSS class name(s)
+ * @param string $icon_name Icon name
+ * @param string $text Text to be shown
+ * @return array
+ */
+ private function getIconAndMsg($class_name, $icon_name, $text) {
+ $icon = id(new PHUIIconView())
+ ->setIcon($icon_name);
+
+ $message = phutil_tag(
+ 'span',
+ array(
+ 'class' => 'connection-status-text '.$class_name,
+ ),
+ $text);
+
+ return array(
+ $icon,
+ $message,
+ );
+ }
+
}

File Metadata

Mime Type
text/plain
Expires
Fri, Dec 20, 02:42 (20 h, 43 m)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
1016405
Default Alt Text
D25312.1734662554.diff (2 KB)

Event Timeline