chamilo / chamilo-lms

main/user/add_users_to_session.php

Indentation +106 added lines, -106 removed lines

@@ -48,29 +48,29 @@ 
     $extra_field_list= UserManager::get_extra_fields();
     $new_field_list = array();
     if (is_array($extra_field_list)) {
-    	foreach ($extra_field_list as $extra_field) {
-    		//if is enabled to filter and is a "<select>" field type
-    		if ($extra_field[8]==1 && $extra_field[2]==4 ) {
+        foreach ($extra_field_list as $extra_field) {
+            //if is enabled to filter and is a "<select>" field type
+            if ($extra_field[8]==1 && $extra_field[2]==4 ) {
                 $new_field_list[] = array(
                     'name' => $extra_field[3],
                     'variable' => $extra_field[1],
                     'data' => $extra_field[9],
                 );
-    		}
-    	}
+            }
+        }
     }
 
     function search_users($needle, $type)
     {
-    	global $id_session;
+        global $id_session;
 
         $tbl_user = Database::get_main_table(TABLE_MAIN_USER);
         $tbl_session_rel_user = Database::get_main_table(TABLE_MAIN_SESSION_USER);
 
-    	$xajax_response = new xajaxResponse();
-    	$return = '';
+        $xajax_response = new xajaxResponse();
+        $return = '';
 
-    	if (!empty($needle) && !empty($type)) {
+        if (!empty($needle) && !empty($type)) {
 
             //normal behaviour
             if ($type == 'any_session' && $needle == 'false')  {
@@ -78,32 +78,32 @@ 
                 $needle = '';
             }
 
-    		// xajax send utf8 datas... datas in db can be non-utf8 datas
-    		$charset = api_get_system_encoding();
-    		$needle = Database::escape_string($needle);
-    		$needle = api_convert_encoding($needle, $charset, 'utf-8');
+            // xajax send utf8 datas... datas in db can be non-utf8 datas
+            $charset = api_get_system_encoding();
+            $needle = Database::escape_string($needle);
+            $needle = api_convert_encoding($needle, $charset, 'utf-8');
 
-    		$order_clause = api_sort_by_first_name() ? ' ORDER BY firstname, lastname, username' : ' ORDER BY lastname, firstname, username';
-    		$cond_user_id = '';
+            $order_clause = api_sort_by_first_name() ? ' ORDER BY firstname, lastname, username' : ' ORDER BY lastname, firstname, username';
+            $cond_user_id = '';
 
             //Only for single & multiple
             if (in_array($type, array('single','multiple'))) {
-        		if (!empty($id_session)) {
-        		    $id_session = intval($id_session);
-        			// check id_user from session_rel_user table
-        			$sql = 'SELECT user_id FROM '.$tbl_session_rel_user.'
+                if (!empty($id_session)) {
+                    $id_session = intval($id_session);
+                    // check id_user from session_rel_user table
+                    $sql = 'SELECT user_id FROM '.$tbl_session_rel_user.'
         			        WHERE session_id ="'.$id_session.'" AND relation_type<>'.SESSION_RELATION_TYPE_RRHH.' ';
-        			$res = Database::query($sql);
-        			$user_ids = array();
-        			if (Database::num_rows($res) > 0) {
-        				while ($row = Database::fetch_row($res)) {
-        					$user_ids[] = (int)$row[0];
-        				}
-        			}
-        			if (count($user_ids) > 0) {
-        				$cond_user_id = ' AND user.user_id NOT IN('.implode(",",$user_ids).')';
-        			}
-        		}
+                    $res = Database::query($sql);
+                    $user_ids = array();
+                    if (Database::num_rows($res) > 0) {
+                        while ($row = Database::fetch_row($res)) {
+                            $user_ids[] = (int)$row[0];
+                        }
+                    }
+                    if (count($user_ids) > 0) {
+                        $cond_user_id = ' AND user.user_id NOT IN('.implode(",",$user_ids).')';
+                    }
+                }
             }
 
             switch ($type) {
@@ -135,11 +135,11 @@ 
                                 user.status <> 6 '.$cond_user_id.
                             $order_clause;
                     break;
-    		}
-    		if (api_is_multiple_url_enabled()) {
-    			$tbl_user_rel_access_url= Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_USER);
-    			$access_url_id = api_get_current_access_url_id();
-    			if ($access_url_id != -1) {
+            }
+            if (api_is_multiple_url_enabled()) {
+                $tbl_user_rel_access_url= Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_USER);
+                $access_url_id = api_get_current_access_url_id();
+                if ($access_url_id != -1) {
                     switch ($type) {
                         case 'single':
                             $sql = 'SELECT user.user_id, username, lastname, firstname
@@ -177,36 +177,36 @@ 
                                         user.status<>6 '.$cond_user_id.
                             $order_clause;
                             break;
-    				}
-    			}
-    		}
+                    }
+                }
+            }
 
-    		$rs = Database::query($sql);
+            $rs = Database::query($sql);
             $i=0;
-    		if ($type == 'single') {
-    			while ($user = Database :: fetch_array($rs)) {
-    	            $i++;
-    	            if ($i<=10) {
-                		$person_name = api_get_person_name($user['firstname'], $user['lastname']);
-    					$return .= '<a href="javascript: void(0);" onclick="javascript: add_user_to_session(\''.$user['user_id'].'\',\''.$person_name.' ('.$user['username'].')'.'\')">'.$person_name.' ('.$user['username'].')</a><br />';
-    	            } else {
-    	            	$return .= '...<br />';
-    	            }
-    			}
-
-    			$xajax_response -> addAssign('ajax_list_users_single','innerHTML',api_utf8_encode($return));
-    		} else {
-    			global $nosessionUsersList;
-    			$return .= '<select id="origin_users" name="nosessionUsersList[]" multiple="multiple" size="15" style="width:360px;">';
-    			while ($user = Database :: fetch_array($rs)) {
-    				$person_name = api_get_person_name($user['firstname'], $user['lastname']);
-    	            $return .= '<option value="'.$user['user_id'].'">'.$person_name.' ('.$user['username'].')</option>';
-    			}
-    			$return .= '</select>';
-    			$xajax_response -> addAssign('ajax_list_users_multiple','innerHTML',api_utf8_encode($return));
-    		}
-    	}
-    	return $xajax_response;
+            if ($type == 'single') {
+                while ($user = Database :: fetch_array($rs)) {
+                    $i++;
+                    if ($i<=10) {
+                        $person_name = api_get_person_name($user['firstname'], $user['lastname']);
+                        $return .= '<a href="javascript: void(0);" onclick="javascript: add_user_to_session(\''.$user['user_id'].'\',\''.$person_name.' ('.$user['username'].')'.'\')">'.$person_name.' ('.$user['username'].')</a><br />';
+                    } else {
+                        $return .= '...<br />';
+                    }
+                }
+
+                $xajax_response -> addAssign('ajax_list_users_single','innerHTML',api_utf8_encode($return));
+            } else {
+                global $nosessionUsersList;
+                $return .= '<select id="origin_users" name="nosessionUsersList[]" multiple="multiple" size="15" style="width:360px;">';
+                while ($user = Database :: fetch_array($rs)) {
+                    $person_name = api_get_person_name($user['firstname'], $user['lastname']);
+                    $return .= '<option value="'.$user['user_id'].'">'.$person_name.' ('.$user['username'].')</option>';
+                }
+                $return .= '</select>';
+                $xajax_response -> addAssign('ajax_list_users_multiple','innerHTML',api_utf8_encode($return));
+            }
+        }
+        return $xajax_response;
     }
 
     $xajax -> processRequests();
