Skip to content

Commit ed64147

Browse files
committed
Reduce local/moodlecheck errors.
1 parent 9d4f8af commit ed64147

21 files changed

+42
-90
lines changed

action.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@
2727
*
2828
* @package mod_pdfannotator
2929
* @copyright 2018 RWTH Aachen (see README.md)
30-
* @authors Rabea de Groot, Anna Heynkes, Friederike Schwager
30+
* @author Rabea de Groot, Anna Heynkes, Friederike Schwager
3131
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
3232
*/
3333

backup/moodle2/restore_pdfannotator_activity_task.class.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -81,9 +81,9 @@ static public function define_decode_rules() {
8181

8282
/**
8383
* Define the restore log rules that will be applied
84-
* by the {@link restore_logs_processor} when restoring
84+
* by the {@see restore_logs_processor} when restoring
8585
* pdfannotator logs. It must return one array
86-
* of {@link restore_log_rule} objects
86+
* of {@see restore_log_rule} objects
8787
*/
8888
static public function define_restore_log_rules() {
8989
$rules = array();
@@ -97,9 +97,9 @@ static public function define_restore_log_rules() {
9797

9898
/**
9999
* Define the restore log rules that will be applied
100-
* by the {@link restore_logs_processor} when restoring
100+
* by the {@see restore_logs_processor} when restoring
101101
* course logs. It must return one array
102-
* of {@link restore_log_rule} objects
102+
* of {@see restore_log_rule} objects
103103
*
104104
* Note this rules are applied when restoring course logs
105105
* by the restore final task, but are defined here at

classes/output/answermenu.php

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,6 @@ class answermenu implements \renderable, \templatable {
3737

3838
/**
3939
* Constructor of renderable for dropdown menu in answerstable.
40-
* @global type $CFG
4140
* @param int $annotationid Id of the annotation the answer belongs to
4241
* @param bool $issubscribed Is the user subscribed to the question?
4342
* @param int $cmid Course module id

classes/output/reportmenu.php

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,6 @@ class reportmenu implements \renderable, \templatable {
3636

3737
/**
3838
* Constructor of renderable for dropdown menu in reportstable.
39-
* @global type $CFG
4039
* @param object $report Report object
4140
* @param int $cmid Course module id
4241
* @param int $currentpage Page of the table on overviewpage

classes/output/statistics.php

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -42,8 +42,6 @@ class statistics implements \renderable, \templatable {
4242

4343
/**
4444
* Constructor of renderable for statistics tab.
45-
* @global type $USER
46-
* @global type $PAGE
4745
* @param int $annotatorid Id of the annotator
4846
* @param int $courseid ID of the course
4947
* @param object $capabilities Some of the capabilities the user has-

classes/privacy/provider.php

Lines changed: 14 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,10 @@
1414
// You should have received a copy of the GNU General Public License
1515
// along with Moodle. If not, see <http://www.gnu.org/licenses/>.
1616
/**
17+
* Privacy class for requesting user data.
18+
*
1719
* @package mod_pdfannotator
18-
* @category backup
20+
* @category privacy
1921
* @copyright 2018 RWTH Aachen (see README.md)
2022
* @author Anna Heynkes
2123
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
@@ -309,9 +311,9 @@ public static function delete_data_for_all_users_in_context(\context $context) {
309311

310312
/**
311313
*
312-
* @global type $DB
313-
* @param \mod_pdfannotator\privacy\approved_contextlist $contextlist
314-
* @return type
314+
* Delete personal data for the user in a list of contexts.
315+
*
316+
* @param \mod_pdfannotator\privacy\approved_contextlist $contextlist List of contexts to delete data from.
315317
*/
316318
public static function delete_data_for_user(approved_contextlist $contextlist) {
317319

@@ -372,8 +374,7 @@ public static function delete_data_for_user(approved_contextlist $contextlist) {
372374
/**
373375
* Function deletes an annotation and all comments and subscriptions attached to it.
374376
*
375-
* @global \mod_pdfannotator\privacy\type $DB
376-
* @param type $annotationid
377+
* @param int $annotationid One annotationid (int) to delete
377378
*/
378379
public static function delete_annotation($annotationid) {
379380

@@ -400,6 +401,13 @@ public static function delete_annotation($annotationid) {
400401
$DB->delete_records('pdfannotator_annotations', array("id" => $annotationid));
401402
}
402403

404+
/**
405+
* Function empties or deletes a comment.
406+
*
407+
* @param \mod_pdfannotator\output\comment $comment comment to be emptied or deleted
408+
*
409+
* @throws \dml_exception
410+
*/
403411
public static function empty_or_delete_comment($comment) {
404412

405413
global $DB;

constants.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616
/**
1717
* @package mod_pdfannotator
1818
* @copyright 2018 RWTH Aachen (see README.md)
19-
* @authors Rabea de Groot, Anna Heynkes, Friederike Schwager
19+
* @author Rabea de Groot, Anna Heynkes, Friederike Schwager
2020
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
2121
*/
2222

controller.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
/**
1818
* @package mod_pdfannotator
1919
* @copyright 2018 RWTH Aachen (see README.md)
20-
* @authors Anna Heynkes, Friederike Schwager
20+
* @author Anna Heynkes, Friederike Schwager
2121
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
2222
*/
2323

db/access.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
/**
1818
* @package mod_pdfannotator
1919
* @copyright 2018 RWTH Aachen (see README.md)
20-
* @authors Rabea de Groot and Anna Heynkes
20+
* @author Rabea de Groot and Anna Heynkes
2121
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
2222
*/
2323
defined('MOODLE_INTERNAL') || die();

db/install.php

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@
2222
/**
2323
* Function initialises the pdfannotator_annotationtypes table with its 6 standard records.
2424
*
25-
* @global type $DB
2625
*/
2726
defined('MOODLE_INTERNAL') || die();
2827

0 commit comments

Comments
 (0)