Kill some PhutilSafeHTML

Summary: Also couple of unrelated Ponder changes.

Test Plan: /Q5

Reviewers: epriestley

Reviewed By: epriestley

CC: aran, Korvin

Differential Revision: https://secure.phabricator.com/D4858

resources/sql/quickstart.sql

@@ -1666,7 +1666,7 @@ CREATE TABLE `ponder_question` (
   UNIQUE KEY `phid` (`phid`),
   KEY `authorPHID` (`authorPHID`),
   KEY `heat` (`heat`)
-) ENGINE=InnoDB DEFAULT CHARSET=utf8;
+) ENGINE=InnoDB DEFAULT CHARSET=utf8 AUTO_INCREMENT=11;
 
 
 CREATE DATABASE `{$NAMESPACE}_project` /*!40100 DEFAULT CHARACTER SET utf8 */;

src/applications/config/view/PhabricatorSetupIssueView.php

@@ -22,7 +22,7 @@ final class PhabricatorSetupIssueView extends AphrontView {
       array(
         'class' => 'setup-issue-instructions',
       ),
-      new PhutilSafeHTML(nl2br(phutil_escape_html($issue->getMessage()))));
+      phutil_escape_html_newlines($issue->getMessage()));
 
     $configs = $issue->getPHPConfig();
     if ($configs) {

src/applications/directory/controller/PhabricatorDirectoryMainController.php

@@ -316,9 +316,10 @@ final class PhabricatorDirectoryMainController
       array(
         'class' => 'phabricator-jump-nav-caption',
       ),
-      new PhutilSafeHTML(
+      hsprintf(
         'Enter the name of an object like <tt>D123</tt> to quickly jump to '.
-        'it. See '.$doc_link.' or type <tt>help</tt>.'));
+          'it. See %s or type <tt>help</tt>.',
+        $doc_link));
 
     $panel = new AphrontPanelView();
     $panel->setHeader('Jump Nav');

src/applications/ponder/application/PhabricatorApplicationPonder.php

@@ -39,7 +39,7 @@ final class PhabricatorApplicationPonder extends PhabricatorApplication {
     return true;
   }
 
-  public function getroutes() {
+  public function getRoutes() {
     return array(
       '/Q(?P<id>[1-9]\d*)' => 'PonderQuestionViewController',
       '/ponder/' => array(
@@ -53,7 +53,8 @@ final class PhabricatorApplicationPonder extends PhabricatorApplication {
         'comment/add/' => 'PonderCommentSaveController',
         '(?P<kind>question)/vote/' => 'PonderVoteSaveController',
         '(?P<kind>answer)/vote/' => 'PonderVoteSaveController'
-      ));
+      ),
+    );
   }
-}
 
+}

src/applications/ponder/view/PonderCommentListView.php

@@ -51,24 +51,20 @@ final class PonderCommentListView extends AphrontView {
         $comment->getMarkupField(),
         $this->user);
 
-      $comment_anchor = '<a name="comment-' . $comment->getID() . '" />';
+      $comment_anchor = '';
-      $comment_markup[] =
+      $comment_markup[] = hsprintf(
         '<tr>'.
-          '<th>'.
+          '<th><a name="comment-%s" /></th>'.
-            $comment_anchor.
-          '</th>'.
           '<td>'.
             '<div class="phabricator-remarkup ponder-comment-markup">'.
-              $body.
+              '%s&nbsp;&mdash;%s&nbsp;<span class="ponder-datestamp">%s</span>'.
-              '&nbsp;&mdash;'.
-              $handle->renderLink().
-              '&nbsp;'.
-              '<span class="ponder-datestamp">'.
-                phabricator_datetime($comment->getDateCreated(), $user).
-              '</span>'.
             '</div>'.
           '</td>'.
-        '</tr>';
+        '</tr>',
+        $comment->getID(),
+        $body,
+        $handle->renderLink(),
+        phabricator_datetime($comment->getDateCreated(), $user));
     }
 
     $addview = id(new PonderAddCommentView)
@@ -77,20 +73,16 @@ final class PonderCommentListView extends AphrontView {
       ->setQuestionID($this->questionID)
       ->setActionURI($this->actionURI);
 
-    $comment_markup[] =
+    $comment_markup[] = hsprintf(
-      '<tr>'.
+      '<tr><th>&nbsp;</th><td>%s</td></tr>',
-       '<th>&nbsp;</th>'.
+      $addview->render());
-       '<td>'.$addview->render().'</td>'.
-      '</tr>';
 
     $comment_markup = phutil_tag(
       'table',
       array(
         'class' => 'ponder-comments',
       ),
-      new PhutilSafeHTML(implode("\n", $comment_markup))
+      $comment_markup);
-    );
-
 
     return $comment_markup;
   }