@@ -342,8 +342,8 @@ 
 
         if ($use_extra_fields) {
             $final_result = array();
-           	if (count($extra_field_result)>1) {
-    	    for($i=0;$i<count($extra_field_result)-1;$i++) {
+                if (count($extra_field_result)>1) {
+            for($i=0;$i<count($extra_field_result)-1;$i++) {
                     if (is_array($extra_field_result[$i+1])) {
                         $final_result = array_intersect(
                             $extra_field_result[$i],
@@ -421,7 +421,7 @@ 
                     'un' => $user['username']
                 );
                 unset($users[$uid]);
-    	}
+        }
         }
         unset($users); //clean to free memory
 
@@ -466,13 +466,13 @@ 
     }
 
     if ($add_type == 'multiple') {
-    	$link_add_type_unique = '<a href="'.api_get_self().'?id_session='.$id_session.'&add='.Security::remove_XSS($_GET['add']).'&add_type=unique">'.Display::return_icon('single.gif').get_lang('SessionAddTypeUnique').'</a>';
-    	$link_add_type_multiple = Display::return_icon('multiple.gif').get_lang('SessionAddTypeMultiple');
+        $link_add_type_unique = '<a href="'.api_get_self().'?id_session='.$id_session.'&add='.Security::remove_XSS($_GET['add']).'&add_type=unique">'.Display::return_icon('single.gif').get_lang('SessionAddTypeUnique').'</a>';
+        $link_add_type_multiple = Display::return_icon('multiple.gif').get_lang('SessionAddTypeMultiple');
     } else {
-    	$link_add_type_unique = Display::return_icon('single.gif').get_lang('SessionAddTypeUnique');
-    	$link_add_type_multiple = '<a href="'.api_get_self().'?id_session='.$id_session.'&amp;add='.Security::remove_XSS($_GET['add']).'&amp;add_type=multiple">'.Display::return_icon('multiple.gif').get_lang('SessionAddTypeMultiple').'</a>';
+        $link_add_type_unique = Display::return_icon('single.gif').get_lang('SessionAddTypeUnique');
+        $link_add_type_multiple = '<a href="'.api_get_self().'?id_session='.$id_session.'&amp;add='.Security::remove_XSS($_GET['add']).'&amp;add_type=multiple">'.Display::return_icon('multiple.gif').get_lang('SessionAddTypeMultiple').'</a>';
     }
-    	$link_add_group = '<a href="usergroups.php">'.Display::return_icon('multiple.gif',get_lang('RegistrationByUsersGroups')).get_lang('RegistrationByUsersGroups').'</a>';
+        $link_add_group = '<a href="usergroups.php">'.Display::return_icon('multiple.gif',get_lang('RegistrationByUsersGroups')).get_lang('RegistrationByUsersGroups').'</a>';
     ?>
     <div class="actions">
     	<?php echo $link_add_type_unique ?>&nbsp;|&nbsp;<?php echo $link_add_type_multiple ?>&nbsp;|&nbsp;<?php echo $link_add_group; ?>
@@ -481,30 +481,30 @@ 
     <?php echo '<legend>'.$tool_name.' ('.$session_info['name'].') </legend>'; ?>
     <?php
     if ($add_type == 'multiple') {
-    	if (is_array($extra_field_list)) {
-    		if (is_array($new_field_list) && count($new_field_list)>0 ) {
-    			echo '<h3>'.get_lang('FilterUsers').'</h3>';
-    			foreach ($new_field_list as $new_field) {
-    				echo $new_field['name'];
-    				$varname = 'field_'.$new_field['variable'];
-    				echo '&nbsp;<select name="'.$varname.'">';
-    				echo '<option value="0">--'.get_lang('Select').'--</option>';
-    				foreach	($new_field['data'] as $option) {
-    					$checked='';
-    					if (isset($_POST[$varname])) {
-    						if ($_POST[$varname] == $option[1]) {
-    							$checked = 'selected="true"';
-    						}
-    					}
-    					echo '<option value="'.$option[1].'" '.$checked.'>'.$option[1].'</option>';
-    				}
-    				echo '</select>';
-    				echo '&nbsp;&nbsp;';
-    			}
-    			echo '<input type="button" value="'.get_lang('Filter').'" onclick="validate_filter()" />';
-    			echo '<br /><br />';
-    		}
-    	}
+        if (is_array($extra_field_list)) {
+            if (is_array($new_field_list) && count($new_field_list)>0 ) {
+                echo '<h3>'.get_lang('FilterUsers').'</h3>';
+                foreach ($new_field_list as $new_field) {
+                    echo $new_field['name'];
+                    $varname = 'field_'.$new_field['variable'];
+                    echo '&nbsp;<select name="'.$varname.'">';
+                    echo '<option value="0">--'.get_lang('Select').'--</option>';
+                    foreach	($new_field['data'] as $option) {
+                        $checked='';
+                        if (isset($_POST[$varname])) {
+                            if ($_POST[$varname] == $option[1]) {
+                                $checked = 'selected="true"';
+                            }
+                        }
+                        echo '<option value="'.$option[1].'" '.$checked.'>'.$option[1].'</option>';
+                    }
+                    echo '</select>';
+                    echo '&nbsp;&nbsp;';
+                }
+                echo '<input type="button" value="'.get_lang('Filter').'" onclick="validate_filter()" />';
+                echo '<br /><br />';
+            }
+        }
     }
     ?>
 
@@ -513,7 +513,7 @@ 
 
     <?php
     if (!empty($errorMsg)) {
-    	Display::display_normal_message($errorMsg); //main API
+        Display::display_normal_message($errorMsg); //main API
     }
     ?>
     <div class="row">
@@ -534,7 +534,7 @@ 
                 <div id="content_source">
                 <?php
                 if (!($add_type == 'multiple')) {
-                  ?>
+                    ?>
                   <input type="text" id="user_to_add" onkeyup="xajax_search_users(this.value,'single')" />
                   <div id="ajax_list_users_single"></div>
                   <?php
@@ -543,12 +543,12 @@ 
                 <div id="ajax_list_users_multiple">
                 <select id="origin_users" name="nosessionUsersList[]" multiple="multiple" size="15" class="span5">
                   <?php
-                  foreach ($nosessionUsersList as $uid => $enreg) {
-                  ?>
+                    foreach ($nosessionUsersList as $uid => $enreg) {
+                    ?>
                       <option value="<?php echo $uid; ?>" <?php if(in_array($uid,$UserList)) echo 'selected="selected"'; ?>><?php echo api_get_person_name($enreg['fn'], $enreg['ln']).' ('.$enreg['un'].')'; ?></option>
                   <?php
-                  }
-                  ?>
+                    }
+                    ?>
                 </select>
                 </div>
                     <input type="checkbox" onchange="checked_in_no_session(this.checked);" name="user_with_any_session" id="user_with_any_session_id">
@@ -556,7 +556,7 @@ 
                 <?php
                 }
                 unset($nosessionUsersList);
-               ?>
+                ?>
             </div>
         </div>
 
@@ -583,13 +583,13 @@ 
             <br />
             <br />
     		<?php
-    		if (isset($_GET['add'])) {
-    			echo '<button class="btn btn-primary" type="button" value="" onclick="valide()" >'.get_lang('FinishSessionCreation').'</button>';
+            if (isset($_GET['add'])) {
+                echo '<button class="btn btn-primary" type="button" value="" onclick="valide()" >'.get_lang('FinishSessionCreation').'</button>';
             } else {
                 //@todo see that the call to "valide()" doesn't duplicate the onsubmit of the form (necessary to avoid delete on "enter" key pressed)
-    			echo '<button class="save" type="button" value="" onclick="valide()" >'.get_lang('SubscribeUsersToSession').'</button>';
+                echo '<button class="save" type="button" value="" onclick="valide()" >'.get_lang('SubscribeUsersToSession').'</button>';
             }
-    		?>
+            ?>
         </div>
         <div class="span5">
             <div class="multiple_select_header">

main/user/session_list.php

Indentation +4 added lines, -4 removed lines

@@ -65,13 +65,13 @@ 
         array('name'=>'access_end_date',       'index'=>'access_end_date',      'width'=>'40',   'align'=>'left', 'search' => 'true'),
         array('name'=>'coach_name',     'index'=>'coach_name',    'width'=>'80',   'align'=>'left', 'search' => 'false'),
         array('name'=>'status',         'index'=>'session_active','width'=>'40',   'align'=>'left', 'search' => 'true', 'stype'=>'select',
-          //for the bottom bar
+            //for the bottom bar
             'searchoptions' => array(
                 'defaultValue' => '1',
                 'value' => '1:'.get_lang('Active').';0:'.get_lang('Inactive')
             ),
-          //for the top bar
-          'editoptions' => array('value' => ':'.get_lang('All').';1:'.get_lang('Active').';0:'.get_lang('Inactive'))),
+            //for the top bar
+            'editoptions' => array('value' => ':'.get_lang('All').';1:'.get_lang('Active').';0:'.get_lang('Inactive'))),
         array('name'=>'visibility',     'index'=>'visibility',      'width'=>'40',   'align'=>'left', 'search' => 'false'),
         array('name'=>'actions',        'index'=>'actions',         'width'=>'100',  'align'=>'left','formatter'=>'action_formatter','sortable'=>'false', 'search' => 'false')
     );
@@ -86,7 +86,7 @@ 
     //With this function we can add actions to the jgrid (edit, delete, etc)
     $action_links = 'function action_formatter(cellvalue, options, rowObject) {
          return \'&nbsp;<a href="add_users_to_session.php?page=session_list.php&id_session=\'+options.rowId+\'">'.Display::return_icon('user_subscribe_session.png',get_lang('SubscribeUsersToSession'),'',ICON_SIZE_SMALL).'</a>'.
-         '\';
+            '\';
     }';
     ?>
     <script>

whoisonlinesession.php

Indentation +39 added lines, -39 removed lines

@@ -41,10 +41,10 @@ 
 		</th>
 	</tr>
 <?php
