Matthias Mullie has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/166418

Change subject: Headers have no postId
......................................................................

Headers have no postId

Change-Id: I5e1af40dd0e0ba6ab6b23795207f18fc369046a1
---
M includes/Formatter/RevisionViewFormatter.php
1 file changed, 16 insertions(+), 6 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Flow 
refs/changes/18/166418/1

diff --git a/includes/Formatter/RevisionViewFormatter.php 
b/includes/Formatter/RevisionViewFormatter.php
index 4e7171c..f4b14b4 100644
--- a/includes/Formatter/RevisionViewFormatter.php
+++ b/includes/Formatter/RevisionViewFormatter.php
@@ -2,6 +2,7 @@
 
 namespace Flow\Formatter;
 
+use Flow\Model\Header;
 use Flow\Model\PostRevision;
 use Flow\Model\PostSummary;
 use Flow\Templating;
@@ -62,12 +63,6 @@
                $links = array(
                        'hist' => $this->urlGenerator->boardHistoryLink( $title 
),
                        'board' => $this->urlGenerator->boardLink( $boardTitle 
),
-                       'single-view' => $this->urlGenerator->postRevisionLink(
-                               $title,
-                               $workflowId,
-                               $row->revision->getPostId(),
-                               $row->revision->getRevisionId()
-                       ),
                );
                $links['single-view']->setMessage( $title->getPrefixedText() );
 
@@ -77,6 +72,21 @@
                                $workflowId
                        );
                        $links['root']->setMessage( $title->getPrefixedText() );
+
+                       $links['single-view'] = 
$this->urlGenerator->postRevisionLink(
+                               $title,
+                               $workflowId,
+                               $row->revision->getPostId(),
+                               $row->revision->getRevisionId()
+                       );
+                       $links['single-view']->setMessage( 
$title->getPrefixedText() );
+               } elseif ( $row->revision instanceof Header ) {
+                       $links['single-view'] = 
$this->urlGenerator->headerRevisionLink(
+                               $title,
+                               $workflowId,
+                               $row->revision->getRevisionId()
+                       );
+                       $links['single-view']->setMessage( 
$title->getPrefixedText() );
                }
 
                if ( $row->revision->getPrevRevisionId() !== null ) {

-- 
To view, visit https://gerrit.wikimedia.org/r/166418
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I5e1af40dd0e0ba6ab6b23795207f18fc369046a1
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Flow
Gerrit-Branch: master
Gerrit-Owner: Matthias Mullie <mmul...@wikimedia.org>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to