UFSC / moodle-local-relationship

classes/progress_tracker.php

Indentation -1 removed lines

@@ -16,7 +16,6 @@
 
 /**
  * class based on core/admin/uu_progress_tracker by Petr Skoda
-
  * @package    local_relationship
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */

Spacing +4 added lines, -4 removed lines

@@ -57,16 +57,16 @@
         $table->tablealign = 'center';
 
         $table->head = array(get_string('titleline', 'local_relationship'));
-        $table->align = array ('right');
-        foreach($this->columns AS $col) {
-            $table->head[] = get_string('title'.$col, 'local_relationship');
+        $table->align = array('right');
+        foreach ($this->columns AS $col) {
+            $table->head[] = get_string('title' . $col, 'local_relationship');
             $table->align[] = 'left';
         }
 
         $table->data = array();
         foreach ($this->rows AS $linenum => $r) {
             $row = array($linenum . '.');
-            foreach($r AS $col=>$occurs) {
+            foreach ($r AS $col=>$occurs) {
                 $text = '';
                 foreach ($occurs AS $occur) {
                     $msg = html_writer::tag('div', $occur['msg'], array('class' => 'alert-' . $occur['level']));

settings.php

Spacing +2 added lines, -2 removed lines

@@ -21,11 +21,11 @@
     $settings->add(new admin_setting_heading('local_relationship/settings', get_string('massassignusers', 'local_relationship'), get_string('massassignusers_desc', 'local_relationship')));
 
     $optionsauthtype = array();
-    foreach(get_enabled_auth_plugins(false) as $opt){
+    foreach (get_enabled_auth_plugins(false) as $opt) {
         $optionsauthtype[$opt] = $opt;
     }
     $settings->add(new admin_setting_configselect('local_relationship/authtype', get_string('authtype', 'local_relationship'),
-        get_string('authtype_desc', 'local_relationship') ,'manual', $optionsauthtype));
+        get_string('authtype_desc', 'local_relationship'), 'manual', $optionsauthtype));
 
     $options = array(0=>get_string('no'), 1=>get_string('yes'));
     $setting = new admin_setting_configselect('local_relationship/searchsccp', get_string('searchsccp', 'local_relationship'),

locallib.php

Spacing +15 added lines, -15 removed lines

@@ -26,7 +26,7 @@ 
 require_once($CFG->dirroot . '/user/selector/lib.php');
 require_once($CFG->dirroot . '/tag/lib.php');
 
-function relationship_set_header($context, $url, $relationship=null, $module=null) {
+function relationship_set_header($context, $url, $relationship = null, $module = null) {
     global $PAGE, $COURSE, $DB;
 
     if ($context->contextlevel != CONTEXT_COURSECAT) {
@@ -43,27 +43,27 @@ 
 
     $PAGE->navbar->add($category->name, new moodle_url('/course/index.php', array('categoryid'=>$category->id)));
     $PAGE->navbar->add($navtitle, new moodle_url('/local/relationship/index.php', array('contextid'=>$context->id)));
-    if($module) {
+    if ($module) {
         $PAGE->navbar->add(get_string($module, 'local_relationship'),
                            new moodle_url("/local/relationship/{$module}.php", array('relationshipid'=>$relationship->id)));
     }
 }
 
-function relationship_set_title($relationship=null, $action=null, $param=null) {
+function relationship_set_title($relationship = null, $action = null, $param = null) {
     global $OUTPUT;
 
     echo $OUTPUT->header();
-    if($relationship) {
+    if ($relationship) {
         echo $OUTPUT->heading(get_string('relationship', 'local_relationship') . ': ' . format_string($relationship->name));
         echo html_writer::empty_tag('BR');
     }
-    if($action) {
+    if ($action) {
         echo $OUTPUT->heading(get_string($action, 'local_relationship', $param), '4');
     }
 }
 
-function relationship_groups_parse_name($format, $value, $value_is_a_name=false) {
-    if($value_is_a_name) {
+function relationship_groups_parse_name($format, $value, $value_is_a_name = false) {
+    if ($value_is_a_name) {
         if (strstr($format, '@') !== false) {
             $str = str_replace('@', $value, $format);
         } else {
@@ -72,7 +72,7 @@ 
     } else {
         if (strstr($format, '@') !== false) { // Convert $value to a character series
             $letter = 'A';
-            for($i=0; $i<$value; $i++) {
+            for ($i = 0; $i < $value; $i++) {
                 $letter++;
             }
             $str = str_replace('@', $letter, $format);
@@ -87,8 +87,8 @@ 
     $all_roles = role_get_names();
     $ctx_roles = get_roles_for_contextlevels(CONTEXT_COURSE);
     $roles = array();
-    foreach($ctx_roles AS $id=>$roleid) {
-        if($roleid > 2) {
+    foreach ($ctx_roles AS $id=>$roleid) {
+        if ($roleid > 2) {
             $roles[$roleid] = $all_roles[$roleid]->localname;
         }
     }
@@ -103,7 +103,7 @@ 
     $context = context::instance_by_id($relationship->contextid);
 
     $contextids = array();
-    foreach($context->get_parent_context_ids(true) as $ctxid) {
+    foreach ($context->get_parent_context_ids(true) as $ctxid) {
         $context = context::instance_by_id($ctxid);
         if (has_capability('moodle/cohort:view', $context)) {
             $contextids[] = $ctxid;
@@ -139,7 +139,7 @@ 
                AND g.idnumber NOT LIKE 'relationship_{$relationshipid}_%'
           ORDER BY g.name";
     $groups = array();
-    foreach($DB->get_records_sql($sql, array('relationshipid'=>$relationshipid)) as $r) {
+    foreach ($DB->get_records_sql($sql, array('relationshipid'=>$relationshipid)) as $r) {
         $groups[$r->name][] = $r;
     }
     return $groups;
@@ -223,7 +223,7 @@ 
     $allrelationships = $DB->count_records('relationship', array('contextid'=>$contextid));
     $totalrelationships = $DB->count_records_sql($countfields . $sql, $params);
     $relationships = $DB->get_records_sql($fields . $sql . $order, $params, $page*$perpage, $perpage);
-    foreach($relationships as $rl) {
+    foreach ($relationships as $rl) {
         $rl->tags = core_tag_tag::get_item_tags_array('relationship', 'relationship', $rl->id);
     }
 
@@ -241,7 +241,7 @@ 
  * @return mixed
  * @throws coding_exception
  */
-function relationship_get_cohort_by_roleshortname($relationshipid, $roleshortname, $strictness=IGNORE_MISSING) {
+function relationship_get_cohort_by_roleshortname($relationshipid, $roleshortname, $strictness = IGNORE_MISSING) {
     global $DB;
 
     $sql = "SELECT rc.*, ch.name, ch.component, r.name rolename
@@ -266,7 +266,7 @@ 
     return $OUTPUT->box($text, 'generalbox', 'notice');
 }
 
-function relationship_print_tabs($url, $tabs, $action='') {
+function relationship_print_tabs($url, $tabs, $action = '') {
     global $OUTPUT;
 
     if (is_array($tabs) && count($tabs) > 1) {

classes/form/mass_assign.php

Spacing +4 added lines, -4 removed lines

@@ -25,14 +25,14 @@ 
 defined('MOODLE_INTERNAL') || die();
 
 use local_relationship\mass_assign_processor;
-require_once($CFG->dirroot.'/lib/formslib.php');
+require_once($CFG->dirroot . '/lib/formslib.php');
 
 class mass_assign extends \moodleform {
 
     /**
      * Form Definition
      */
-    function definition () {
+    function definition() {
 
         $mform = $this->_form;
         $relationshipgroup = $this->_customdata['data'];
@@ -48,7 +48,7 @@ 
         $rcs = relationship_get_cohort_by_roleshortname($relationshipgroup->relationshipid, 'student');
 
         $cohortrole = get_string('none');
-        if($rcs) {
+        if ($rcs) {
             $r = get_string('role');
             $cohortrole = "{$rcs->name}  ({$r}: $rcs->rolename)";
         }
@@ -68,7 +68,7 @@ 
             $mform->addHelpButton('allowallusers', 'allowallusers', 'local_relationship');
         }
 
-        if(mass_assign_processor::search_sccp()) {
+        if (mass_assign_processor::search_sccp()) {
             $mform->addElement('checkbox', 'registerpendencies', get_string('registerpendencies', 'local_relationship'), false);
             $mform->addHelpButton('registerpendencies', 'registerpendencies', 'local_relationship');
         }

mass_assign.php

Spacing +7 added lines, -7 removed lines

@@ -22,9 +22,9 @@ 
  * @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
 
-require_once(__DIR__.'/../../config.php');
-require($CFG->dirroot.'/local/relationship/lib.php');
-require_once($CFG->dirroot.'/local/relationship/locallib.php');
+require_once(__DIR__ . '/../../config.php');
+require($CFG->dirroot . '/local/relationship/lib.php');
+require_once($CFG->dirroot . '/local/relationship/locallib.php');
 
 require_login();
 
@@ -38,7 +38,7 @@ 
 // Therefore, the mass assign only works in cohort without component.
 // When uniform distribution is enabled, users are uniformly distribute to any group with uniformdistribution = 1 in the relationship,
 // so uniform distribution needs to be disabled to mass assign users in a specific group.
-if(!empty($relationshipcohort->component) || $relationshipcohort->uniformdistribution || $relationshipgroup->uniformdistribution) {
+if (!empty($relationshipcohort->component) || $relationshipcohort->uniformdistribution || $relationshipgroup->uniformdistribution) {
     print_error('cantmassassign', 'local_relationship');
 }
 
@@ -126,7 +126,7 @@ 
             $rcs = relationship_get_cohort_by_roleshortname($relationshipgroup->relationshipid, 'student');
 
             $cohortrole = get_string('none');
-            if($rcs) {
+            if ($rcs) {
                 $r = get_string('role');
                 $cohortrole = "{$rcs->name}  ({$r}: $rcs->rolename)";
             }
@@ -137,9 +137,9 @@ 
 
             $recs = $DB->get_records('relationship_pendencies', array('relationshipgroupid'=>$relationshipgroup->id, 'relationshipcohortid'=>$relationshipcohort->id), 'cpf');
             $table = new html_table();
-            $table->head  = array('CPF', get_string('timecreated', 'local_relationship'));
+            $table->head = array('CPF', get_string('timecreated', 'local_relationship'));
             $table->data = array();
-            foreach($recs AS $rec) {
+            foreach ($recs AS $rec) {
                 $line = array($rec->cpf, userdate($rec->timecreated));
                 $baseurl->param('id', $rec->id);
                 $baseurl->param('delete', 1);

classes/observer.php

Spacing +3 added lines, -3 removed lines

@@ -23,7 +23,7 @@ 
 
 defined('MOODLE_INTERNAL') || die();
 
-require_once($CFG->dirroot.'/local/relationship/lib.php');
+require_once($CFG->dirroot . '/local/relationship/lib.php');
 
 /**
  * Event handler for relationship local plugin.
@@ -114,7 +114,7 @@ 
             $idnumber = $DB->get_field('user', 'idnumber', array('id'=>$event->userid));
             if (!empty($idnumber)) {
                 $pessoas = \local_ufsc\pessoa::by_key('idpessoa', $idnumber);
-                if(!empty($pessoas)) {
+                if (!empty($pessoas)) {
                     $pessoa = reset($pessoas);
                     if (!empty($pessoa->cpf)) {
                         $where = 'rp.cpf = :cpf';
@@ -130,7 +130,7 @@ 
                             if (!empty($rec->relationshipcohortid) && !empty($rec->relationshipgroupid)) {
                                 $massassign = new \local_relationship\mass_assign_processor($rec->relationshipcohortid, $rec->relationshipgroupid, false, $rec->allowallusers);
                                 $result = $massassign->process_relationship($event->userid);
-                                if (in_array($result->summary,['assigneduser', 'alreadyingroup'])) {
+                                if (in_array($result->summary, ['assigneduser', 'alreadyingroup'])) {
                                     $DB->delete_records('relationship_pendencies', array('id' => $rec->id));
                                 }
                             }

classes/mass_assign_processor.php

Spacing +6 added lines, -6 removed lines

@@ -25,7 +25,7 @@ 
 
 defined('MOODLE_INTERNAL') || die();
 
-require_once($CFG->dirroot.'/cohort/lib.php');
+require_once($CFG->dirroot . '/cohort/lib.php');
 
 class mass_assign_processor {
 
@@ -36,7 +36,7 @@ 
     private $registerpendencies;
     private $allowallusers;
 
-    public function __construct($relationshipcohortid, $relationshipgroupid, $registerpendencies=false, $allowallusers=false, $authtype='') {
+    public function __construct($relationshipcohortid, $relationshipgroupid, $registerpendencies = false, $allowallusers = false, $authtype = '') {
         global $DB;
 
         $this->relationshipcohort = relationship_get_cohort($relationshipcohortid);
@@ -45,7 +45,7 @@ 
         }
 
         $this->relationshipgroup = $DB->get_record('relationship_groups', array('id' => $relationshipgroupid), '*', MUST_EXIST);
-        if($this->relationshipgroup->uniformdistribution) {
+        if ($this->relationshipgroup->uniformdistribution) {
             throw new \moodle_exception('cantmassassign', 'local_relationship');
         }
 
@@ -415,7 +415,7 @@ 
                     'level' => 'error'
                 ];
             } else {
-                $data = (object)$params;
+                $data = (object) $params;
                 $data->timecreated = time();
                 $DB->insert_record('relationship_pendencies', $data);
                 $result->summary = 'registeredaspending';
@@ -443,9 +443,9 @@ 
         } else {
             for ($t = 9; $t < 11; $t++) {
                 for ($d = 0, $c = 0; $c < $t; $c++) {
-                    $d += $cpf{$c} * (($t + 1) - $c);
+                    $d += $cpf{$c}*(($t+1)-$c);
                 }
-                $d = ((10 * $d) % 11) % 10;
+                $d = ((10*$d)%11)%10;
                 if ($cpf{$c} != $d) {
                     return false;
                 }

groups.php

Spacing +9 added lines, -9 removed lines

@@ -21,10 +21,10 @@ 
  * @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
 
-require_once(__DIR__.'/../../config.php');
-require($CFG->dirroot.'/local/relationship/lib.php');
-require_once($CFG->dirroot.'/local/relationship/locallib.php');
-require_once($CFG->libdir.'/adminlib.php');
+require_once(__DIR__ . '/../../config.php');
+require($CFG->dirroot . '/local/relationship/lib.php');
+require_once($CFG->dirroot . '/local/relationship/locallib.php');
+require_once($CFG->libdir . '/adminlib.php');
 
 require_login();
 
@@ -65,7 +65,7 @@ 
     }
     if ($editable) {
         $url = new moodle_url('/local/relationship/edit_group.php', array('relationshipgroupid' => $relationshipgroup->id, 'uniformdistribution' => $uniformdistribution));
-        $line[] = $status.' ('.html_writer::link($url, $text).')';
+        $line[] = $status . ' (' . html_writer::link($url, $text) . ')';
     } else {
         $line[] = $status;
     }
@@ -80,7 +80,7 @@ 
     if ($manager || $canassign) {
         $buttons[] = html_writer::link(new moodle_url('/local/relationship/assign.php', array('relationshipgroupid' => $relationshipgroup->id)),
                 html_writer::span($OUTPUT->pix_icon('t/assignroles', get_string('assign', 'local_relationship'), 'moodle', ['class' => 'iconsmall'])));
-        if($relationshipcohort && empty($relationshipcohort->component) && !$relationshipcohort->uniformdistribution && !$relationshipgroup->uniformdistribution) {
+        if ($relationshipcohort && empty($relationshipcohort->component) && !$relationshipcohort->uniformdistribution && !$relationshipgroup->uniformdistribution) {
             $buttons[] = html_writer::link(new moodle_url('/local/relationship/mass_assign.php', array('relationshipgroupid' => $relationshipgroup->id)),
                     html_writer::span($OUTPUT->pix_icon('t/add', get_string('massassign', 'local_relationship'), 'moodle', ['class' => 'iconsmall'])));
         }
@@ -92,8 +92,8 @@ 
 $table = new html_table();
 $table->head = array(get_string('name', 'local_relationship'),
         get_string('memberscount', 'local_relationship'),
-        get_string('limit', 'local_relationship').$OUTPUT->help_icon('userlimit', 'local_relationship'),
-        get_string('uniformdistribute', 'local_relationship').$OUTPUT->help_icon('uniformdistribute', 'local_relationship'),
+        get_string('limit', 'local_relationship') . $OUTPUT->help_icon('userlimit', 'local_relationship'),
+        get_string('uniformdistribute', 'local_relationship') . $OUTPUT->help_icon('uniformdistribute', 'local_relationship'),
         get_string('edit'));
 $table->colclasses = array('leftalign name',
         'leftalign size',
@@ -140,7 +140,7 @@ 
 
     $addgroup = new single_button(new moodle_url('/local/relationship/edit_group.php', array('relationshipid' => $relationshipid)), get_string('addgroup', 'local_relationship'));
     $addgroups = new single_button(new moodle_url('/local/relationship/autogroup.php', array('relationshipid' => $relationshipid)), get_string('autogroup', 'local_relationship'));
-    echo html_writer::tag('div', $OUTPUT->render($addgroup).$OUTPUT->render($addgroups), array('class' => 'buttons'));
+    echo html_writer::tag('div', $OUTPUT->render($addgroup) . $OUTPUT->render($addgroups), array('class' => 'buttons'));
 } else if ($manager) {
     echo $OUTPUT->heading(get_string('noeditable', 'local_relationship'));
 }