Completed
Pull Request — master (#194)
by
unknown
15:06
created
lib/midcom/services/cache/main.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -143,7 +143,7 @@
 block discarded – undo
143 143
 
144 144
         $this->_modules[$name] = new $classname();
145 145
         $this->_modules[$name]->initialize();
146
-        $this->$name =& $this->_modules[$name];
146
+        $this->$name = & $this->_modules[$name];
147 147
         array_unshift($this->_unload_queue, $name);
148 148
     }
149 149
 
Please login to merge, or discard this patch.
lib/midcom/db/attachment.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -336,7 +336,7 @@
 block discarded – undo
336 336
     public function update_cache()
337 337
     {
338 338
         // Check if the attachment can be read anonymously
339
-        if (   midcom::get()->config->get('attachment_cache_enabled')
339
+        if (midcom::get()->config->get('attachment_cache_enabled')
340 340
             && !$this->can_do('midgard:read', 'EVERYONE')) {
341 341
             // Not public file, ensure it is removed
342 342
             $filename = $this->get_cache_path();
Please login to merge, or discard this patch.
lib/midcom/helper/nav/item.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -98,8 +98,8 @@
 block discarded – undo
98 98
         }
99 99
 
100 100
         // Check the Metadata if and only if we are configured to do so.
101
-        if (   is_object($this->object)
102
-            && (   midcom::get()->config->get('show_hidden_objects') == false
101
+        if (is_object($this->object)
102
+            && (midcom::get()->config->get('show_hidden_objects') == false
103 103
                 || midcom::get()->config->get('show_unapproved_objects') == false)) {
104 104
             // Check Hiding, Scheduling and Approval
105 105
             $metadata = $this->object->metadata;
Please login to merge, or discard this patch.
lib/midcom/helper/nav/leaf.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -28,7 +28,7 @@
 block discarded – undo
28 28
 
29 29
     public function is_readable_by($user_id)
30 30
     {
31
-        return (   empty($this->object)
31
+        return (empty($this->object)
32 32
                 || !$this->guid
33 33
                 || !$user_id
34 34
                 || midcom::get()->auth->acl->can_do_byguid('midgard:read', $this->guid, $this->object->__midcom_class_name__, $user_id));
Please login to merge, or discard this patch.
lib/org/openpsa/documents/midcom/interfaces.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -39,7 +39,7 @@
 block discarded – undo
39 39
 
40 40
     public function resolve_object_link(midcom_db_topic $topic, midcom_core_dbaobject $object)
41 41
     {
42
-        if (   $object instanceof org_openpsa_documents_document_dba
42
+        if ($object instanceof org_openpsa_documents_document_dba
43 43
             && $object->topic == $topic->id) {
44 44
             return "document/{$object->guid}/";
45 45
         }
Please login to merge, or discard this patch.
lib/org/openpsa/projects/workflow.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -170,7 +170,7 @@  discard block
 block discarded – undo
170 170
     {
171 171
         debug_add("task->start() called with user #" . midcom_connection::get_user());
172 172
         //PONDER: Check actual status objects for more accurate logic ?
173
-        if (   $task->status >= org_openpsa_projects_task_status_dba::STARTED
173
+        if ($task->status >= org_openpsa_projects_task_status_dba::STARTED
174 174
             && $task->status <= org_openpsa_projects_task_status_dba::APPROVED) {
175 175
             //We already have started status
176 176
             debug_add('Task has already been started');
@@ -382,7 +382,7 @@  discard block
 block discarded – undo
382 382
 
383 383
     private static function is_manager($task)
384 384
     {
385
-        return (   $task->manager == 0
385
+        return ($task->manager == 0
386 386
                 || midcom_connection::get_user() == $task->manager);
387 387
     }
388 388
 }
Please login to merge, or discard this patch.
lib/org/openpsa/projects/task.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -93,7 +93,7 @@  discard block
 block discarded – undo
93 93
     public function _on_updated()
94 94
     {
95 95
         // Sync the object's ACL properties into MidCOM ACL system
96
-        if (   !$this->_skip_acl_refresh) {
96
+        if (!$this->_skip_acl_refresh) {
97 97
             if ($this->orgOpenpsaAccesstype && $this->orgOpenpsaOwnerWg) {
98 98
                 debug_add("Synchronizing task ACLs to MidCOM");
99 99
                 $sync = new org_openpsa_core_acl_synchronizer();
@@ -326,7 +326,7 @@  discard block
 block discarded – undo
326 326
 
327 327
             if ($report_data['invoice']) {
328 328
                 $hours['invoiced'] += $report_hours;
329
-            } elseif (   $report_data['invoiceable']
329
+            } elseif ($report_data['invoiceable']
330 330
                       && ($report_data['isapproved'] || !$invoice_approved_only)) {
331 331
                 $hours['invoiceable'] += $report_hours;
332 332
             }
Please login to merge, or discard this patch.
lib/midcom/core/collector.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -134,7 +134,7 @@  discard block
 block discarded – undo
134 134
         $counter = 0;
135 135
 
136 136
         foreach ($result as $object_guid => $empty_copy) {
137
-            if (    $this->_user_id
137
+            if ($this->_user_id
138 138
                 && !midcom::get()->auth->acl->can_do_byguid('midgard:read', $object_guid, $classname, $this->_user_id)) {
139 139
                 debug_add("Failed to load result, read privilege on {$object_guid} not granted for the current user.", MIDCOM_LOG_INFO);
140 140
                 continue;
@@ -145,7 +145,7 @@  discard block
 block discarded – undo
145 145
                 if ($counter <= $this->_offset) {
146 146
                     continue;
147 147
                 }
148
-                if (   $this->_limit
148
+                if ($this->_limit
149 149
                     && $counter > ($this->_offset + $this->_limit)) {
150 150
                     break;
151 151
                 }
@@ -223,7 +223,7 @@  discard block
 block discarded – undo
223 223
 
224 224
     public function get_subkey($key, $property)
225 225
     {
226
-        if (   $this->_user_id
226
+        if ($this->_user_id
227 227
             && !midcom::get()->auth->acl->can_do_byguid('midgard:read', $key, $this->_real_class, $this->_user_id)) {
228 228
             midcom_connection::set_error(MGD_ERR_ACCESS_DENIED);
229 229
             return false;
@@ -233,7 +233,7 @@  discard block
 block discarded – undo
233 233
 
234 234
     public function get($key)
235 235
     {
236
-        if (   $this->_user_id
236
+        if ($this->_user_id
237 237
             && !midcom::get()->auth->acl->can_do_byguid('midgard:read', $key, $this->_real_class, $this->_user_id)) {
238 238
             midcom_connection::set_error(MGD_ERR_ACCESS_DENIED);
239 239
             return false;
Please login to merge, or discard this patch.
lib/net/nehmer/static/midcom/interfaces.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -68,7 +68,7 @@
 block discarded – undo
68 68
         }
69 69
 
70 70
         $config = $this->get_config_for_topic($topic);
71
-        if (   $object->name == 'index'
71
+        if ($object->name == 'index'
72 72
             && !$config->get('autoindex')) {
73 73
             return '';
74 74
         }
Please login to merge, or discard this patch.