-	$session_is_coach = array();
-	if (isset($_user['user_id']) && $_user['user_id'] != '') {
-		$_user['user_id'] = intval($_user['user_id']);
-		$sql = "SELECT DISTINCT session.id,
+    $session_is_coach = array();
+    if (isset($_user['user_id']) && $_user['user_id'] != '') {
+        $_user['user_id'] = intval($_user['user_id']);
+        $sql = "SELECT DISTINCT session.id,
 					name,
 					access_start_date,
 					access_end_date
@@ -53,13 +53,13 @@ 
 					ON srcru.user_id = ".$_user['user_id']." AND srcru.status=2
 					AND session.id = srcru.session_id
 				ORDER BY access_start_date, access_end_date, name";
-		$result = Database::query($sql);
+        $result = Database::query($sql);
 
-		while ($session = Database:: fetch_array($result)) {
-			$session_is_coach[$session['id']] = $session;
-		}
+        while ($session = Database:: fetch_array($result)) {
+            $session_is_coach[$session['id']] = $session;
+        }
 
-		$sql = "SELECT DISTINCT session.id,
+        $sql = "SELECT DISTINCT session.id,
 					name,
 					access_start_date,
 					access_end_date
@@ -67,9 +67,9 @@ 
 				WHERE session.id_coach = ".$_user['user_id']."
 				ORDER BY access_start_date, access_end_date, name";
         $result = Database::query($sql);
-		while ($session = Database:: fetch_array($result)) {
-			$session_is_coach[$session['id']] = $session;
-		}
+        while ($session = Database:: fetch_array($result)) {
+            $session_is_coach[$session['id']] = $session;
+        }
 
         if (empty($time_limit)) {
             $time_limit = api_get_setting('time_limit_whosonline');
@@ -82,7 +82,7 @@ 
 
         $students_online = array();
         foreach ($session_is_coach as $session) {
-			$sql = "SELECT DISTINCT last_access.access_user_id,
+            $sql = "SELECT DISTINCT last_access.access_user_id,
 						last_access.access_date,
 						last_access.c_id,
 						last_access.access_session_id,
@@ -95,49 +95,49 @@ 
 					AND access_date >= '$current_date'
 					GROUP BY access_user_id";
 
-			$result = Database::query($sql);
+            $result = Database::query($sql);
 
-			while($user_list = Database::fetch_array($result)) {
-				$students_online[$user_list['access_user_id']] = $user_list;
-			}
-		}
+            while($user_list = Database::fetch_array($result)) {
+                $students_online[$user_list['access_user_id']] = $user_list;
+            }
+        }
 
-		if (count($students_online) > 0) {
-			foreach ($students_online as $student_online) {
-				echo "<tr>
+        if (count($students_online) > 0) {
+            foreach ($students_online as $student_online) {
+                echo "<tr>
 						<td>
 					";
-				echo $student_online['name'];
-				echo "	</td>
+                echo $student_online['name'];
+                echo "	</td>
 						<td align='center'>
 					 ";
-				$courseInfo = api_get_course_info_by_id($student_online['c_id']);
-				echo $courseInfo['title'];
-				echo "	</td>
+                $courseInfo = api_get_course_info_by_id($student_online['c_id']);
+                echo $courseInfo['title'];
+                echo "	</td>
 						<td align='center'>
 					 ";
-							 if (!empty($student_online['email'])) {
-								echo $student_online['email'];
-							 } else {
-							 	echo get_lang('NoEmail');
-							 }
-				echo "	</td>
+                                if (!empty($student_online['email'])) {
+                                echo $student_online['email'];
+                                } else {
+                                    echo get_lang('NoEmail');
+                                }
+                echo "	</td>
 						<td align='center'>
 					 ";
-				echo '<a href="main/chat/chat.php?cidReq='.$courseInfo['code'].'&id_session='.$student_online['access_session_id'].'"> -> </a>';
-				echo "	</td>
+                echo '<a href="main/chat/chat.php?cidReq='.$courseInfo['code'].'&id_session='.$student_online['access_session_id'].'"> -> </a>';
+                echo "	</td>
 					</tr>
 					 ";
-			}
-		} else {
-			echo '	<tr>
+            }
+        } else {
+            echo '	<tr>
 						<td colspan="4">
 							'.get_lang('NoOnlineStudents').'
 						</td>
 					</tr>
 				 ';
-		}
-	}
+        }
+    }
 ?>
 </table>
 <?php

plugin/pens/lib/pens/pens_request.php

Indentation +447 added lines, -447 removed lines

