[svn r16480] fix start wiki

improvement: migrate shows from list to tables (first try in the most visited pages)

main/wiki/index.php

@@ -35,8 +35,7 @@ require_once (api_get_path(LIBRARY_PATH).'events.lib.inc.php');
 require_once (api_get_path(LIBRARY_PATH).'security.lib.php'); 
 require_once(api_get_path(INCLUDE_PATH).'lib/mail.lib.inc.php');
 require_once(api_get_path(INCLUDE_PATH).'conf/mail.conf.php');
-
-
+require_once (api_get_path(LIBRARY_PATH).'sortabletable.class.php');
 /*
 -----------------------------------------------------------
   			ADDITIONAL STYLE INFORMATION
@@ -544,37 +543,66 @@ if ($_GET['action']=='mvisited')
 	echo '<b>'.get_lang('MostVisitedPages').'</b><br>'; 
 	echo '<hr>';	
 	
-	
 	if(api_is_allowed_to_edit() || api_is_platform_admin()) //only by professors if page is hidden
 	{
-			$sql='SELECT *, SUM(hits) AS tsum FROM '.$tbl_wiki.'  WHERE  '.$groupfilter.' GROUP BY reflink ORDER BY tsum DESC, reflink LIMIT 10'; //first ten pages	
+		$sql='SELECT *, SUM(hits) AS tsum FROM '.$tbl_wiki.'  WHERE  '.$groupfilter.' GROUP BY reflink ORDER BY tsum DESC, reflink';
 	}
 	else
 	{			
-			$sql='SELECT *, SUM(hits) AS tsum FROM '.$tbl_wiki.'  WHERE  '.$groupfilter.' AND visibility=1 GROUP BY reflink ORDER BY tsum DESC, reflink LIMIT 10'; //first ten pages
+		$sql='SELECT *, SUM(hits) AS tsum FROM '.$tbl_wiki.'  WHERE  '.$groupfilter.' AND visibility=1 GROUP BY reflink ORDER BY tsum DESC, reflink';
 	}		
 	
 	$allpages=api_sql_query($sql,__FILE__,__LINE__);
-	echo '<ul>';	
-	while ($row=Database::fetch_array($allpages))
+
+	//show table
+	if (mysql_num_rows($allpages) > 0)
 	{
-		//fix assignment icon		
-		if($row['assignment']==1)
+		$row = array ();
+		while ($obj = mysql_fetch_object($allpages))
 		{
-			$ShowAssignment='<img src="../img/wiki/assignment.gif" />';
+			//get author
+			$userinfo=Database::get_user_info_from_id($obj->user_id);
+			
+			//get time
+			$year 	 = substr($obj->timestamp, 0, 4);
+			$month	 = substr($obj->timestamp, 5, 2);
+			$day 	 = substr($obj->timestamp, 8, 2);
+			$hours   = substr($obj->timestamp, 11,2);
+			$minutes = substr($obj->timestamp, 14,2);
+			$seconds = substr($obj->timestamp, 17,2);			
+			
+			//get type assignment icon		
+			if($obj->assignment==1)
+			{
+				$ShowAssignment='<img src="../img/wiki/assignment.gif" alt="'.get_lang('AssignmentDesc').'" />';
 			}
-		elseif ($row['assignment']==2)
+			elseif ($obj->assignment==2)
 			{
-			$ShowAssignment='<img src="../img/wiki/works.gif" />'; 
+				$ShowAssignment='<img src="../img/wiki/works.gif" alt="'.get_lang('AssignmentWork').'" />';
 			}
-		elseif ($row['assignment']==0)
+			elseif ($obj->assignment==0)
 			{	
 				$ShowAssignment='<img src="../img/wiki/trans.gif" />';			
 			}		
 		
-		echo '<li>'.$ShowAssignment.'<a href="'.$_SERVER['PHP_SELF'].'?cidReq='.$_course[id].'&action=showpage&title='.urlencode($row['reflink']).'&group_id='.Security::remove_XSS($_GET['group_id']).'">'.$row['title'].'</a> '.get_lang('With').' '.$row['tsum'].' '.get_lang('Visits').'</li>';	//TODO:check if hidden and assignment mode
+			$row = array ();
+			$row[] =$ShowAssignment;
+			$row[] = '<a href="'.$_SERVER['PHP_SELF'].'?cidReq='.$_course[id].'&action=showpage&title='.urlencode($obj->reflink).'&group_id='.Security::remove_XSS($_GET['group_id']).'">'.$obj->title.'</a>';			
+			$row[] ='<a href="../user/userInfo.php?uInfo='.$userinfo['user_id'].'">'.$userinfo['lastname'].', '.$userinfo['firstname'].'</a>';
+			$row[] = $day.' '.$MonthsLong[$month-1].' '.$year.' '.$hours.":".$minutes.":".$seconds;
+			$row[] = $obj->tsum;			
+			$rows[] = $row;
+		}
+	
+		$table = new SortableTableFromArrayConfig($rows,4,10,'MostVisitedPages_table','','','DESC');
+		$table->set_additional_parameters(array('cidReq' =>$_GET['cidReq'],'action'=>$_GET['action'],'group_id'=>Security::remove_XSS($_GET['group_id'])));		
+		$table->set_header(0,get_lang('Type'), true, array ('style' => 'width:30px;'));
+		$table->set_header(1,get_lang('Title'), true);
+		$table->set_header(2,get_lang('Author'), true);
+		$table->set_header(3,get_lang('Date'), true);
+		$table->set_header(4,get_lang('Visits'), true);
+		$table->display();
 	}	
-	echo '</ul>';
 }
 
 /////////////////////// Wanted pages /////////////////////// Juan Carlos Raña Trabado
@@ -1051,7 +1079,7 @@ if ($_GET['action']=='edit')
 
 
 	//Only teachers and platform admin can edit the index page. Only teachers and platform admin can edit an assignment teacher
-	if(($row['reflink']=='index' || $row['reflink']=='' || $row['assignment']==1) && (!api_is_allowed_to_edit() || !api_is_platform_admin()))
+	if(($row['reflink']=='index' || $row['reflink']=='' || $row['assignment']==1) && (!api_is_allowed_to_edit()))
 	{
   
       Display::display_normal_message(get_lang('OnlyEditPagesCourseManager'));