Passed
Push — 25073_inscricao_massa ( 897954...b0b425 )
by
unknown
06:01
created
groups.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -21,10 +21,10 @@  discard block
 block discarded – undo
21 21
  * @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
22 22
  */
23 23
 
24
-require_once(__DIR__.'/../../config.php');
25
-require($CFG->dirroot.'/local/relationship/lib.php');
26
-require_once($CFG->dirroot.'/local/relationship/locallib.php');
27
-require_once($CFG->libdir.'/adminlib.php');
24
+require_once(__DIR__ . '/../../config.php');
25
+require($CFG->dirroot . '/local/relationship/lib.php');
26
+require_once($CFG->dirroot . '/local/relationship/locallib.php');
27
+require_once($CFG->libdir . '/adminlib.php');
28 28
 
29 29
 require_login();
30 30
 
@@ -65,7 +65,7 @@  discard block
 block discarded – undo
65 65
     }
66 66
     if ($editable) {
67 67
         $url = new moodle_url('/local/relationship/edit_group.php', array('relationshipgroupid' => $relationshipgroup->id, 'uniformdistribution' => $uniformdistribution));
68
-        $line[] = $status.' ('.html_writer::link($url, $text).')';
68
+        $line[] = $status . ' (' . html_writer::link($url, $text) . ')';
69 69
     } else {
70 70
         $line[] = $status;
71 71
     }
@@ -80,7 +80,7 @@  discard block
 block discarded – undo