@@ -43,453 +43,453 @@
 
 abstract class PENSRequest extends PENSMessage {
 
-	/**
-	 * PENS Version to be used. Currently, the only valid value is 1.0.0. Required.
-	 * @var string
-	 */
-	protected $_pens_version = null;
-	
-	/**
-	 * Command being used. The only valid values are collect, alert and receipt.Required
-	 * @var string
-	 */
-	protected $_command = null;
-	
-	/**
-	 * Package type being used. The only valid values are aicc-pkg, scorm-pif, ims-qti. Required
-	 * @var string
-	 */
-	protected $_package_type = null;
-	
-	/**
-	 * Package type version. Required
-	 * @var string
-	 */
-	protected $_package_type_version = null;
-	
-	/**
-	 * Package format. The only valid values are zip, url, jar, war and xml. Required
-	 * @var string
-	 */
-	protected $_package_format = null;
-	
-	/**
-	 * Package id. Requires a valid URI according to RFC 2396. Required
-	 * @var string
-	 */
-	protected $_package_id = null;
-	
-	/**
-	 * Package url. Requires a valid, fully qualified URL including transport protocol and filename extension. Required
-	 * @var string
-	 */
-	protected $_package_url = null;
-	
-	/**
-	 * User id required for system to retrieve package from URL. Optional.
-	 * @var string
-	 */
-	protected $_package_url_user_id = null;
-	
-	/**
-	 * Account required for system to retrieve package from URL. Optional.
-	 * @var string
-	 */
-	protected $_package_url_account = null;
-	
-	/**
-	 * Password required for system to retrieve package from URL. Optional.
-	 * @var string
-	 */
-	protected $_package_url_password = null;
-	
-	/**
-	 * Expiry date for package URL. ISO 8601 format expressed as UTC. Will be transformed into a PHP DateTime object during construction. Required
-	 * @var DateTime
-	 */
-	protected $_package_url_expiry = null;
-	
-	/**
-	 * Name or ID for client submitting the content package to the target system. Required.
-	 * @var string
-	 */
-	protected $_client = null;
-	
-	/**
-	 * User-id or sign-on for target system. Optional
-	 * @var string
-	 */
-	protected $_system_user_id = null;
-	
-	/**
-	 * Either a URL-encoded password token or the null string. If the 
-	 * target system requires a password and the null string value is 
-	 * passed, then the target system is responsible for prompting for a 
-	 * password for target system. Optional
-	 * @var string
-	 */
-	protected $_system_password = null;
-	
-	/**
-	 * URL to send acknowledgement receipt after collecting a package. Any URL, including mailto (as per RFC 2368 and RFC 2822). Required.
-	 * @var string
-	 */
-	protected $_receipt = null;
-	
-	/**
-	 * URL to send alerts to while processing the package. Any URL, including mailto (as per RFC 2368 and RFC 2822). Optional.
-	 * @var string
-	 */
-	protected $_alerts = null;
-	
-	/**
-	 * Unstructured character string that may be used to transfer vendor-specific data such as processing hints or deployment information. Optional.
-	 * @var string
-	 */
-	protected $_vendor_data = null;
-	
-	/**
-	 * Constructor
-	 * 
-	 * Constructs a PENSRequest based class using the arguments given
-	 * 
-	 * @param array Arguments
-	 */
-	public function __construct($arguments) {
-		$this->setPensVersion($arguments["pens-version"]);
-		$this->setPackageType($arguments["package-type"]);
-		$this->setPackageTypeVersion($arguments["package-type-version"]);
-		$this->setPackageFormat($arguments["package-format"]);
-		$this->setPackageId($arguments["package-id"]);
-		$this->setPackageUrl($arguments["package-url"]);
-		$this->setPackageUrlUserId($arguments["package-url-user-id"]);
-		$this->setPackageUrlAccount($arguments["package-url-account"]);
-		$this->setPackageUrlPassword($arguments["package-url-password"]);
-		$this->setPackageUrlExpiry($arguments["package-url-expiry"]);
-		$this->setClient($arguments["client"]);
-		$this->setSystemUserId($arguments["system-user-id"]);
-		$this->setSystemPassword($arguments["system-password"]);
-		$this->setReceipt($arguments["receipt"]);
-		$this->setAlerts($arguments["alerts"]);
-		$this->setVendorData($arguments["vendor-data"]);
-	}
-	
-	public function getPensVersion() {
-		return $this->_pens_version;
-	}
-	
-	/**
-	 * Sets the PENS version
-	 * 
-	 * @param string PENS version
-	 * 
-	 * @throws PENSException with code 2001 if invalid
-	 */
-	public function setPensVersion($pens_version) {
-		if($pens_version == PENSConfig::$version) {
-			$this->_pens_version = $pens_version;
-		} else {
-			throw new PENSException(2001);
-		}
-	}
-	
-	public function getCommand() {
-		return $this->_command;
-	}
-	
-	/**
-	 * Sets the command
-	 * 
-	 * @param string command
-	 * 
-	 * @throws PENSException with code 2002 if invalid
-	 */
-	protected function setCommand($command) {
-		if(in_array($command, PENSConfig::$allowed_commands)) {
-			$this->_command = $command;
-		} else {
-			throw new PENSException(2002);
-		}
-	}
-	
-	public function getPackageType() {
-		return $this->_package_type;
-	}
-	
-	/**
-	 * Sets the package type
-	 * 
-	 * @param string package type
-	 * 
-	 * @throws PENSException with code 2003 if invalid
-	 */
-	public function setPackageType($package_type) {
-		if(in_array($package_type, PENSConfig::$allowed_package_types)) {
-			$this->_package_type = $package_type;
-		} else {
-			throw new PENSException(2003);
-		}
-	}
-	
-	public function getPackageTypeVersion() {
-		return $this->_package_type_version;
-	}
-	
-	/**
-	 * Sets the package type version
-	 * 
-	 * @param string package type version
-	 * 
-	 * @throws PENSException with code 2004 if invalid
-	 */
-	public function setPackageTypeVersion($package_type_version) {
-		if(empty($package_type_version)) {
-			throw new PENSException(2004);
-		} else {
-			$this->_package_type_version = $package_type_version;
-		}
-	}
-	
-	public function getPackageFormat() {
-		return $this->_package_format;
-	}
-	
-	/**
-	 * Sets the package format
-	 * 
-	 * @param string package format
-	 * 
-	 * @throws PENSException with code 2005 if invalid
-	 */
-	public function setPackageFormat($package_format) {
-		if(in_array($package_format, PENSConfig::$allowed_package_formats)) {
-			$this->_package_format = $package_format;
-		} else {
-			throw new PENSException(2005);
-		}
-	}
-	
-	public function getPackageId() {
-		return $this->_package_id;
-	}
-	
-	/**
-	 * Sets the package Id
-	 * 
-	 * @param string package Id
-	 * 
-	 * @throws PENSException with code 2007 if invalid
-	 */
-	public function setPackageId($package_id) {
-		if (preg_match('/'.ABSOLUTEURI_2396.'/', $package_id)) {
-			$this->_package_id = $package_id;
-		} else {
-			throw new PENSException(2007);
-		}
-	}
-	
-	public function getPackageUrl() {
-		return $this->_package_url;
-	}
-	
-	/**
-	 * Sets the package url
-	 * 
-	 * @param string package url
-	 * 
-	 * @throws PENSException with code 2008 if invalid
-	 */
-	public function setPackageUrl($package_url) {
-		if (preg_match('/'.ABSOLUTEURI_2396.'/', $package_url) && substr($package_url, -4) == ".".$this->_package_format) {
-			$this->_package_url = $package_url;
-		} else {
-			throw new PENSException(2008);
-		}
-	}
-	
-	public function getFilename() {
-		return substr(strrchr($this->_package_url, "/"), 1);
-	}
-	
-	public function getPackageUrlUserId() {
-		return $this->_package_url_user_id;
-	}
-	
-	public function setPackageUrlUserId($package_url_user_id) {
-		if(!empty($package_url_user_id)) {
-			$this->_package_url_user_id = $package_url_user_id;
-		}
-	}
-	
-	public function getPackageUrlAccount() {
-		return $this->_package_url_account;
-	}
-	
-	public function setPackageUrlAccount($package_url_account) {
-		if(!empty($package_url_account)) {
-			$this->_package_url_account = $package_url_account;
-		}
-	}
-	
-	public function getPackageUrlPassword() {
-		return $this->_package_url_password;
-	}
-	
-	public function setPackageUrlPassword($package_url_password) {
-		if(!empty($package_url_password)) {
-			$this->_package_url_password = $package_url_password;
-		}
-	}
-	
-	public function getPackageUrlExpiry() {
-		return $this->_package_url_expiry;
-	}
-	
-	/**
-	 * Sets the package url expiry and transforms it into a php DateTime object
-	 * 
-	 * @param string package url expiry
-	 * 
-	 * @throws PENSException with code 2009 if invalid
-	 * @todo Perform a better validation of the date
-	 */
-	public function setPackageUrlExpiry($package_url_expiry) {
-		if(empty($package_url_expiry)) {
-			throw new PENSException(2009);
-		} else {
-			try {
-				$expiry = new DateTime($package_url_expiry, new DateTimeZone('UTC'));
-				$this->_package_url_expiry = $expiry;
-			} catch(Exception $e) {
-				throw new PENSException(2009);
-			}
-		}
-	}
-	
-	public function getClient() {
-		return $this->_client;
-	}
-	
-	/**
-	 * Sets the client
-	 * 
-	 * @param string client
-	 * 
-	 * @throws PENSException with code 2010 if invalid
-	 */
-	public function setClient($client) {
-		if(!empty($client)) {
-			$this->_client = $client;
-		} else {
-			throw new PENSException(2010);
-		}
-	}
-	
-	public function getSystemUserId() {
-		return $this->_system_user_id;
-	}
-	
-	public function setSystemUserId($system_user_id) {
-		if(!empty($system_user_id)) {
-			$this->_system_user_id = $system_user_id;
-		}
-	}
-	
-	public function getSystemPassword() {
-		return $this->_system_password;
-	}
-	
-	public function setSystemPassword($system_password) {
-		if(!empty($system_password)) {
-			$this->_system_password = $system_password;
-		}
-	}
-	
-	public function getReceipt() {
-		return $this->_receipt;
-	}
-	
-	/**
-	 * Sets the receipt url
-	 * 
-	 * @param string receipt url
-	 * 
-	 * @throws PENSException with code 2011 if invalid
-	 */
-	public function setReceipt($receipt) {
-		if($this instanceof PENSRequestCollect) {
-			if (preg_match('/'.ABSOLUTEURI_2396.'/', $receipt)) {
-				$this->_receipt = $receipt;
-			} else {
-				throw new PENSException(2011);
-			}
-		}
-	}
-	
-	public function getAlerts() {
-		return $this->_alerts;
-	}
-	
-	public function setAlerts($alerts) {
-		if(!empty($alerts)) {
-			if(preg_match('/'.ABSOLUTEURI_2396.'/', $alerts)) {
-				$this->_alerts = $alerts;
-			} else {
-				throw new PENSException(1201);
-			}
-		}
-	}
-	
-	public function getVendorData() {
-		return $this->_vendor_data;
-	}
-	
-	public function setVendorData($vendor_data) {
-		if(!empty($vendor_data)) {
-			$this->_vendor_data = $vendor_data;
-		}
-	}
-	
-	/**
-	 * Returns an associative that contains all the fields needed to send a
-	 * receipt or an alert to the client
-	 * 
-	 * @return array Associative array
-	 */
-	protected function getSendReceiptAlertArray() {
-		return array("pens-version" => $this->getPensVersion(),
-			"package-type" => $this->getPackageType(),
-			"package-type-version" => $this->getPackageTypeVersion(),
-			"package-format" => $this->getPackageFormat(),
-			"package-id" => $this->getPackageId(),
-			"package-url" => $this->getPackageUrl(),
-			"package-url-expiry" => $this->getPackageUrlExpiry()->format(DateTime::ISO8601),
-			"client" => $this->getClient());
-	}
-	
-	/**
-	 * Returns an associative that contains all the fields needed to send a
-	 * receipt to the client
-	 * 
-	 * @return array Associative array
-	 */
-	public function getSendReceiptArray() {
-		$receipt = $this->getSendReceiptAlertArray();
-		$receipt["command"] = "receipt";
-		return $receipt;
-	}
-	
-	/**
-	 * Returns an associative that contains all the fields needed to send an
-	 * alert to the client
-	 * 
-	 * @return array Associative array
-	 */
-	public function getSendAlertArray() {
-		$alert = $this->getSendReceiptAlertArray();
-		$alert["command"] = "alert";
-		return $alert;
-	}
+    /**
+     * PENS Version to be used. Currently, the only valid value is 1.0.0. Required.
+     * @var string
+     */
+    protected $_pens_version = null;
+	
+    /**
+     * Command being used. The only valid values are collect, alert and receipt.Required
+     * @var string
+     */
+    protected $_command = null;
+	
+    /**
+     * Package type being used. The only valid values are aicc-pkg, scorm-pif, ims-qti. Required
+     * @var string
+     */
+    protected $_package_type = null;
+	
+    /**
+     * Package type version. Required
+     * @var string
+     */
+    protected $_package_type_version = null;
+	
+    /**
+     * Package format. The only valid values are zip, url, jar, war and xml. Required
+     * @var string
+     */
+    protected $_package_format = null;
+	
+    /**
+     * Package id. Requires a valid URI according to RFC 2396. Required
+     * @var string
+     */
+    protected $_package_id = null;
+	
+    /**
+     * Package url. Requires a valid, fully qualified URL including transport protocol and filename extension. Required
+     * @var string
+     */
+    protected $_package_url = null;
+	
+    /**
+     * User id required for system to retrieve package from URL. Optional.
+     * @var string
+     */
+    protected $_package_url_user_id = null;
+	
+    /**
+     * Account required for system to retrieve package from URL. Optional.
+     * @var string
+     */
+    protected $_package_url_account = null;
+	
+    /**
+     * Password required for system to retrieve package from URL. Optional.
+     * @var string
+     */
+    protected $_package_url_password = null;
+	
+    /**
+     * Expiry date for package URL. ISO 8601 format expressed as UTC. Will be transformed into a PHP DateTime object during construction. Required
+     * @var DateTime
+     */
+    protected $_package_url_expiry = null;
+	
+    /**
+     * Name or ID for client submitting the content package to the target system. Required.
+     * @var string
+     */
+    protected $_client = null;
+	
+    /**
+     * User-id or sign-on for target system. Optional
+     * @var string
+     */
+    protected $_system_user_id = null;
+	
+    /**
+     * Either a URL-encoded password token or the null string. If the 
+     * target system requires a password and the null string value is 
+     * passed, then the target system is responsible for prompting for a 
+     * password for target system. Optional
+     * @var string
+     */
+    protected $_system_password = null;
+	
+    /**
+     * URL to send acknowledgement receipt after collecting a package. Any URL, including mailto (as per RFC 2368 and RFC 2822). Required.
+     * @var string
+     */
+    protected $_receipt = null;
+	
+    /**
+     * URL to send alerts to while processing the package. Any URL, including mailto (as per RFC 2368 and RFC 2822). Optional.
+     * @var string
+     */
+    protected $_alerts = null;
+	
+    /**
+     * Unstructured character string that may be used to transfer vendor-specific data such as processing hints or deployment information. Optional.
+     * @var string
+     */
+    protected $_vendor_data = null;
+	
+    /**
+     * Constructor
+     * 
+     * Constructs a PENSRequest based class using the arguments given
+     * 
+     * @param array Arguments
+     */
+    public function __construct($arguments) {
+        $this->setPensVersion($arguments["pens-version"]);
+        $this->setPackageType($arguments["package-type"]);
+        $this->setPackageTypeVersion($arguments["package-type-version"]);
+        $this->setPackageFormat($arguments["package-format"]);
+        $this->setPackageId($arguments["package-id"]);
+        $this->setPackageUrl($arguments["package-url"]);
+        $this->setPackageUrlUserId($arguments["package-url-user-id"]);
+        $this->setPackageUrlAccount($arguments["package-url-account"]);
+        $this->setPackageUrlPassword($arguments["package-url-password"]);
+        $this->setPackageUrlExpiry($arguments["package-url-expiry"]);
+        $this->setClient($arguments["client"]);
+        $this->setSystemUserId($arguments["system-user-id"]);
+        $this->setSystemPassword($arguments["system-password"]);
+        $this->setReceipt($arguments["receipt"]);
+        $this->setAlerts($arguments["alerts"]);
+        $this->setVendorData($arguments["vendor-data"]);
+    }
+	
+    public function getPensVersion() {
+        return $this->_pens_version;
+    }
+	
+    /**
+     * Sets the PENS version
+     * 
+     * @param string PENS version
+     * 
+     * @throws PENSException with code 2001 if invalid
+     */
+    public function setPensVersion($pens_version) {
+        if($pens_version == PENSConfig::$version) {
+            $this->_pens_version = $pens_version;
+        } else {
+            throw new PENSException(2001);
+        }
+    }
+	
+    public function getCommand() {
+        return $this->_command;
+    }
+	
+    /**
+     * Sets the command
+     * 
+     * @param string command
+     * 
+     * @throws PENSException with code 2002 if invalid
+     */
+    protected function setCommand($command) {
+        if(in_array($command, PENSConfig::$allowed_commands)) {
+            $this->_command = $command;
+        } else {
+            throw new PENSException(2002);
+        }
+    }
+	
+    public function getPackageType() {
+        return $this->_package_type;
+    }
+	
+    /**
+     * Sets the package type
+     * 
+     * @param string package type
+     * 
+     * @throws PENSException with code 2003 if invalid
+     */
+    public function setPackageType($package_type) {
+        if(in_array($package_type, PENSConfig::$allowed_package_types)) {
+            $this->_package_type = $package_type;
+        } else {
+            throw new PENSException(2003);
+        }
+    }
+	
+    public function getPackageTypeVersion() {
+        return $this->_package_type_version;
+    }
+	
+    /**
+     * Sets the package type version
+     * 
+     * @param string package type version
+     * 
+     * @throws PENSException with code 2004 if invalid
+     */
+    public function setPackageTypeVersion($package_type_version) {
+        if(empty($package_type_version)) {
+            throw new PENSException(2004);
+        } else {
+            $this->_package_type_version = $package_type_version;
+        }
+    }
+	
+    public function getPackageFormat() {
+        return $this->_package_format;
+    }
+	
+    /**
+     * Sets the package format
+     * 
+     * @param string package format
+     * 
+     * @throws PENSException with code 2005 if invalid
+     */
+    public function setPackageFormat($package_format) {
+        if(in_array($package_format, PENSConfig::$allowed_package_formats)) {
+            $this->_package_format = $package_format;
+        } else {
+            throw new PENSException(2005);
+        }
+    }
+	
+    public function getPackageId() {
+        return $this->_package_id;
+    }
+	
+    /**
+     * Sets the package Id
+     * 
+     * @param string package Id
+     * 
+     * @throws PENSException with code 2007 if invalid
+     */
+    public function setPackageId($package_id) {
+        if (preg_match('/'.ABSOLUTEURI_2396.'/', $package_id)) {
+            $this->_package_id = $package_id;
+        } else {
+            throw new PENSException(2007);
+        }
+    }
+	
+    public function getPackageUrl() {
+        return $this->_package_url;
+    }
+	
+    /**
+     * Sets the package url
+     * 
+     * @param string package url
+     * 
+     * @throws PENSException with code 2008 if invalid
+     */
+    public function setPackageUrl($package_url) {
+        if (preg_match('/'.ABSOLUTEURI_2396.'/', $package_url) && substr($package_url, -4) == ".".$this->_package_format) {
+            $this->_package_url = $package_url;
+        } else {
+            throw new PENSException(2008);
+        }
+    }
+	
+    public function getFilename() {
+        return substr(strrchr($this->_package_url, "/"), 1);
+    }
+	
+    public function getPackageUrlUserId() {
+        return $this->_package_url_user_id;
+    }
+	
+    public function setPackageUrlUserId($package_url_user_id) {
+        if(!empty($package_url_user_id)) {
+            $this->_package_url_user_id = $package_url_user_id;
+        }
+    }
+	
+    public function getPackageUrlAccount() {
+        return $this->_package_url_account;
+    }
+	
+    public function setPackageUrlAccount($package_url_account) {
+        if(!empty($package_url_account)) {
+            $this->_package_url_account = $package_url_account;
+        }
+    }
+	
+    public function getPackageUrlPassword() {
+        return $this->_package_url_password;
+    }
+	
+    public function setPackageUrlPassword($package_url_password) {
+        if(!empty($package_url_password)) {
+            $this->_package_url_password = $package_url_password;
+        }
+    }
+	
+    public function getPackageUrlExpiry() {
+        return $this->_package_url_expiry;
+    }
+	
+    /**
+     * Sets the package url expiry and transforms it into a php DateTime object
+     * 
+     * @param string package url expiry
+     * 
+     * @throws PENSException with code 2009 if invalid
+     * @todo Perform a better validation of the date
+     */
+    public function setPackageUrlExpiry($package_url_expiry) {
+        if(empty($package_url_expiry)) {
+            throw new PENSException(2009);
+        } else {
+            try {
+                $expiry = new DateTime($package_url_expiry, new DateTimeZone('UTC'));
+                $this->_package_url_expiry = $expiry;
+            } catch(Exception $e) {
+                throw new PENSException(2009);
+            }
+        }
+    }
+	
+    public function getClient() {
+        return $this->_client;
+    }
+	
+    /**
+     * Sets the client
+     * 
+     * @param string client
+     * 
+     * @throws PENSException with code 2010 if invalid
+     */
+    public function setClient($client) {
+        if(!empty($client)) {
+            $this->_client = $client;
+        } else {
+            throw new PENSException(2010);
+        }
+    }
+	
+    public function getSystemUserId() {
+        return $this->_system_user_id;
+    }
+	
+    public function setSystemUserId($system_user_id) {
+        if(!empty($system_user_id)) {
+            $this->_system_user_id = $system_user_id;
+        }
+    }
+	
+    public function getSystemPassword() {
+        return $this->_system_password;
+    }
+	
+    public function setSystemPassword($system_password) {
+        if(!empty($system_password)) {
+            $this->_system_password = $system_password;
+        }
+    }
+	
+    public function getReceipt() {
+        return $this->_receipt;
+    }
+	
+    /**
+     * Sets the receipt url
+     * 
+     * @param string receipt url
+     * 
+     * @throws PENSException with code 2011 if invalid
+     */
+    public function setReceipt($receipt) {
+        if($this instanceof PENSRequestCollect) {
+            if (preg_match('/'.ABSOLUTEURI_2396.'/', $receipt)) {
+                $this->_receipt = $receipt;
+            } else {
+                throw new PENSException(2011);
+            }
+        }
+    }
+	
+    public function getAlerts() {
+        return $this->_alerts;
+    }
+	
+    public function setAlerts($alerts) {
+        if(!empty($alerts)) {
+            if(preg_match('/'.ABSOLUTEURI_2396.'/', $alerts)) {
+                $this->_alerts = $alerts;
+            } else {
+                throw new PENSException(1201);
+            }
+        }
+    }
+	
+    public function getVendorData() {
+        return $this->_vendor_data;
+    }
+	
+    public function setVendorData($vendor_data) {
+        if(!empty($vendor_data)) {
+            $this->_vendor_data = $vendor_data;
+        }
+    }
+	
+    /**
+     * Returns an associative that contains all the fields needed to send a
+     * receipt or an alert to the client
+     * 
+     * @return array Associative array
+     */
+    protected function getSendReceiptAlertArray() {
+        return array("pens-version" => $this->getPensVersion(),
+            "package-type" => $this->getPackageType(),
+            "package-type-version" => $this->getPackageTypeVersion(),
+            "package-format" => $this->getPackageFormat(),
+            "package-id" => $this->getPackageId(),
+            "package-url" => $this->getPackageUrl(),
+            "package-url-expiry" => $this->getPackageUrlExpiry()->format(DateTime::ISO8601),
+            "client" => $this->getClient());
+    }
+	
+    /**
+     * Returns an associative that contains all the fields needed to send a
+     * receipt to the client
+     * 
+     * @return array Associative array
+     */
+    public function getSendReceiptArray() {
+        $receipt = $this->getSendReceiptAlertArray();
+        $receipt["command"] = "receipt";
+        return $receipt;
+    }
+	
+    /**
+     * Returns an associative that contains all the fields needed to send an
+     * alert to the client
+     * 
+     * @return array Associative array
+     */
+    public function getSendAlertArray() {
+        $alert = $this->getSendReceiptAlertArray();
+        $alert["command"] = "alert";
+        return $alert;
+    }
 	
 	
 }

