Minor: Code and documentation styling - refs BT#19984

pull/4346/head
Yannick Warnier 3 years ago
parent e5b3637526
commit f2544d9499
  1. 20
      main/dropbox/dropbox_class.inc.php
  2. 3
      main/dropbox/dropbox_functions.inc.php
  3. 8
      main/exercise/oral_expression.class.php
  4. 5
      main/inc/lib/sessionmanager.lib.php

@ -436,12 +436,6 @@ class Dropbox_Person
/** /**
* Constructor for recreating the Dropbox_Person object. * Constructor for recreating the Dropbox_Person object.
*
* @param int $userId
* @param bool $isCourseAdmin
* @param bool $isCourseTutor
* @param int $courseId
* @param int $sessionId
*/ */
public function __construct( public function __construct(
int $userId, int $userId,
@ -507,12 +501,6 @@ class Dropbox_Person
/** /**
* Deletes all the received categories and work of this person. * Deletes all the received categories and work of this person.
*
* @param int $id
* @param int $courseId
* @param int $sessionId
*
* @return bool
*/ */
public function deleteReceivedWorkFolder(int $id, int $courseId = 0, int $sessionId = 0): bool public function deleteReceivedWorkFolder(int $id, int $courseId = 0, int $sessionId = 0): bool
{ {
@ -542,10 +530,6 @@ class Dropbox_Person
/** /**
* Deletes a received dropbox file of this person with id=$id. * Deletes a received dropbox file of this person with id=$id.
*
* @param int $id
* @param int $courseId
* @param int $sessionId
*/ */
public function deleteReceivedWork(int $id, int $courseId = 0, int $sessionId = 0): void public function deleteReceivedWork(int $id, int $courseId = 0, int $sessionId = 0): void
{ {
@ -579,10 +563,6 @@ class Dropbox_Person
/** /**
* Deletes a sent dropbox file of this person with id=$id. * Deletes a sent dropbox file of this person with id=$id.
*
* @param int $id
* @param int $courseId
* @param int $sessionId
*/ */
public function deleteSentWork(int $id, int $courseId = 0, int $sessionId = 0): void public function deleteSentWork(int $id, int $courseId = 0, int $sessionId = 0): void
{ {

@ -761,9 +761,6 @@ function display_add_form($viewReceivedCategory, $viewSentCategory, $view, $id =
/** /**
* Checks if there are files in the dropbox_file table that aren't used anymore in dropbox_person table. * Checks if there are files in the dropbox_file table that aren't used anymore in dropbox_person table.
* If there are, all entries concerning the file are deleted from the db + the file is deleted from the server. * If there are, all entries concerning the file are deleted from the db + the file is deleted from the server.
*
* @param int $courseId
* @param int $sessionId
*/ */
function removeUnusedFiles(int $courseId = 0, int $sessionId = 0) function removeUnusedFiles(int $courseId = 0, int $sessionId = 0)
{ {

@ -80,14 +80,6 @@ class OralExpression extends Question
/** /**
* Initialize the attributes to generate the file path. * Initialize the attributes to generate the file path.
*
* @param int $sessionId
* @param int $userId
* @param int $exerciseId
* @param int $exeId
* @param int $courseId
*
* @return void
*/ */
public function initFile( public function initFile(
int $sessionId, int $sessionId,

@ -1768,7 +1768,6 @@ class SessionManager
$trackEExercises = Database::get_main_table(TABLE_STATISTIC_TRACK_E_EXERCISES); $trackEExercises = Database::get_main_table(TABLE_STATISTIC_TRACK_E_EXERCISES);
$trackEAttempt = Database::get_main_table(TABLE_STATISTIC_TRACK_E_ATTEMPT); $trackEAttempt = Database::get_main_table(TABLE_STATISTIC_TRACK_E_ATTEMPT);
$ticket = Database::get_main_table(TABLE_TICKET_TICKET); $ticket = Database::get_main_table(TABLE_TICKET_TICKET);
$em = Database::getManager(); $em = Database::getManager();
$userId = api_get_user_id(); $userId = api_get_user_id();
@ -1890,8 +1889,8 @@ class SessionManager
'where' => [ 'where' => [
'session_id = ? AND c_id = ? AND (filename IS NOT NULL AND filename != \'\')' => [ 'session_id = ? AND c_id = ? AND (filename IS NOT NULL AND filename != \'\')' => [
$id_checked, $id_checked,
$courseId $courseId,
] ],
], ],
] ]
); );

Loading…
Cancel
Save