80 80
     if ($manager || $canassign) {
81 81
         $buttons[] = html_writer::link(new moodle_url('/local/relationship/assign.php', array('relationshipgroupid' => $relationshipgroup->id)),
82 82
                 html_writer::empty_tag('img', array('src' => $OUTPUT->pix_url('t/assignroles'), 'alt' => get_string('assign', 'local_relationship'), 'title' => get_string('assign', 'local_relationship'), 'class' => 'iconsmall')));
83
-        if($relationshipcohort && !$relationshipcohort->uniformdistribution) {
83
+        if ($relationshipcohort && !$relationshipcohort->uniformdistribution) {
84 84
             $buttons[] = html_writer::link(new moodle_url('/local/relationship/mass_assign.php', array('relationshipgroupid' => $relationshipgroup->id)),
85 85
                 html_writer::empty_tag('img', array('src' => $OUTPUT->pix_url('t/add'), 'alt' => get_string('massassign', 'local_relationship'), 'title' => get_string('massassign', 'local_relationship'), 'class' => 'iconsmall')));
86 86
         }
@@ -92,8 +92,8 @@  discard block
 block discarded – undo
92 92
 $table = new html_table();
93 93
 $table->head = array(get_string('name', 'local_relationship'),
94 94
         get_string('memberscount', 'local_relationship'),
95
-        get_string('limit', 'local_relationship').$OUTPUT->help_icon('userlimit', 'local_relationship'),
96
-        get_string('uniformdistribute', 'local_relationship').$OUTPUT->help_icon('uniformdistribute', 'local_relationship'),
95
+        get_string('limit', 'local_relationship') . $OUTPUT->help_icon('userlimit', 'local_relationship'),
96
+        get_string('uniformdistribute', 'local_relationship') . $OUTPUT->help_icon('uniformdistribute', 'local_relationship'),
97 97
         get_string('edit'));
98 98
 $table->colclasses = array('leftalign name',
99 99
         'leftalign size',
@@ -140,7 +140,7 @@  discard block
 block discarded – undo
140 140
 
141 141
     $addgroup = new single_button(new moodle_url('/local/relationship/edit_group.php', array('relationshipid' => $relationshipid)), get_string('addgroup', 'local_relationship'));
142 142
     $addgroups = new single_button(new moodle_url('/local/relationship/autogroup.php', array('relationshipid' => $relationshipid)), get_string('autogroup', 'local_relationship'));
143
-    echo html_writer::tag('div', $OUTPUT->render($addgroup).$OUTPUT->render($addgroups), array('class' => 'buttons'));
143
+    echo html_writer::tag('div', $OUTPUT->render($addgroup) . $OUTPUT->render($addgroups), array('class' => 'buttons'));
144 144
 } else if ($manager) {
145 145
     echo $OUTPUT->heading(get_string('noeditable', 'local_relationship'));
146 146
 }
Please login to merge, or discard this patch.
classes/form/mass_assign.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -24,14 +24,14 @@  discard block
 block discarded – undo
24 24
 namespace local_relationship\form;
25 25
 defined('MOODLE_INTERNAL') || die();
26 26
 
27
-require_once($CFG->dirroot.'/lib/formslib.php');
27
+require_once($CFG->dirroot . '/lib/formslib.php');
28 28
 
29 29
 class mass_assign extends \moodleform {
30 30
 
31 31
     /**
32 32
      * Form Definition
33 33
      */
34
-    function definition () {
34
+    function definition() {
35 35
 
36 36
         $mform = $this->_form;
37 37
         $relationshipgroup = $this->_customdata['data'];
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
         $rcs = relationship_get_cohort_by_roleshortname($relationshipgroup->relationshipid, 'student');
48 48
 
49 49
         $cohortrole = get_string('none');
50
-        if($rcs) {
50
+        if ($rcs) {
51 51
             $r = get_string('role');
52 52
             $cohortrole = "{$rcs->name}  ({$r}: $rcs->rolename)";
53 53
         }
Please login to merge, or discard this patch.
classes/progress_tracker.php 2 patches
Indentation   -1 removed lines patch added patch discarded remove patch
@@ -16,7 +16,6 @@
 block discarded – undo
16 16
 
17 17
 /**
18 18
  * class based on core/admin/uu_progress_tracker by Petr Skoda
19
-
20 19
  * @package    local_relationship
21 20
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
22 21
  */
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -57,16 +57,16 @@
 block discarded – undo
57 57
         $table->tablealign = 'center';
58 58
 
59 59
         $table->head = array(get_string('titleline', 'local_relationship'));
60
-        $table->align = array ('right');
61
-        foreach($this->columns AS $col) {
62
-            $table->head[] = get_string('title'.$col, 'local_relationship');
60
+        $table->align = array('right');
61
+        foreach ($this->columns AS $col) {
62
+            $table->head[] = get_string('title' . $col, 'local_relationship');
63 63
             $table->align[] = 'left';
64 64
         }
65 65
 
66 66
         $table->data = array();
67 67
         foreach ($this->rows AS $linenum => $r) {
68 68
             $row = array($linenum . '.');
69
-            foreach($r AS $col=>$occurs) {
69
+            foreach ($r AS $col=>$occurs) {
70 70
                 $text = '';
71 71
                 foreach ($occurs AS $occur) {
72 72
                     $msg = html_writer::tag('div', $occur['msg'], array('class' => 'alert-' . $occur['level']));
Please login to merge, or discard this patch.
locallib.php 1 patch
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -26,7 +26,7 @@  discard block
 block discarded – undo
26 26
 require_once($CFG->dirroot . '/user/selector/lib.php');
27 27
 require_once($CFG->dirroot . '/tag/lib.php');
28 28
 
29
-function relationship_set_header($context, $url, $relationship=null, $module=null) {
29
+function relationship_set_header($context, $url, $relationship = null, $module = null) {
30 30
     global $PAGE, $COURSE, $DB;
31 31
 
32 32
     if ($context->contextlevel != CONTEXT_COURSECAT) {
@@ -43,27 +43,27 @@  discard block
 block discarded – undo
43 43
 
44 44
     $PAGE->navbar->add($category->name, new moodle_url('/course/index.php', array('categoryid'=>$category->id)));
45 45
     $PAGE->navbar->add($navtitle, new moodle_url('/local/relationship/index.php', array('contextid'=>$context->id)));
46
-    if($module) {
46
+    if ($module) {
47 47
         $PAGE->navbar->add(get_string($module, 'local_relationship'),
48 48
                            new moodle_url("/local/relationship/{$module}.php", array('relationshipid'=>$relationship->id)));
49 49
     }
50 50
 }
51 51
 
52
-function relationship_set_title($relationship=null, $action=null, $param=null) {
52
+function relationship_set_title($relationship = null, $action = null, $param = null) {
53 53
     global $OUTPUT;
54 54
 
55 55
     echo $OUTPUT->header();
56
-    if($relationship) {
56
+    if ($relationship) {
57 57
         echo $OUTPUT->heading(get_string('relationship', 'local_relationship') . ': ' . format_string($relationship->name));
58 58
         echo html_writer::empty_tag('BR');
59 59
     }
60
-    if($action) {
60
+    if ($action) {
61 61
         echo $OUTPUT->heading(get_string($action, 'local_relationship', $param), '4');
62 62
     }
63 63
 }
64 64
 
65
-function relationship_groups_parse_name($format, $value, $value_is_a_name=false) {
66
-    if($value_is_a_name) {
65
+function relationship_groups_parse_name($format, $value, $value_is_a_name = false) {
66
+    if ($value_is_a_name) {
67 67
         if (strstr($format, '@') !== false) {
68 68
             $str = str_replace('@', $value, $format);
69 69
         } else {
@@ -72,7 +72,7 @@  discard block
 block discarded – undo
72 72
     } else {
73 73
         if (strstr($format, '@') !== false) { // Convert $value to a character series
74 74
             $letter = 'A';
75
-            for($i=0; $i<$value; $i++) {
75
+            for ($i = 0; $i < $value; $i++) {
76 76
                 $letter++;
77 77
             }
78 78
             $str = str_replace('@', $letter, $format);
@@ -87,8 +87,8 @@  discard block
 block discarded – undo
87 87
     $all_roles = role_get_names();
88 88
     $ctx_roles = get_roles_for_contextlevels(CONTEXT_COURSE);
89 89
     $roles = array();
90
-    foreach($ctx_roles AS $id=>$roleid) {
91
-        if($roleid > 2) {
90
+    foreach ($ctx_roles AS $id=>$roleid) {
91
+        if ($roleid > 2) {
92 92
             $roles[$roleid] = $all_roles[$roleid]->localname;
93 93
         }
94 94
     }
@@ -103,7 +103,7 @@  discard block
 block discarded – undo
103 103
     $context = context::instance_by_id($relationship->contextid);
104 104
 
105 105
     $contextids = array();
106
-    foreach($context->get_parent_context_ids(true) as $ctxid) {
106
+    foreach ($context->get_parent_context_ids(true) as $ctxid) {
107 107
         $context = context::instance_by_id($ctxid);
108 108
         if (has_capability('moodle/cohort:view', $context)) {
109 109
             $contextids[] = $ctxid;
@@ -139,7 +139,7 @@  discard block
 block discarded – undo
139 139
                AND g.idnumber NOT LIKE 'relationship_{$relationshipid}_%'
140 140
           ORDER BY g.name";
141 141
     $groups = array();
142
-    foreach($DB->get_records_sql($sql, array('relationshipid'=>$relationshipid)) as $r) {
142
+    foreach ($DB->get_records_sql($sql, array('relationshipid'=>$relationshipid)) as $r) {
143 143
         $groups[$r->name][] = $r;
144 144
     }
145 145
     return $groups;
@@ -223,7 +223,7 @@  discard block
 block discarded – undo
223 223
     $allrelationships = $DB->count_records('relationship', array('contextid'=>$contextid));
224 224
     $totalrelationships = $DB->count_records_sql($countfields . $sql, $params);
225 225
     $relationships = $DB->get_records_sql($fields . $sql . $order, $params, $page*$perpage, $perpage);
226
-    foreach($relationships as $rl) {
226
+    foreach ($relationships as $rl) {
227 227
         $rl->tags = core_tag_tag::get_item_tags_array('relationship', 'relationship', $rl->id);
228 228
     }
229 229
 
@@ -244,9 +244,9 @@  discard block
 block discarded – undo
244 244
     } else {
245 245
         for ($t = 9; $t < 11; $t++) {
246 246
             for ($d = 0, $c = 0; $c < $t; $c++) {
247
-                $d += $cpf{$c} * (($t + 1) - $c);
247
+                $d += $cpf{$c}*(($t+1)-$c);
248 248
             }
249
-            $d = ((10 * $d) % 11) % 10;
249
+            $d = ((10*$d)%11)%10;
250 250
             if ($cpf{$c} != $d) {
251 251
                 return false;
252 252
             }
@@ -266,7 +266,7 @@  discard block
 block discarded – undo
266 266
  * @return mixed
267 267
  * @throws coding_exception
268 268
  */
269
-function relationship_get_cohort_by_roleshortname($relationshipid, $roleshortname, $strictness=IGNORE_MISSING) {
269
+function relationship_get_cohort_by_roleshortname($relationshipid, $roleshortname, $strictness = IGNORE_MISSING) {
270 270
     global $DB;
271 271
 
272 272
     $sql = "SELECT rc.*, ch.name, r.name rolename
Please login to merge, or discard this patch.
mass_assign.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -22,11 +22,11 @@  discard block
 block discarded – undo
22 22
  * @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
23 23
  */
24 24
 
25
-require_once(__DIR__.'/../../config.php');
26
-require_once($CFG->dirroot.'/course/lib.php');
27
-require_once($CFG->dirroot.'/cohort/lib.php');
28
-require($CFG->dirroot.'/local/relationship/lib.php');
29
-require_once($CFG->dirroot.'/local/relationship/locallib.php');
25
+require_once(__DIR__ . '/../../config.php');
26
+require_once($CFG->dirroot . '/course/lib.php');
27
+require_once($CFG->dirroot . '/cohort/lib.php');
28
+require($CFG->dirroot . '/local/relationship/lib.php');
29
+require_once($CFG->dirroot . '/local/relationship/locallib.php');
30 30
 
31 31
 require_login();
32 32
 
@@ -35,7 +35,7 @@  discard block
 block discarded – undo
35 35
 $relationship = $DB->get_record('relationship', array('id' => $relationshipgroup->relationshipid), '*', MUST_EXIST);
36 36
 
37 37
 $relationshipcohort = relationship_get_cohort_by_roleshortname($relationship->id, 'student', MUST_EXIST);
38
-if($relationshipcohort->uniformdistribution) {
38
+if ($relationshipcohort->uniformdistribution) {
39 39
     print_error('cantmassassign', 'local_relationship');
40 40
 }
41 41
 
Please login to merge, or discard this patch.