Revision: 48616
Author:   brion
Date:     2009-03-20 01:33:05 +0000 (Fri, 20 Mar 2009)

Log Message:
-----------
Fix for r48111 "(bug 17826) Make path links clickable in CodeRevisionListView."
Links now actually work. :)
The ? was being included in the page title, thus escaped, thus failing to be in 
the query string.

Modified Paths:
--------------
    trunk/extensions/CodeReview/CodeRevisionListView.php

Modified: trunk/extensions/CodeReview/CodeRevisionListView.php
===================================================================
--- trunk/extensions/CodeReview/CodeRevisionListView.php        2009-03-20 
01:15:41 UTC (rev 48615)
+++ trunk/extensions/CodeReview/CodeRevisionListView.php        2009-03-20 
01:33:05 UTC (rev 48616)
@@ -263,8 +263,10 @@
                case 'cr_path':
                        return Xml::openElement( 'div', array( 'title' => 
(string)$value ) ) .
                                        $this->mView->mSkin->link(
-                                       SpecialPage::getTitleFor( 'Code', 
$this->mRepo->getName() . '/path?path=' . (string)$value ),
-                                       $wgLang->truncate( (string)$value, 30 ) 
) . "</div>";
+                                       SpecialPage::getTitleFor( 'Code', 
$this->mRepo->getName() . '/path' ),
+                                       $wgLang->truncate( (string)$value, 30 ),
+                                       array(),
+                                       array( 'path' => (string)$value ) ) . 
"</div>";
                }
        }
 



_______________________________________________
MediaWiki-CVS mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-cvs

Reply via email to