Minor - Format code, add docs

main/admin/system_announcements.php

@@ -6,8 +6,6 @@
  *	@package chamilo.admin.announcement
  */
 
-/* INIT SECTION */
-
 // Resetting the course id.
 $cidReset = true;
 
@@ -25,7 +23,10 @@ $action_todo = false;
 api_protect_admin_script(true);
 
 // Setting breadcrumbs.
-$interbreadcrumb[] = array ("url" => 'index.php', "name" => get_lang('PlatformAdmin'));
+$interbreadcrumb[] = array(
+    "url" => 'index.php',
+    "name" => get_lang('PlatformAdmin'),
+);
 
 $tool_name = null;
 
@@ -90,8 +91,8 @@ switch($action) {
         // Add an announcement.
         $values['action'] = 'add';
         // Set default time window: NOW -> NEXT WEEK
-        $values['start'] = date('Y-m-d H:i:s',api_strtotime(api_get_local_time()));
-        $values['end']   = date('Y-m-d H:i:s',api_strtotime(api_get_local_time()) + (7 * 24 * 60 * 60));
+        $values['start'] = date('Y-m-d H:i:s', api_strtotime(api_get_local_time()));
+        $values['end']   = date('Y-m-d H:i:s', api_strtotime(api_get_local_time()) + (7 * 24 * 60 * 60));
 
         $values['range'] =
             substr(api_get_local_time(time()), 0, 16).' / '.
@@ -230,7 +231,10 @@ if ($action_todo) {
                 );
 
                 if ($announcement_id !== false) {
-                    SystemAnnouncementManager::announcement_for_groups($announcement_id, array($values['group']));
+                    SystemAnnouncementManager::announcement_for_groups(
+                        $announcement_id,
+                        array($values['group'])
+                    );
                     Display :: display_confirmation_message(get_lang('AnnouncementAdded'));
                 } else {
                     $show_announcement_list = false;
@@ -309,5 +313,5 @@ if ($show_announcement_list) {
     $table->set_form_actions($form_actions);
     $table->display();
 }
-/* FOOTER */
+
 Display :: display_footer();

main/inc/lib/template.lib.php

@@ -733,7 +733,7 @@ class Template
      */
     private function set_header_parameters()
     {
-        global $httpHeadXtra, $_course, $interbreadcrumb, $language_file, $noPHP_SELF, $_configuration, $this_section;
+        global $httpHeadXtra, $_course, $interbreadcrumb, $language_file, $_configuration, $this_section;
         $help = $this->help;
         $nameTools             = $this->title;
         $navigation            = return_navigation_array();
@@ -827,17 +827,14 @@ class Template
         }
 
         $this->assign('favico', $favico);
-
-        $this->set_help();
+        $this->setHelp();
 
         //@todo move this in the template
         $bug_notification_link = '';
         if (api_get_setting('show_link_bug_notification') == 'true' && $this->user_is_logged_in) {
             $bug_notification_link = '<li class="report">
 		        						<a href="http://support.chamilo.org/projects/chamilo-18/wiki/How_to_report_bugs" target="_blank">
-		        						<img src="'.api_get_path(WEB_IMG_PATH).'bug.large.png" style="vertical-align: middle;" alt="'.get_lang('ReportABug').'" title="'.get_lang(
-                    'ReportABug'
-                ).'"/></a>
+		        						<img src="'.api_get_path(WEB_IMG_PATH).'bug.large.png" style="vertical-align: middle;" alt="'.get_lang('ReportABug').'" title="'.get_lang('ReportABug').'"/></a>
 		    						  </li>';
         }
 
@@ -846,9 +843,9 @@ class Template
         $notification = return_notification_menu();
         $this->assign('notification_menu', $notification);
 
-        //Preparing values for the menu
+        // Preparing values for the menu
 
-        //Logout link
+        // Logout link
         $hideLogout = api_get_setting('hide_logout_button');
         if ($hideLogout === 'true') {
             $this->assign('logout_link', null);
@@ -904,7 +901,6 @@ class Template
                 GROUP_USER_PERMISSION_PENDING_INVITATION,
                 false
             );
-            $group_pending_invitations = 0;
             if (!empty($group_pending_invitations)) {
                 $group_pending_invitations = count($group_pending_invitations);
             }
@@ -967,11 +963,6 @@ class Template
      */
     private function set_footer_parameters()
     {
-        global $_configuration;
-
-        //Show admin data
-        //$this->assign('show_administrator_data', api_get_setting('show_administrator_data'));
-
         if (api_get_setting('show_administrator_data') == 'true') {
             //Administrator name
             $administrator_data = get_lang('Manager').' : '.Display::encrypted_mailto_link(
@@ -1038,8 +1029,6 @@ class Template
                 $this->assign('teachers', $teacher_data);
             }
         }
-        /* $stats = '';
-          $this->assign('execution_stats', $stats); */
     }
 
     /**
@@ -1264,12 +1253,6 @@ class Template
         $form->addButton('submitAuth', get_lang('LoginEnter'), null, 'primary', null, 'btn-block');
 
         $html = $form->returnForm();
-        // The validation is located in the local.inc
-        /*if ($form->validate()) {
-            // Prevent re-use of the same CAPTCHA phrase
-            $captcha_question->destroy();
-        }*/
-
         if (api_get_setting('openid_authentication') == 'true') {
             include_once 'main/auth/openid/login.php';
             $html .= '<div>'.openid_form().'</div>';
@@ -1292,5 +1275,4 @@ class Template
 
         $this->assign('_admin', $_admin);
     }
-
 }