plugin/pens/lib/pens/pens_config.php

Indentation +43 added lines, -43 removed lines

@@ -16,54 +16,54 @@
  * along with php-pens.  If not, see <http://www.gnu.org/licenses/>.
  */
  
- /**
- * PENSConfig
- * 
- * Provides the PENSConfig class
- * 
- * @package PENS
- * @author Guillaume Viguier-Just <guillaume@viguierjust.com>
- * @licence http://www.gnu.org/licenses/gpl.txt
- */
+    /**
+     * PENSConfig
+     * 
+     * Provides the PENSConfig class
+     * 
+     * @package PENS
+     * @author Guillaume Viguier-Just <guillaume@viguierjust.com>
+     * @licence http://www.gnu.org/licenses/gpl.txt
+     */
  
- /**
- * PENSConfig
- * 
- * This class provides different static variables to configure php-pens.
- * 
- * @package PENS
- * @author Guillaume Viguier-Just <guillaume@viguierjust.com>
- * @licence http://www.gnu.org/licenses/gpl.txt
- */
+    /**
+     * PENSConfig
+     * 
+     * This class provides different static variables to configure php-pens.
+     * 
+     * @package PENS
+     * @author Guillaume Viguier-Just <guillaume@viguierjust.com>
+     * @licence http://www.gnu.org/licenses/gpl.txt
+     */
 class PENSConfig {
 
-	/**
-	 * PENS version
-	 * @var string
-	 */
-	public static $version = "1.0.0";
+    /**
+     * PENS version
+     * @var string
+     */
+    public static $version = "1.0.0";
 	
-	/**
-	 * Allowed commands
-	 * @var array
-	 */
-	public static $allowed_commands = array("collect", "receipt", "alert");
+    /**
+     * Allowed commands
+     * @var array
+     */
+    public static $allowed_commands = array("collect", "receipt", "alert");
 	
-	/**
-	 * Allowed package types
-	 * @var array
-	 */
-	public static $allowed_package_types = array("aicc-pkg", "scorm-pif", "ims-qti");
+    /**
+     * Allowed package types
+     * @var array
+     */
+    public static $allowed_package_types = array("aicc-pkg", "scorm-pif", "ims-qti");
 	
-	/**
-	 * Allowed package formats
-	 * @var array
-	 */
-	public static $allowed_package_formats = array("zip", "url", "jar", "war", "xml");
+    /**
+     * Allowed package formats
+     * @var array
+     */
+    public static $allowed_package_formats = array("zip", "url", "jar", "war", "xml");
 	
-	/**
-	 * End of line as specified by the PENS specification
-	 * @var string
-	 */
-	public static $eol = "\r\n";
+    /**
+     * End of line as specified by the PENS specification
+     * @var string
+     */
+    public static $eol = "\r\n";
 }

