@@ -23,7 +23,8 @@ discard block |
||
23 | 23 | try { |
24 | 24 | $person = new midcom_db_person($this->uid); |
25 | 25 | $grp = new midcom_db_group($this->gid); |
26 | - } catch (midcom_error $e) { |
|
26 | + } |
|
27 | + catch (midcom_error $e) { |
|
27 | 28 | $e->log(); |
28 | 29 | return 'Invalid membership record'; |
29 | 30 | } |
@@ -40,7 +41,8 @@ discard block |
||
40 | 41 | } |
41 | 42 | try { |
42 | 43 | $person = new midcom_db_person($this->uid); |
43 | - } catch (midcom_error $e) { |
|
44 | + } |
|
45 | + catch (midcom_error $e) { |
|
44 | 46 | return; |
45 | 47 | } |
46 | 48 | midcom::get()->cache->invalidate($person->guid); |
@@ -37,7 +37,7 @@ |
||
37 | 37 | { |
38 | 38 | return preg_replace_callback( |
39 | 39 | "%(<\s*)+(/\s*)+{$element}%i", |
40 | - function ($matches) { |
|
40 | + function($matches) { |
|
41 | 41 | return htmlentities($matches[0]); |
42 | 42 | }, |
43 | 43 | $input |
@@ -84,7 +84,7 @@ |
||
84 | 84 | $variable = $parts[0] . '->' . $parts[1]; |
85 | 85 | } |
86 | 86 | |
87 | - if ( isset($variable_parts[1]) |
|
87 | + if (isset($variable_parts[1]) |
|
88 | 88 | && array_key_exists($variable_parts[1], self::$_filters)) { |
89 | 89 | switch ($variable_parts[1]) { |
90 | 90 | case 's': |
@@ -88,7 +88,8 @@ |
||
88 | 88 | $style = midcom_db_style::get_cached($style_id); |
89 | 89 | $styleeditor_url = midcom_connection::get_url('self') . "__mfa/asgard/object/view/{$style->guid}/"; |
90 | 90 | $enabled = true; |
91 | - } catch (midcom_error $e) { |
|
91 | + } |
|
92 | + catch (midcom_error $e) { |
|
92 | 93 | $e->log(); |
93 | 94 | } |
94 | 95 | } |
@@ -56,7 +56,7 @@ discard block |
||
56 | 56 | } |
57 | 57 | $buttons = []; |
58 | 58 | $workflow = new midcom\workflow\datamanager; |
59 | - if ( $this->topic->can_do('midgard:update') |
|
59 | + if ($this->topic->can_do('midgard:update') |
|
60 | 60 | && $this->topic->can_do('midcom.admin.folder:topic_management')) { |
61 | 61 | $buttons[] = $workflow->get_button("__ais/folder/edit/", [ |
62 | 62 | MIDCOM_TOOLBAR_LABEL => midcom::get()->i18n->get_string('edit folder', 'midcom.admin.folder'), |
@@ -68,7 +68,7 @@ discard block |
||
68 | 68 | ]); |
69 | 69 | } |
70 | 70 | |
71 | - if ( $urltopic->can_do('midgard:update') |
|
71 | + if ($urltopic->can_do('midgard:update') |
|
72 | 72 | && $urltopic->can_do('midcom.admin.folder:topic_management')) { |
73 | 73 | // Allow to move other than root folder |
74 | 74 | if ($urltopic->guid !== midcom::get()->config->get('midcom_root_topic_guid')) { |
@@ -80,7 +80,7 @@ discard block |
||
80 | 80 | } |
81 | 81 | } |
82 | 82 | |
83 | - if ( $this->topic->can_do('midgard:update') |
|
83 | + if ($this->topic->can_do('midgard:update') |
|
84 | 84 | && $this->topic->can_do('midcom.admin.folder:topic_management')) { |
85 | 85 | $viewer = new midcom\workflow\viewer; |
86 | 86 | $buttons[] = $viewer->get_button("__ais/folder/order/", [ |
@@ -97,7 +97,7 @@ discard block |
||
97 | 97 | ]; |
98 | 98 | } |
99 | 99 | $buttons = array_merge($buttons, $this->get_approval_controls($this->topic, false)); |
100 | - if ( $this->topic->can_do('midcom.admin.folder:template_management') |
|
100 | + if ($this->topic->can_do('midcom.admin.folder:template_management') |
|
101 | 101 | && midcom::get()->auth->can_user_do('midgard.admin.asgard:manage_objects', null, 'midgard_admin_asgard_plugin')) { |
102 | 102 | $enabled = false; |
103 | 103 | $styleeditor_url = ''; |
@@ -122,7 +122,7 @@ discard block |
||
122 | 122 | ]; |
123 | 123 | } |
124 | 124 | |
125 | - if ( $this->topic->can_do('midgard:create') |
|
125 | + if ($this->topic->can_do('midgard:create') |
|
126 | 126 | && $this->topic->can_do('midcom.admin.folder:topic_management')) { |
127 | 127 | $buttons[] = $workflow->get_button("__ais/folder/create/", [ |
128 | 128 | MIDCOM_TOOLBAR_LABEL => midcom::get()->i18n->get_string('create subfolder', 'midcom.admin.folder'), |
@@ -130,7 +130,7 @@ discard block |
||
130 | 130 | MIDCOM_TOOLBAR_ACCESSKEY => 'f', |
131 | 131 | ]); |
132 | 132 | } |
133 | - if ( $urltopic->guid !== midcom::get()->config->get('midcom_root_topic_guid') |
|
133 | + if ($urltopic->guid !== midcom::get()->config->get('midcom_root_topic_guid') |
|
134 | 134 | && $urltopic->can_do('midgard:delete') |
135 | 135 | && $urltopic->can_do('midcom.admin.folder:topic_management')) { |
136 | 136 | $workflow = new midcom\workflow\delete(['object' => $urltopic, 'recursive' => true]); |
@@ -4,7 +4,8 @@ |
||
4 | 4 | try { |
5 | 5 | $person = new midcom_db_person($metadata->locker); |
6 | 6 | $name = $person->name; |
7 | -} catch (midcom_error $e) { |
|
7 | +} |
|
8 | +catch (midcom_error $e) { |
|
8 | 9 | $name = $this->data['handler']->_l10n_midcom->get('unknown user'); |
9 | 10 | $e->log(); |
10 | 11 | } |
@@ -16,7 +16,8 @@ |
||
16 | 16 | foreach ($atts as $att) { |
17 | 17 | try { |
18 | 18 | $att->file_to_cache(); |
19 | - } catch (midcom_error $e) { |
|
19 | + } |
|
20 | + catch (midcom_error $e) { |
|
20 | 21 | echo 'Error: ' . $e->getMessage() . "\n"; |
21 | 22 | } |
22 | 23 | } |
@@ -55,7 +55,7 @@ |
||
55 | 55 | |
56 | 56 | $http_client = new org_openpsa_httplib(); |
57 | 57 | $http_client->set_param('timeout', 300); |
58 | -if ( !empty($_SERVER['PHP_AUTH_USER']) |
|
58 | +if (!empty($_SERVER['PHP_AUTH_USER']) |
|
59 | 59 | && !empty($_SERVER['PHP_AUTH_PW'])) { |
60 | 60 | $http_client->basicauth['user'] = $_SERVER['PHP_AUTH_USER']; |
61 | 61 | $http_client->basicauth['password'] = $_SERVER['PHP_AUTH_PW']; |
@@ -463,7 +463,7 @@ discard block |
||
463 | 463 | { |
464 | 464 | // Complete fields |
465 | 465 | $this->indexed = time(); |
466 | - if ( $this->author == '' |
|
466 | + if ($this->author == '' |
|
467 | 467 | && isset($this->creator->name)) { |
468 | 468 | $this->author = $this->creator->name; |
469 | 469 | } |
@@ -606,7 +606,7 @@ discard block |
||
606 | 606 | public function read_metadata_from_object($object) |
607 | 607 | { |
608 | 608 | // Published is set to non-empty value, use it as creation data |
609 | - if ( !empty($object->metadata->published) |
|
609 | + if (!empty($object->metadata->published) |
|
610 | 610 | && !preg_match('/0{1,4}-0{1,2}0{1,2}\s+0{1,2}:0{1,2}:0{1,2}/', $object->metadata->published)) { |
611 | 611 | $this->created = $this->read_unixtime($object->metadata->published); |
612 | 612 | } elseif (isset($object->metadata->created)) { |
@@ -645,7 +645,7 @@ discard block |
||
645 | 645 | return @strtotime($stamp); |
646 | 646 | } |
647 | 647 | // Unix timestamp |
648 | - return (int)$stamp; |
|
648 | + return (int) $stamp; |
|
649 | 649 | } |
650 | 650 | |
651 | 651 | /** |
@@ -680,7 +680,8 @@ |
||
680 | 680 | { |
681 | 681 | try { |
682 | 682 | return midcom_db_person::get_cached($id); |
683 | - } catch (midcom_error $e) { |
|
683 | + } |
|
684 | + catch (midcom_error $e) { |
|
684 | 685 | return false; |
685 | 686 | } |
686 | 687 | } |
@@ -63,7 +63,8 @@ |
||
63 | 63 | |
64 | 64 | try { |
65 | 65 | $attachment = new midcom_db_attachment($guid); |
66 | - } catch (midcom_error $e) { |
|
66 | + } |
|
67 | + catch (midcom_error $e) { |
|
67 | 68 | $e->log(); |
68 | 69 | return "<span class=\"missing_photo\" title=\"{$guid}\">{$fulltag}</span>{$after}"; |
69 | 70 | } |
@@ -223,7 +223,7 @@ discard block |
||
223 | 223 | return $this->render_link($parts[1], $parts[2]) . $after; |
224 | 224 | |
225 | 225 | // Macro [something: <data>] (for example [abbr: BOFH - Bastard Operator From Hell] or [photo: <GUID>]) |
226 | - case ( preg_match('/^(.*?): (.*)/', $fulltext, $macro_parts) |
|
226 | + case (preg_match('/^(.*?): (.*)/', $fulltext, $macro_parts) |
|
227 | 227 | && method_exists($this, "_run_macro_{$macro_parts[1]}")): |
228 | 228 | $method = "_run_macro_{$macro_parts[1]}"; |
229 | 229 | return $this->$method($macro_parts[2], $match[0], $after); |
@@ -256,7 +256,7 @@ discard block |
||
256 | 256 | $folder = $wikipage_match['latest_parent']; |
257 | 257 | } |
258 | 258 | |
259 | - if ( isset($folder[MIDCOM_NAV_OBJECT]) |
|
259 | + if (isset($folder[MIDCOM_NAV_OBJECT]) |
|
260 | 260 | && $folder[MIDCOM_NAV_OBJECT]->can_do('midgard:create')) { |
261 | 261 | $workflow = $this->get_workflow('datamanager'); |
262 | 262 | return "<a href=\"{$folder[MIDCOM_NAV_ABSOLUTEURL]}create/?wikiword={$wikipage_match['remaining_path']}\" " . $workflow->render_attributes() . " class=\"wiki_missing\" title=\"" . $this->_l10n->get('click to create') . "\">{$text}</a>"; |
@@ -291,7 +291,7 @@ discard block |
||
291 | 291 | $links[$parts[1]] = $parts[2]; |
292 | 292 | } |
293 | 293 | // Ignore macros [something: <data>] (for example [abbr: BOFH - Bastard Operator From Hell] or [photo: <GUID>]) |
294 | - elseif ( !preg_match('/^(.*?): (.*)/', $fulltext, $macro_parts) |
|
294 | + elseif (!preg_match('/^(.*?): (.*)/', $fulltext, $macro_parts) |
|
295 | 295 | || !method_exists($this, "_run_macro_{$macro_parts[1]}")) { |
296 | 296 | // MediaWiki-style link [wikipage] (no text) |
297 | 297 | $links[$fulltext] = $fulltext; |