change double->single quotes for string literal in sql query

[lhc/web/wiklou.git] / includes / Credits.php

diff --git a/includes/Credits.php b/includes/Credits.php
index 1f370f1..a92c9f9 100644 (file)
--- a/includes/Credits.php
+++ b/includes/Credits.php
@@ -1,6 +1,7 @@
 <?php
 
-/* Credits.php -- formats credits for articles
+/**
+ * Credits.php -- formats credits for articles
  * Copyright 2004, Evan Prodromou <evan@wikitravel.org>.
  * 
  *  This program is free software; you can redistribute it and/or modify
@@ -16,26 +17,28 @@
  *  You should have received a copy of the GNU General Public License
  *  along with this program; if not, write to the Free Software
  *  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
+ *
+ * @author <evan@wikitravel.org>
  */
 
-/* This is largely cadged from PageHistory::history */
-
-function showCreditsPage($article)
-{
+/**
+ * This is largely cadged from PageHistory::history
+ */
+function showCreditsPage($article) {
     global $wgOut;
 
-    $fname = "showCreditsPage";
+    $fname = 'showCreditsPage';
     
     wfProfileIn( $fname );
 
     $wgOut->setPageTitle( $article->mTitle->getPrefixedText() );
-    $wgOut->setSubtitle( wfMsg( "creditspage" ) );
+    $wgOut->setSubtitle( wfMsg( 'creditspage' ) );
     $wgOut->setArticleFlag( false );
     $wgOut->setArticleRelated( true );
-    $wgOut->setRobotpolicy( "noindex,nofollow" );
+    $wgOut->setRobotpolicy( 'noindex,nofollow' );
 
     if( $article->mTitle->getArticleID() == 0 ) {
-       $s = wfMsg( "nocredits" );
+       $s = wfMsg( 'nocredits' );
     } else {
        $s = getCredits($article, -1);
     }
@@ -45,22 +48,24 @@ function showCreditsPage($article)
     wfProfileOut( $fname );
 }
 
-function getCredits($article, $cnt) {
+function getCredits($article, $cnt, $showIfMax=true) {
     
     $s = '';
     
     if (isset($cnt) && $cnt != 0) {
        $s = getAuthorCredits($article);
        if ($cnt > 1 || $cnt < 0) {
-           $s .= ' ' . getContributorCredits($article, $cnt - 1);
+           $s .= ' ' . getContributorCredits($article, $cnt - 1, $showIfMax);
        }
     }
     
     return $s;
 }
 
-function getAuthorCredits($article) {
-    
+/**
+ *
+ */
+function getAuthorCredits($article) {    
     global $wgLang;
     
     $last_author = $article->getUser();
@@ -88,50 +93,94 @@ function getAuthorCredits($article) {
     return wfMsg('lastmodifiedby', $d, $author_credit);
 }
 
-function getContributorCredits($article, $cnt) {
+/**
+ *
+ */
+function getContributorCredits($article, $cnt, $showIfMax) {
            
     global $wgLang, $wgAllowRealName;
     
-    $contributors = $article->getContributors($cnt);
+    $contributors = $article->getContributors();
+
+    $others_link = '';
     
+    # Hmm... too many to fit!
+
+    if ($cnt > 0 && count($contributors) > $cnt) {
+       $others_link = creditOthersLink($article);
+       if (!$showIfMax) {
+           return wfMsg('othercontribs', $others_link);
+       } else {
+           $contributors = array_slice($contributors, 0, $cnt);
+       }
+    }
+       
     $real_names = array();
     $user_names = array();
 
+    $anon = '';
+    
     # Sift for real versus user names
     
-    foreach ($contributors as $user_id => $user_parts) {
-       if ($user_id != 0) {
-           if ($wgAllowRealName && !empty($user_parts[1])) {
-               $real_names[$user_id] = creditLink($user_parts[0], $user_parts[1]);
+    foreach ($contributors as $user_parts) {
+       if ($user_parts[0] != 0) {
+           if ($wgAllowRealName && !empty($user_parts[2])) {
+               $real_names[] = creditLink($user_parts[1], $user_parts[2]);
            } else {
-               $user_names[$user_id] = creditLink($user_parts[0]);
+               $user_names[] = creditLink($user_parts[1]);
            }
+       } else {
+           $anon = wfMsg('anonymous');
        }
     }
+       
+    # Two strings: real names, and user names
     
-    $real = $wgLang->listToText(array_values($real_names));
-    $user = $wgLang->listToText(array_values($user_names));
+    $real = $wgLang->listToText($real_names);
+    $user = $wgLang->listToText($user_names);
+
+    # "ThisSite user(s) A, B and C"
     
     if (!empty($user)) {
-       $user = wfMsg('siteusers', $user);
+        $user = wfMsg('siteusers', $user);
     }
+
+    # This is the big list, all mooshed together. We sift for blank strings
     
-    if ($contributors[0] && $contributors[0][0] > 0) {
-       $anon = wfMsg('anonymous');
-    } else {
-       $anon = '';
+    $fulllist = array();
+      
+    foreach (array($real, $user, $anon, $others_link) as $s) {
+       if (!empty($s)) {
+           array_push($fulllist, $s);
+       }
     }
+
+    # Make the list into text...
     
-    $creds = $wgLang->listToText(array($real, $user, $anon));
+    $creds = $wgLang->listToText($fulllist);
+
+    # "Based on work by ..."
     
-    return wfMsg('othercontribs', $creds);
+    return (empty($creds)) ? '' : wfMsg('othercontribs', $creds);
 }
 
+/**
+ *
+ */
 function creditLink($user_name, $link_text = '') {
     global $wgUser, $wgLang;
     $skin = $wgUser->getSkin();
-    return $skin->makeKnownLink($wgLang->getNsText(NS_USER) . ":" . $user_name,
-                               (empty($link_text)) ? $user_name : $link_text);
+    return $skin->makeLink($wgLang->getNsText(NS_USER) . ':' . $user_name,
+                          (empty($link_text)) ? $user_name : $link_text);
+}
+
+/**
+ *
+ */
+function creditOthersLink($article) {
+    global $wgUser, $wgLang;
+    $skin = $wgUser->getSkin();
+    return $skin->makeKnownLink($article->mTitle->getPrefixedText(), wfMsg('others'), 'action=credits');
 }
 
 ?>