plugin/pens/lib/pens/pens_exception.php

Indentation +55 added lines, -55 removed lines

@@ -37,62 +37,62 @@
  */
 class PENSException extends Exception {
 
-	/**
-	 * Array that provides an association between exception codes and messages
-	 * @var array
-	 */
-	protected static $_code_to_messages = array(
-		1101 => "Unable to parse PENS command",
-		1201 => "Attempt to pass an invalid argument",
-		1301 => "Unable to retrieve package",
-		1302 => "Unable to retrieve package via HTTPS",
-		1304 => "Unable to retrieve package via FTP",
-		1306 => "Unable to retrieve package via FTPS",
-		1310 => "Unable to retrieve package at specified URL due to error in URL or lack of response from URL",
-		1312 => "Unable to retrieve package at specified URL due to error with access credential for package URL",
-		1320 => "Expiration date is non-null and in an improper format",
-		1322 => "Current time indicates expiry date has passed",
-		1420 => "PENS version not supported",
-		1421 => "Command not supported",
-		1430 => "Package type not supported",
-		// The following error code is not specified in the PENS specification and was added in this implementation
-		1431 => "Package format not supported",
-		1432 => "Internal package error",
-		1440 => "Insufficient host space/storage available",
-		1500 => "Unable to communicate with provided acknowledgement URL",
-		1510 => "Unsupported acknowledgement protocol",
-		1520 => "Unsupported alert protocol",
-		2001 => "PENS version invalid or not specified",
-		2002 => "PENS command invalid or not specified",
-		2003 => "package-type invalid or not specified",
-		2004 => "package-type-version invalid or not specified",
-		2005 => "package-format invalid or not specified",
-		2007 => "package-id invalid or not specified",
-		2008 => "package-url invalid or not specified",
-		2009 => "package-url-expiry date invalid or not specified",
-		2010 => "client submitting package invalid or not specified",
-		2011 => "receipt url invalid or not specified"
-	);
+    /**
+     * Array that provides an association between exception codes and messages
+     * @var array
+     */
+    protected static $_code_to_messages = array(
+        1101 => "Unable to parse PENS command",
+        1201 => "Attempt to pass an invalid argument",
+        1301 => "Unable to retrieve package",
+        1302 => "Unable to retrieve package via HTTPS",
+        1304 => "Unable to retrieve package via FTP",
+        1306 => "Unable to retrieve package via FTPS",
+        1310 => "Unable to retrieve package at specified URL due to error in URL or lack of response from URL",
+        1312 => "Unable to retrieve package at specified URL due to error with access credential for package URL",
+        1320 => "Expiration date is non-null and in an improper format",
+        1322 => "Current time indicates expiry date has passed",
+        1420 => "PENS version not supported",
+        1421 => "Command not supported",
+        1430 => "Package type not supported",
+        // The following error code is not specified in the PENS specification and was added in this implementation
+        1431 => "Package format not supported",
+        1432 => "Internal package error",
+        1440 => "Insufficient host space/storage available",
+        1500 => "Unable to communicate with provided acknowledgement URL",
+        1510 => "Unsupported acknowledgement protocol",
+        1520 => "Unsupported alert protocol",
+        2001 => "PENS version invalid or not specified",
+        2002 => "PENS command invalid or not specified",
+        2003 => "package-type invalid or not specified",
+        2004 => "package-type-version invalid or not specified",
+        2005 => "package-format invalid or not specified",
+        2007 => "package-id invalid or not specified",
+        2008 => "package-url invalid or not specified",
+        2009 => "package-url-expiry date invalid or not specified",
+        2010 => "client submitting package invalid or not specified",
+        2011 => "receipt url invalid or not specified"
+    );
 	
-	/**
-	 * Redefines the constructor so that code is the first argument
-	 * 
-	 * @param int Exception code
-	 * @param string Message to display
-	 * @return PENSException Exception created
-	 */
-	public function __construct($code, $message = null) {
-		parent::__construct($message, $code);
-		$this->setMessageBasedOnCode();
-	}
+    /**
+     * Redefines the constructor so that code is the first argument
+     * 
+     * @param int Exception code
+     * @param string Message to display
+     * @return PENSException Exception created
+     */
+    public function __construct($code, $message = null) {
+        parent::__construct($message, $code);
+        $this->setMessageBasedOnCode();
+    }
 	
-	/**
-	 * Sets the message based on the code
-	 */
-	protected function setMessageBasedOnCode() {
-		if(empty($this->message) && !empty(self::$_code_to_messages[$this->code])) {
-			$this->message = self::$_code_to_messages[$this->code];
-		}
-	}
+    /**
+     * Sets the message based on the code
+     */
+    protected function setMessageBasedOnCode() {
+        if(empty($this->message) && !empty(self::$_code_to_messages[$this->code])) {
+            $this->message = self::$_code_to_messages[$this->code];
+        }
+    }
 	
 }

