diff --git a/src/applications/transactions/storage/PhabricatorModularTransaction.php b/src/applications/transactions/storage/PhabricatorModularTransaction.php
--- a/src/applications/transactions/storage/PhabricatorModularTransaction.php
+++ b/src/applications/transactions/storage/PhabricatorModularTransaction.php
@@ -176,7 +176,7 @@
     return parent::attachViewer($viewer);
   }
 
-  /* final */ public function hasChangeDetails() {
+  final public function hasChangeDetails() {
     if ($this->getTransactionImplementation()->hasChangeDetailView()) {
       return true;
     }
@@ -184,7 +184,7 @@
     return parent::hasChangeDetails();
   }
 
-  /* final */ public function renderChangeDetails(PhabricatorUser $viewer) {
+  final public function renderChangeDetails(PhabricatorUser $viewer) {
     $impl = $this->getTransactionImplementation();
     $impl->setViewer($viewer);
     $view = $impl->newChangeDetailView();