Fix moodlecodestyle from backuprestore template

Author: justusdieckmann

backup/moodle2/backup_groupmembers_activity_task.class.php

@@ -1,5 +1,4 @@
 <?php
-
 // This file is part of Moodle - http://moodle.org/
 //
 // Moodle is free software: you can redistribute it and/or modify
@@ -55,15 +54,15 @@ protected function define_my_steps() {
     static public function encode_content_links($content) {
         global $CFG;
 
-        $base = preg_quote($CFG->wwwroot,"/");
+        $base = preg_quote($CFG->wwwroot, "/");
 
         // Link to the list of groupmembers
-        $search="/(".$base."\/mod\/groupmembers\/index.php\?id\=)([0-9]+)/";
-        $content= preg_replace($search, '$@GROUPMEMBERINDEX*$2@$', $content);
+        $search = "/(".$base."\/mod\/groupmembers\/index.php\?id\=)([0-9]+)/";
+        $content = preg_replace($search, '$@GROUPMEMBERINDEX*$2@$', $content);
 
         // Link to view by moduleid
-        $search="/(".$base."\/mod\/groupmembers\/view.php\?id\=)([0-9]+)/";
-        $content= preg_replace($search, '$@GROUPMEMBERVIEWBYID*$2@$', $content);
+        $search = "/(".$base."\/mod\/groupmembers\/view.php\?id\=)([0-9]+)/";
+        $content = preg_replace($search, '$@GROUPMEMBERVIEWBYID*$2@$', $content);
 
         return $content;
     }

backup/moodle2/backup_groupmembers_stepslib.php

@@ -1,5 +1,4 @@
 <?php
-
 // This file is part of Moodle - http://moodle.org/
 //
 // Moodle is free software: you can redistribute it and/or modify
@@ -32,23 +31,23 @@ class backup_groupmembers_activity_structure_step extends backup_activity_struct
 
     protected function define_structure() {
 
-        // To know if we are including userinfo
+        // To know if we are including userinfo.
         $userinfo = $this->get_setting_value('userinfo');
 
-        // Define each element separated
+        // Define each element separated.
         $groupmembers = new backup_nested_element('groupmembers', array('id'),
                 array('name', 'intro', 'introformat', 'listgroupingid', 'showgroups', 'showemail', 'timemodified'));
 
-        // Define sources
+        // Define sources.
         $groupmembers->set_source_table('groupmembers', array('id' => backup::VAR_ACTIVITYID));
 
-        // Define id annotations
+        // Define id annotations.
 
-        // Define file annotations
+        // Define file annotations.
         $groupmembers->annotate_files('mod_groupmembers', 'intro', null);
         $groupmembers->annotate_files('mod_groupmembers', 'content', null);
 
-        // Return the root element (groupmembers), wrapped into standard activity structure
+        // Return the root element (groupmembers), wrapped into standard activity structure.
         return $this->prepare_activity_structure($groupmembers);
     }
 }

backup/moodle2/restore_groupmembers_activity_task.class.php

@@ -1,5 +1,4 @@
 <?php
-
 // This file is part of Moodle - http://moodle.org/
 //
 // Moodle is free software: you can redistribute it and/or modify
@@ -23,7 +22,7 @@
 
 defined('MOODLE_INTERNAL') || die();
 
-require_once($CFG->dirroot . '/mod/groupmembers/backup/moodle2/restore_groupmembers_stepslib.php'); // Because it exists (must)
+require_once($CFG->dirroot . '/mod/groupmembers/backup/moodle2/restore_groupmembers_stepslib.php'); // Because it exists (must).
 
 /**
  * folder restore task that provides all the settings and steps to perform one
@@ -41,7 +40,7 @@ protected function define_my_settings() {
      * Define (add) particular steps this activity can have
      */
     protected function define_my_steps() {
-        // groupmembers only has one structure step
+        // Groupmembers only has one structure step.
         $this->add_step(new restore_groupmembers_activity_structure_step('groupmembers_structure', 'groupmembers.xml'));
     }
 

backup/moodle2/restore_groupmembers_stepslib.php

@@ -1,5 +1,4 @@
 <?php
-
 // This file is part of Moodle - http://moodle.org/
 //
 // Moodle is free software: you can redistribute it and/or modify
@@ -35,25 +34,24 @@ protected function define_structure() {
         $paths = array();
         $paths[] = new restore_path_element('groupmembers', '/activity/groupmembers');
 
-        // Return the paths wrapped into standard activity structure
+        // Return the paths wrapped into standard activity structure.
         return $this->prepare_activity_structure($paths);
     }
 
     protected function process_groupmembers($data) {
         global $DB;
 
         $data = (object)$data;
-        $oldid = $data->id;
         $data->course = $this->get_courseid();
 
-        // insert the folder record
+        // Insert the folder record.
         $newitemid = $DB->insert_record('groupmembers', $data);
-        // immediately after inserting "activity" record, call this
+        // Immediately after inserting "activity" record, call this.
         $this->apply_activity_instance($newitemid);
     }
 
     protected function after_execute() {
-        // Add folder related files, no need to match by itemname (just internally handled context)
+        // Add folder related files, no need to match by itemname (just internally handled context).
         $this->add_related_files('mod_groupmembers', 'intro', null);
         $this->add_related_files('mod_groupmembers', 'content', null);
     }