plugin/pens/pens.php

Indentation +13 added lines, -13 removed lines

@@ -29,19 +29,19 @@
 require_once __DIR__ . '/chamilo_pens.php';
 
 class ChamiloPackageHandler extends PENSPackageHandler {
-	public function processPackage($request, $path_to_package) {
-		$server = PENSServer::singleton();
-		// Moves the package to archive/pens
-		$path_to_archives = api_get_path(SYS_ARCHIVE_PATH) . 'pens';
-		if(!is_dir($path_to_archives)) {
-			mkdir($path_to_archives, 0777, true);
-		}
-		rename($path_to_package, $path_to_archives . '/' . $request->getFilename());
-		// Insert the request in the database
-		$chamilo_pens = new ChamiloPens($request);
-		$chamilo_pens->save();
-		$server->sendAlert($request, new PENSResponse(0, 'Package successfully processed'));
-	}
+    public function processPackage($request, $path_to_package) {
+        $server = PENSServer::singleton();
+        // Moves the package to archive/pens
+        $path_to_archives = api_get_path(SYS_ARCHIVE_PATH) . 'pens';
+        if(!is_dir($path_to_archives)) {
+            mkdir($path_to_archives, 0777, true);
+        }
+        rename($path_to_package, $path_to_archives . '/' . $request->getFilename());
+        // Insert the request in the database
+        $chamilo_pens = new ChamiloPens($request);
+        $chamilo_pens->save();
+        $server->sendAlert($request, new PENSResponse(0, 'Package successfully processed'));
+    }
 }
 
 $handler = new ChamiloPackageHandler();

plugin/dashboard/block_student/block_student.class.php

Indentation +141 added lines, -141 removed lines

