Adding static declarations

Author: jmontoyaa

main/exercice/testcategory.class.php

@@ -134,7 +134,7 @@ function display($in_color="#E0EBF5") {
 	If in_field=="" Return an array of all category objects in the database
 	Otherwise, return an array of all in_field value in the database (in_field = id or name or description)
 	 */
-	public function getCategoryListInfo($in_field="", $in_courseid="") {
+	public static function getCategoryListInfo($in_field="", $in_courseid="") {
 		if (empty($in_courseid) || $in_courseid=="") {
 			$in_courseid = api_get_course_int_id();
 		}		
@@ -313,7 +313,7 @@ public static function getNumberOfQuestionRandomByCategory($in_testid, $in_nbran
 	 * tabresult[0] = get_lang('NoCategory');
 	 *
 	 */
-	function getCategoriesIdAndName($in_courseid="") {
+	static function getCategoriesIdAndName($in_courseid="") {
 		if (empty($in_courseid) || $in_courseid=="") {
 			$in_courseid = api_get_course_int_id();
 		}		

main/inc/lib/pear/HTML/QuickForm/advanced_settings.php

@@ -34,18 +34,15 @@ function HTML_QuickForm_advanced_settings($text = null)
         $this->HTML_QuickForm_static(null, null, $text);
         $this->_type = 'html';
     }
-
-    // }}}
-    // {{{ accept()
-
+    
    /**
     * Accepts a renderer
     *
     * @param HTML_QuickForm_Renderer    renderer object (only works with Default renderer!)
     * @access public
     * @return void
     */
-    function accept(&$renderer)
+    function accept(&$renderer, $required=false, $error=null)
     {
         $renderer->renderHtml($this);
     } // end func accept
@@ -61,8 +58,4 @@ function toHtml() {
                                         
                 ';
     } //end func toHtml
-
-    
-
-    // }}}
 } //end class HTML_QuickForm_html

main/inc/lib/pear/HTML/QuickForm/label.php

@@ -44,7 +44,7 @@ function HTML_QuickForm_label($label = null, $text = null) {
     * @access public
     * @return void
     */
-    function accept(&$renderer) {
+    function accept(&$renderer, $required=false, $error=null) {
         $renderer->renderHtml($this);
     }
 

main/newscorm/learnpath.class.php

@@ -6667,9 +6667,12 @@ public function display_item_form($item_type, $title = '', $action = 'add_item',
         // POSITION
         for ($i = 0; $i < count($arrLP); $i++) {
             if ($arrLP[$i]['parent_item_id'] == $parent && $arrLP[$i]['id'] != $id) {
-                if ($extra_info['previous_item_id'] == $arrLP[$i]['id'])
+                //this is the same!
+                if (isset($extra_info['previous_item_id']) && $extra_info['previous_item_id'] == $arrLP[$i]['id']) {
                     $s_selected_position = $arrLP[$i]['id'];
-                elseif ($action == 'add') $s_selected_position = $arrLP[$i]['id'];
+                } elseif ($action == 'add') { 
+                    $s_selected_position = $arrLP[$i]['id'];
+                }
 
                 $arrHide[$arrLP[$i]['id']]['value'] = get_lang('After') . ' "' . $arrLP[$i]['title'] . '"';
             }