Minor: Format code

pull/4422/head
Angel Fernando Quiroz Campos 3 years ago
parent 36e0313759
commit d03bd55ea4
  1. 5
      main/inc/lib/myspace.lib.php
  2. 2
      main/inc/lib/tracking.lib.php
  3. 6
      main/inc/lib/webservices/Rest.php
  4. 2
      main/wiki/wiki.inc.php
  5. 10
      plugin/userremoteservice/Entity/UserRemoteService.php
  6. 2
      plugin/userremoteservice/lang/english.php
  7. 1
      plugin/userremoteservice/redirect.php
  8. 4
      plugin/userremoteservice/src/user_remote_service_plugin.class.php

@ -3250,11 +3250,6 @@ class MySpace
* Get all the data for the sortable table of the reporting progress of * Get all the data for the sortable table of the reporting progress of
* all users and all the courses the user is subscribed to. * all users and all the courses the user is subscribed to.
* *
* @param int $from
* @param int $numberItems
* @param int $column
* @param string $direction
*
* @return array * @return array
*/ */
public static function get_user_data_tracking_overview(int $from, int $numberItems, int $column, string $direction) public static function get_user_data_tracking_overview(int $from, int $numberItems, int $column, string $direction)

@ -2247,7 +2247,7 @@ class Tracking
); );
} else { } else {
if (api_is_drh() && $allowDhrAccessToAllStudents) { if (api_is_drh() && $allowDhrAccessToAllStudents) {
$studentList = UserManager::get_user_list( $studentList = UserManager::get_user_list(
['status' => STUDENT], ['status' => STUDENT],
[], [],
false, false,

@ -1131,7 +1131,7 @@ class Rest extends WebService
} }
/** /**
* Get one's own profile * Get one's own profile.
*/ */
public function getUserProfile(): array public function getUserProfile(): array
{ {
@ -1164,7 +1164,7 @@ class Rest extends WebService
} }
/** /**
* Get one's own (avg) progress in learning paths * Get one's own (avg) progress in learning paths.
*/ */
public function getCourseLpProgress(): array public function getCourseLpProgress(): array
{ {
@ -1295,7 +1295,7 @@ class Rest extends WebService
} }
/** /**
* Start login for a user. Then make a redirect to show the learnpath * Start login for a user. Then make a redirect to show the learnpath.
*/ */
public function showLearningPath(int $lpId) public function showLearningPath(int $lpId)
{ {

@ -1097,7 +1097,7 @@ class Wiki
} }
} else { } else {
if (true === api_get_configuration_value('wiki_html_strict_filtering')) { if (true === api_get_configuration_value('wiki_html_strict_filtering')) {
$content = Security::remove_XSS($row['content'],COURSEMANAGERLOWSECURITY); $content = Security::remove_XSS($row['content'], COURSEMANAGERLOWSECURITY);
} else { } else {
$content = Security::remove_XSS($row['content']); $content = Security::remove_XSS($row['content']);
} }

@ -96,16 +96,17 @@ class UserRemoteService
return $this; return $this;
} }
/** /**
* @return string * @return string
*/ */
public function getAccessURL($pluginName) public function getAccessURL($pluginName)
{ {
$accessUrl = api_get_path(WEB_PLUGIN_PATH) . $pluginName . "/redirect.php?serviceId=" . $this->getId(); $accessUrl = api_get_path(WEB_PLUGIN_PATH).$pluginName."/redirect.php?serviceId=".$this->getId();
return $accessUrl; return $accessUrl;
} }
/** /**
* Returns a user-specific URL, with two extra query string parameters : 'username' and 'hash'. * Returns a user-specific URL, with two extra query string parameters : 'username' and 'hash'.
* 'hash' is generated using $salt and $userId. * 'hash' is generated using $salt and $userId.
@ -138,13 +139,12 @@ class UserRemoteService
); );
} }
/** /**
* Returns a user-specific URL, with two extra query string parameters : 'uid' and 'hash'. * Returns a user-specific URL, with two extra query string parameters : 'uid' and 'hash'.
* 'hash' is generated using $salt and $userId. * 'hash' is generated using $salt and $userId.
* *
* @param string $userId the user identifier, to build the hash and to include for the uid parameter * @param string $userId the user identifier, to build the hash and to include for the uid parameter
* @param string $salt the salt, to build the hash * @param string $salt the salt, to build the hash
* *
* @throws Exception on hash generation failure * @throws Exception on hash generation failure
* *

@ -3,7 +3,7 @@
$strings['plugin_title'] = 'User Remote Services'; $strings['plugin_title'] = 'User Remote Services';
$strings['plugin_comment'] = 'Appends site-specific iframe-targetted user-identifying links to the menu bar.'; $strings['plugin_comment'] = 'Appends site-specific iframe-targetted user-identifying links to the menu bar.';
$strings['salt'] = 'Salt'; $strings['salt'] = 'Salt';
$strings['salt_help'] = $strings['salt_help'] =
'Secret character string, used to generate the <em>hash</em> URL parameter. The longest, the best. 'Secret character string, used to generate the <em>hash</em> URL parameter. The longest, the best.

@ -11,4 +11,3 @@ $plugin = UserRemoteServicePlugin::create();
header('Location: '.$plugin->getActiveServiceSpecificUserUrl()); header('Location: '.$plugin->getActiveServiceSpecificUserUrl());
exit; exit;

@ -70,7 +70,7 @@ class UserRemoteServicePlugin extends Plugin
} }
/** /**
* @return boolean the value of hide_link_from_navigation_menu setting * @return bool the value of hide_link_from_navigation_menu setting
*/ */
public function get_hide_link_from_navigation_menu() public function get_hide_link_from_navigation_menu()
{ {
@ -296,7 +296,7 @@ class UserRemoteServicePlugin extends Plugin
* *
* @throws Exception on hash generation failure * @throws Exception on hash generation failure
* *
* @return string the specific user redirect URL * @return string the specific user redirect URL
*/ */
public function getActiveServiceSpecificUserUrl() public function getActiveServiceSpecificUserUrl()
{ {

Loading…
Cancel
Save