@@ -14,36 +14,36 @@ 
 class BlockStudent extends Block
 {
     private $user_id;
-	private $students;
-	private $path;
-	private $permission = array(DRH);
+    private $students;
+    private $path;
+    private $permission = array(DRH);
 
-	/**
-	 * Constructor
-	 */
+    /**
+     * Constructor
+     */
     public function __construct ($user_id)
     {
-    	$this->user_id  = $user_id;
-    	$this->path 	= 'block_student';
-    	if ($this->is_block_visible_for_user($user_id)) {
+        $this->user_id  = $user_id;
+        $this->path 	= 'block_student';
+        if ($this->is_block_visible_for_user($user_id)) {
             $this->students =  UserManager::get_users_followed_by_drh($user_id, STUDENT);
-    	}
+        }
     }
 
     /**
-	 * This method check if a user is allowed to see the block inside dashboard interface
-	 * @param	int		User id
-	 * @return	bool	Is block visible for user
-	 */
+     * This method check if a user is allowed to see the block inside dashboard interface
+     * @param	int		User id
+     * @return	bool	Is block visible for user
+     */
     public function is_block_visible_for_user($user_id)
     {
-    	$user_info = api_get_user_info($user_id);
-		$user_status = $user_info['status'];
-		$is_block_visible_for_user = false;
-    	if (UserManager::is_admin($user_id) || in_array($user_status, $this->permission)) {
-    		$is_block_visible_for_user = true;
-    	}
-    	return $is_block_visible_for_user;
+        $user_info = api_get_user_info($user_id);
+        $user_status = $user_info['status'];
+        $is_block_visible_for_user = false;
+        if (UserManager::is_admin($user_id) || in_array($user_status, $this->permission)) {
+            $is_block_visible_for_user = true;
+        }
+        return $is_block_visible_for_user;
     }
 
     /**
@@ -53,12 +53,12 @@ 
      */
     public function get_block()
     {
-    	global $charset;
-    	$column = 1;
-    	$data   = array();
+        global $charset;
+        $column = 1;
+        $data   = array();
 
         $student_content_html = $this->get_students_content_html_for_drh();
-		$html = '<div class="panel panel-default" id="intro">
+        $html = '<div class="panel panel-default" id="intro">
                     <div class="panel-heading">
                         '.get_lang('StudentsInformationsList').'
                         <div class="pull-right"><a class="btn btn-danger btn-xs" onclick="javascript:if(!confirm(\''.addslashes(api_htmlentities(get_lang('ConfirmYourChoice'),ENT_QUOTES,$charset)).'\')) return false;" href="index.php?action=disable_block&path='.$this->path.'">
@@ -70,24 +70,24 @@ 
                         '.$student_content_html.'
                     </div>
                     </div>';
-    	$data['column'] = $column;
-    	$data['content_html'] = $html;
-    	return $data;
+        $data['column'] = $column;
+        $data['content_html'] = $html;
+        return $data;
     }
 
     /**
- 	 * This method return a content html, it's used inside get_block method for showing it inside dashboard interface
- 	 * @return string  content html
- 	 */
+     * This method return a content html, it's used inside get_block method for showing it inside dashboard interface
+     * @return string  content html
+     */
     public function get_students_content_html_for_platform_admin()
     {
- 		$students = $this->students;
- 		//$content = '<div style="margin:10px;">';
- 		$content = '<h4>'.get_lang('YourStudents').'</h4>';
+            $students = $this->students;
+            //$content = '<div style="margin:10px;">';
+            $content = '<h4>'.get_lang('YourStudents').'</h4>';
         $students_table = null;
- 		if (count($students) > 0) {
-	 		$students_table .= '<table class="data_table">';
-	 		$students_table .= '<tr>
+            if (count($students) > 0) {
+                $students_table .= '<table class="data_table">';
+                $students_table .= '<tr>
 									<th width="10%" rowspan="2">'.get_lang('FirstName').'</th>
 									<th width="10%" rowspan="2">'.get_lang('LastName').'</th>
 									<th width="30%" colspan="2">'.get_lang('CourseInformation').'</th>
@@ -97,146 +97,146 @@ 
 									<th width="10%">'.get_lang('Time').'</th>
 								</tr>';
 
-	 		$i = 1;
-	 		foreach ($students as $student) {
+                $i = 1;
+                foreach ($students as $student) {
 
-	 			$courses_by_user = CourseManager::get_courses_list_by_user_id($student['user_id'], true);
-	 			$count_courses = count($courses_by_user);
-				$rowspan = $count_courses?$count_courses+1:2;
+                    $courses_by_user = CourseManager::get_courses_list_by_user_id($student['user_id'], true);
+                    $count_courses = count($courses_by_user);
+                $rowspan = $count_courses?$count_courses+1:2;
 
-				if ($i%2 == 0) {
+                if ($i%2 == 0) {
                     $style = ' style="background-color:#F2F2F2" ';
                 } else {
                     $style = ' style="background-color:#FFF" ';
                 }
 
-	 			$students_table .= '<tr '.$style.'>
+                    $students_table .= '<tr '.$style.'>
 										<td rowspan="'.$rowspan.'">'.$student['firstname'].'</td>
 										<td rowspan="'.$rowspan.'">'.$student['lastname'].'</td>
 									</tr>';
 
-	 			// courses information about the student
-	 			if (!empty($courses_by_user)) {
-		 			foreach ($courses_by_user as $course) {
-		 				$course_code = $course['code'];
-						$courseInfo = api_get_course_info($course_code);
-						$courseId = $courseInfo['real_id'];
-		 				$course_title = $course['title'];
-		 				$time = api_time_to_hms(Tracking :: get_time_spent_on_the_course($student['user_id'], $courseId));
-		 				$students_table .= '<tr '.$style.'>
+                    // courses information about the student
+                    if (!empty($courses_by_user)) {
+                        foreach ($courses_by_user as $course) {
+                            $course_code = $course['code'];
+                        $courseInfo = api_get_course_info($course_code);
+                        $courseId = $courseInfo['real_id'];
+                            $course_title = $course['title'];
+                            $time = api_time_to_hms(Tracking :: get_time_spent_on_the_course($student['user_id'], $courseId));
+                            $students_table .= '<tr '.$style.'>
 											<td align="right">'.$course_title.'</td>
 											<td align="right">'.$time.'</td>
 											</tr>';
-		 			}
-	 			} else {
-	 				$students_table .= '<tr '.$style.'>
+                        }
+                    } else {
+                        $students_table .= '<tr '.$style.'>
 											<td align="center" colspan="2"><i>'.get_lang('Empty').'</i></td>
 										</tr>';
-	 			}
-	 			$i++;
-	 		}
-	 		$students_table .= '</table>';
- 		} else {
- 			$students_table .= get_lang('ThereIsNoInformationAboutYourStudents');
- 		}
-
- 		$content .= $students_table;
-
- 		if (count($students) > 0) {
-			$content .= '<div style="text-align:right;margin-top:10px;"><a href="'.api_get_path(WEB_CODE_PATH).'mySpace/index.php?view=admin&display=useroverview">'.get_lang('SeeMore').'</a></div>';
-		}
-		//$content .= '</div>';
-
- 		return $content;
- 	}
-
-  	public function get_students_content_html_for_drh() {
-  		$attendance = new Attendance();
-  		$students = $this->students;
- 		//$content = '<div style="margin:5px;">';
- 		$content = '<h4>'.get_lang('YourStudents').'</h4>';
+                    }
+                    $i++;
+                }
+                $students_table .= '</table>';
+            } else {
+                $students_table .= get_lang('ThereIsNoInformationAboutYourStudents');
+            }
+
+            $content .= $students_table;
+
+            if (count($students) > 0) {
+            $content .= '<div style="text-align:right;margin-top:10px;"><a href="'.api_get_path(WEB_CODE_PATH).'mySpace/index.php?view=admin&display=useroverview">'.get_lang('SeeMore').'</a></div>';
+        }
+        //$content .= '</div>';
+
+            return $content;
+        }
+
+        public function get_students_content_html_for_drh() {
+            $attendance = new Attendance();
+            $students = $this->students;
+            //$content = '<div style="margin:5px;">';
+            $content = '<h4>'.get_lang('YourStudents').'</h4>';
         $students_table = null;
- 		if (count($students) > 0) {
-	 		$students_table .= '<table class="data_table">';
-	 		$students_table .= '<tr>
+            if (count($students) > 0) {
+                $students_table .= '<table class="data_table">';
+                $students_table .= '<tr>
 									<th>'.get_lang('User').'</th>
 									<th>'.get_lang('AttendancesFaults').'</th>
 									<th>'.get_lang('Evaluations').'</th>
 								</tr>';
-	 		$i = 1;
-	 		foreach ($students as $student) {
-
-	 			$student_id = $student['user_id'];
-	 			$firstname  = $student['firstname'];
-	 			$lastname   = $student['lastname'];
-	 			$username	= $student['username'];
-
-
-				// get average of faults in attendances by student
-	 			$results_faults_avg = $attendance->get_faults_average_inside_courses($student_id);
-	 			if (!empty($results_faults_avg)) {
-	 				$attendances_faults_avg = '<a title="'.get_lang('GoToStudentDetails').'" href="'.api_get_path(WEB_CODE_PATH).'mySpace/myStudents.php?student='.$student_id.'">'.$results_faults_avg['faults'].'/'.$results_faults_avg['total'].' ('.$results_faults_avg['porcent'].'%)</a>';
-	 			} else {
-	 				$attendances_faults_avg = '0%';
-	 			}
-
-	 			$courses_by_user = CourseManager::get_courses_list_by_user_id($student_id, true);
-				$evaluations_avg  = 0;
-				$score = $weight = 0;
-	 			foreach ($courses_by_user as $course) {
-	 				$course_code = $course['code'];
-	 				$cats = Category::load(null, null, $course_code, null, null, null, false);
-	 				$scoretotal = array();
-	 				if (isset($cats) && isset($cats[0])) {
-		 				$scoretotal= $cats[0]->calc_score($student_id, null, $course_code);
-	 				}
-
-	 				if (!empty($scoretotal)) {
-	 					$score  += $scoretotal[0];
-	 					$weight += $scoretotal[1];
-	 				}
-	 			}
-
-	 			if (!empty($weight)) {
-	 				$evaluations_avg = '<a title="'.get_lang('GoToStudentDetails').'" href="'.api_get_path(WEB_CODE_PATH).'mySpace/myStudents.php?student='.$student_id.'">'.round($score,2).'/'.round($weight,2).'('.round(($score / $weight) * 100,2) . ' %)</a>';
-	 			}
-
-	 			if ($i%2 == 0) {
+                $i = 1;
+                foreach ($students as $student) {
+
+                    $student_id = $student['user_id'];
+                    $firstname  = $student['firstname'];
+                    $lastname   = $student['lastname'];
+                    $username	= $student['username'];
+
+
+                // get average of faults in attendances by student
+                    $results_faults_avg = $attendance->get_faults_average_inside_courses($student_id);
+                    if (!empty($results_faults_avg)) {
+                        $attendances_faults_avg = '<a title="'.get_lang('GoToStudentDetails').'" href="'.api_get_path(WEB_CODE_PATH).'mySpace/myStudents.php?student='.$student_id.'">'.$results_faults_avg['faults'].'/'.$results_faults_avg['total'].' ('.$results_faults_avg['porcent'].'%)</a>';
+                    } else {
+                        $attendances_faults_avg = '0%';
+                    }
+
+                    $courses_by_user = CourseManager::get_courses_list_by_user_id($student_id, true);
+                $evaluations_avg  = 0;
+                $score = $weight = 0;
+                    foreach ($courses_by_user as $course) {
+                        $course_code = $course['code'];
+                        $cats = Category::load(null, null, $course_code, null, null, null, false);
+                        $scoretotal = array();
+                        if (isset($cats) && isset($cats[0])) {
+                            $scoretotal= $cats[0]->calc_score($student_id, null, $course_code);
+                        }
+
+                        if (!empty($scoretotal)) {
+                            $score  += $scoretotal[0];
+                            $weight += $scoretotal[1];
+                        }
+                    }
+
+                    if (!empty($weight)) {
+                        $evaluations_avg = '<a title="'.get_lang('GoToStudentDetails').'" href="'.api_get_path(WEB_CODE_PATH).'mySpace/myStudents.php?student='.$student_id.'">'.round($score,2).'/'.round($weight,2).'('.round(($score / $weight) * 100,2) . ' %)</a>';
+                    }
+
+                    if ($i%2 == 0) {
                     $class_tr = 'row_odd';
                 } else {
                     $class_tr = 'row_even';
                 }
-	    		$students_table .= '<tr class="'.$class_tr.'">
+                $students_table .= '<tr class="'.$class_tr.'">
 										<td>'.api_get_person_name($firstname, $lastname).' ('.$username.')</td>
 										<td>'.$attendances_faults_avg.'</td>
 										<td>'.$evaluations_avg.'</td>
 									</tr>';
 
-	 			$i++;
-	 		}
-	 		$students_table .= '</table>';
- 		} else {
- 			$students_table .= get_lang('ThereIsNoInformationAboutYourStudents');
- 		}
+                    $i++;
+                }
+                $students_table .= '</table>';
+            } else {
+                $students_table .= get_lang('ThereIsNoInformationAboutYourStudents');
+            }
 
-  		$content .= $students_table;
+            $content .= $students_table;
 
- 		if (count($students) > 0) {
-			$content .= '<div style="text-align:right;margin-top:10px;">
+            if (count($students) > 0) {
+            $content .= '<div style="text-align:right;margin-top:10px;">
                             <a href="'.api_get_path(WEB_CODE_PATH).'mySpace/index.php?view=admin&display=yourstudents">'.get_lang('SeeMore').'</a>
                          </div>';
-		}
-		//$content .= '</div>';
+        }
+        //$content .= '</div>';
 
-  		return $content;
-  	}
+            return $content;
+        }
 
     /**
-	 * Get number of students
-	 * @return int
-	 */
-	function get_number_of_students()
+     * Get number of students
+     * @return int
+     */
+    function get_number_of_students()
     {
-		return count($this->students);
-	}
+        return count($this->students);
+    }
 }

src/Chamilo/CoreBundle/Entity/AccessUrlRelUserGroup.php

Indentation -1 removed lines

@@ -16,7 +16,6 @@
  * @author Patrick Cool
  * @author René Haentjens, added CSV file import (October 2004)
  * @package chamilo.link
-
  */
 
 // Including libraries