From 8dcf8089ce6370767a41af5b352af97ad6023d16 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Luca=20B=C3=B6sch?= <luca.boesch@bfh.ch>
Date: Fri, 17 Jun 2022 05:40:08 +0200
Subject: [PATCH] Coding style fix.

---
 .DS_Store       | Bin 6148 -> 0 bytes
 README.md       |  10 +++++-----
 db/services.php |   3 ++-
 section.php     |   9 +++++++--
 4 files changed, 14 insertions(+), 8 deletions(-)
 delete mode 100644 .DS_Store

diff --git a/.DS_Store b/.DS_Store
deleted file mode 100644
index 1897711a2f40a1aae2608ee1032a8fdb0a793fc7..0000000000000000000000000000000000000000
GIT binary patch
literal 0
HcmV?d00001

literal 6148
zcmeHKy-EW?5S~2`4BA{_VG}&C6vPzvmvCYw2&wF15=n%_3z$?kS6TTGzJoTlVqqm#
zK7nW@_y~TpySd!nC6+2O1H0ee{Ortru)DWJq#BcMjVK_Z3L2xof^Lqmom)vft62jo
zo@3CfH9N_0RPq#sFdz*4MFx29`m{|wI;F9*f7QiD-Zo6)xIIYP&{bNmkC*RT4|Uhx
z{nkF}=gtQ28KJa1%}afnP}|8)-ci{jU&Hmw&E1=+p)k)dnN_mb%c)17f2gNJ1MqM{
zXEdgBx1RdGuE+7n*K@V;d7sr&@S4@L*vqZwkoobbTV$oWIo0QLAM9qibxfXtqRq{-
z!+CtH`WS2OxhoF%e9vt8d{5QeDfc=1Y?bruZr+gdiu^7ExU*R*s|KYP2801&AY*{f
zhY*c1vX~mwM+Z9n1OWO_Yr#0+0Wh=}SxgPW0#POuXhM~JVki?1eQ5n6i>W~qPD*FS
zJT|kkFBGM-Lmz5)QjtOFg#lr}WnjsC27LeTVGl2O|GPo*B@74y|B3<S$IZBbCE2^R
xv^c(NL$nqe3;U%8bqG3p9qR#K#p`HVU<-Kwj4Y-Gp@GPcfTlq@Vc<s@_yPqlqVxa&

diff --git a/README.md b/README.md
index 326d02f..a964913 100644
--- a/README.md
+++ b/README.md
@@ -70,7 +70,7 @@ The script in the [clients](/clients/) folder can be invoked without commands:
 `python3 Rsync.py`
 
 After the start of the script, you will be asked to input a token and a host url.
-If you leave both inputs empty, the script wil lused the default values, which are coded in the script ([Will be changed](#12))
+If you leave both inputs empty, the script will use the default values which are coded in the script.
 
 ## Python with options
 
@@ -112,9 +112,9 @@ Command description:
 
 ## Uploading files
 
-There is a possibility to upload files to a moodle course. For that you need the `Moodle mobile web service`.
-If you use the script u can change the variable `web_service_token` in the Rsync.py file.
-An option to override the default is comming in the future.
+There is a possibility to upload files to a Moodle course. For that you need the `Moodle mobile web service`.
+If you use the script you can change the variable `web_service_token` in the Rsync.py file.
+An option to override the default is coming in the future.
 
 If you use cURL you need to make 3 calls:
 
@@ -125,4 +125,4 @@ If you use cURL you need to make 3 calls:
 
 ## Acknowledgements
 
-Thanks a lot to @lucaboesch for helping us in this project.
+Thanks a lot to [@lucaboesch](https://github.com/lucaboesch) for helping us in this project.
diff --git a/db/services.php b/db/services.php
index 791228b..20df35c 100644
--- a/db/services.php
+++ b/db/services.php
@@ -142,7 +142,8 @@
             'local_rsync_remove_file_from_section', 'local_rsync_rename_section', 'local_rsync_remove_section',
             'local_rsync_set_file_visibility', 'local_rsync_remove_all_files_from_section', 'local_rsync_change_course_visibility',
             'local_rsync_move_file_to_other_section', 'local_rsync_move_all_modules_to_other_section', 'local_rsync_copy_course',
-            'local_rsync_remove_all_sections', 'local_rsync_copy_module', 'local_rsync_create_section', 'local_rsync_copy_all_section_modules'),
+            'local_rsync_remove_all_sections', 'local_rsync_copy_module', 'local_rsync_create_section',
+            'local_rsync_copy_all_section_modules'),
         'restrictedusers' => 0,
         'enabled' => 1,
     )
diff --git a/section.php b/section.php
index e0620e2..041af18 100644
--- a/section.php
+++ b/section.php
@@ -779,9 +779,14 @@ public static function create_section($courseid, $sectionname) {
 
     /**
      * Lets the user copy all modules of a section and place it in a target section
-     * 
+     *
+     * @param int $courseid courseid
+     * @param int $sectionnumber section number
+     * @param int $targetsectionnumber target section number
+     * @return string A string describing the result.
+     * @throws moodle_exception if the specified course doesn't exist or the user doesn't have the rights
      */
-    public static function copy_all_section_modules($courseid, $sectionnumber, $targetsectionnumber){
+    public static function copy_all_section_modules($courseid, $sectionnumber, $targetsectionnumber) {
         global $DB, $USER;
 
         // Check parameters.