@@ -25,7 +25,7 @@ |
||
| 25 | 25 | public function configureOptions(OptionsResolver $resolver) |
| 26 | 26 | { |
| 27 | 27 | $resolver->setDefault('error_bubbling', false); |
| 28 | - $resolver->setNormalizer('type_config', function (Options $options, $value) { |
|
| 28 | + $resolver->setNormalizer('type_config', function(Options $options, $value) { |
|
| 29 | 29 | $type_defaults = [ |
| 30 | 30 | 'options' => [], |
| 31 | 31 | 'allow_other' => true, |
@@ -26,14 +26,14 @@ discard block |
||
| 26 | 26 | */ |
| 27 | 27 | public function configureOptions(OptionsResolver $resolver) |
| 28 | 28 | { |
| 29 | - $choice_loader = function (Options $options) { |
|
| 29 | + $choice_loader = function(Options $options) { |
|
| 30 | 30 | if (!empty($options['choices'])) { |
| 31 | 31 | return null; |
| 32 | 32 | } |
| 33 | 33 | return new loader($options['type_config']); |
| 34 | 34 | }; |
| 35 | 35 | |
| 36 | - $map_multiple = function (Options $options) { |
|
| 36 | + $map_multiple = function(Options $options) { |
|
| 37 | 37 | return !empty($options['type_config']['allow_multiple']); |
| 38 | 38 | }; |
| 39 | 39 | |
@@ -43,7 +43,7 @@ discard block |
||
| 43 | 43 | 'placeholder' => false |
| 44 | 44 | ]); |
| 45 | 45 | |
| 46 | - $resolver->setNormalizer('type_config', function (Options $options, $value) { |
|
| 46 | + $resolver->setNormalizer('type_config', function(Options $options, $value) { |
|
| 47 | 47 | $type_defaults = [ |
| 48 | 48 | 'options' => [], |
| 49 | 49 | 'option_callback' => null, |
@@ -9,7 +9,7 @@ |
||
| 9 | 9 | continue; |
| 10 | 10 | } |
| 11 | 11 | |
| 12 | - if ( !$object->can_do('midgard:update') |
|
| 12 | + if (!$object->can_do('midgard:update') |
|
| 13 | 13 | || !$object->can_do('midgard:attachments')) { |
| 14 | 14 | echo sprintf(midcom::get()->i18n->get_string('not recreating object %s %s, reason %s', 'midcom'), $type, $object->guid, 'Insufficient privileges') . "\n"; |
| 15 | 15 | continue; |
@@ -30,7 +30,7 @@ discard block |
||
| 30 | 30 | <?php |
| 31 | 31 | foreach ($type_choices as $value => $label) { |
| 32 | 32 | $selected = ''; |
| 33 | - if ( isset($data['type_filter']) |
|
| 33 | + if (isset($data['type_filter']) |
|
| 34 | 34 | && $data['type_filter'] == $value) { |
| 35 | 35 | $selected = ' selected="selected"'; |
| 36 | 36 | } |
@@ -45,7 +45,7 @@ discard block |
||
| 45 | 45 | <?php |
| 46 | 46 | foreach ($revised_after_choices as $value => $label) { |
| 47 | 47 | $selected = ''; |
| 48 | - if ( isset($data['revised_after']) |
|
| 48 | + if (isset($data['revised_after']) |
|
| 49 | 49 | && $data['revised_after'] == date('Y-m-d', $value)) { |
| 50 | 50 | $selected = ' selected="selected"'; |
| 51 | 51 | } |
@@ -18,7 +18,7 @@ |
||
| 18 | 18 | public function initialize(midcom_baseclasses_components_viewer $master, Router $router) |
| 19 | 19 | { |
| 20 | 20 | $this->router = $router; |
| 21 | - $this->_request_data =& $master->_request_data; |
|
| 21 | + $this->_request_data = & $master->_request_data; |
|
| 22 | 22 | $this->_topic = $master->_topic; |
| 23 | 23 | $this->_request_data['l10n'] = $this->_l10n; |
| 24 | 24 | |
@@ -134,7 +134,8 @@ |
||
| 134 | 134 | try { |
| 135 | 135 | $this->$function($request->request); |
| 136 | 136 | $this->_response->success = true; |
| 137 | - } catch (midcom_error $e) { |
|
| 137 | + } |
|
| 138 | + catch (midcom_error $e) { |
|
| 138 | 139 | $this->_response->success = false; |
| 139 | 140 | $this->_response->error = $e->getMessage(); |
| 140 | 141 | } |
@@ -61,7 +61,7 @@ discard block |
||
| 61 | 61 | $images = $qb->execute(); |
| 62 | 62 | $failed = 0; |
| 63 | 63 | foreach ($images as $image) { |
| 64 | - if ( !$image->generate_image('thumbnail', $this->_config->get('thumbnail_filter')) |
|
| 64 | + if (!$image->generate_image('thumbnail', $this->_config->get('thumbnail_filter')) |
|
| 65 | 65 | || !$image->generate_image('image', $this->_config->get('image_filter'))) { |
| 66 | 66 | $failed++; |
| 67 | 67 | } |
@@ -241,7 +241,7 @@ discard block |
||
| 241 | 241 | $attachment->title = $title; |
| 242 | 242 | $attachment->mimetype = $file['type']; |
| 243 | 243 | $attachment->parentguid = $image->guid; |
| 244 | - if ( !$attachment->create() |
|
| 244 | + if (!$attachment->create() |
|
| 245 | 245 | || !$attachment->copy_from_file($file['tmp_name'])) { |
| 246 | 246 | throw new midcom_error('Failed to create attachment: ' . midcom_connection::get_error_string()); |
| 247 | 247 | } |
@@ -138,7 +138,8 @@ |
||
| 138 | 138 | try { |
| 139 | 139 | $parent = new org_openpsa_projects_project($task->project); |
| 140 | 140 | $options['success_url'] = $this->router->generate('project', ['guid' => $parent->guid]); |
| 141 | - } catch (midcom_error $e) { |
|
| 141 | + } |
|
| 142 | + catch (midcom_error $e) { |
|
| 142 | 143 | $e->log(); |
| 143 | 144 | } |
| 144 | 145 | |
@@ -64,7 +64,7 @@ discard block |
||
| 64 | 64 | return new midcom_response_relocate($this->router->generate('task_view', ['guid' => $guid])); |
| 65 | 65 | } |
| 66 | 66 | |
| 67 | - if ( $request->request->has('save') |
|
| 67 | + if ($request->request->has('save') |
|
| 68 | 68 | && $prospects = $request->request->get('org_openpsa_projects_prospects')) { |
| 69 | 69 | $qb = org_openpsa_projects_task_resource_dba::new_query_builder(); |
| 70 | 70 | $qb->add_constraint('guid', 'IN', array_keys($prospects)); |
@@ -95,7 +95,7 @@ discard block |
||
| 95 | 95 | // create relatedto |
| 96 | 96 | org_openpsa_relatedto_plugin::create($event, 'org.openpsa.calendar', $this->_task, 'org.openpsa.projects'); |
| 97 | 97 | } |
| 98 | - if ( $update_prospect |
|
| 98 | + if ($update_prospect |
|
| 99 | 99 | && !$prospect->update()) { |
| 100 | 100 | debug_add('Failed to update prospect: ' . midcom_connection::get_error_string(), MIDCOM_LOG_ERROR); |
| 101 | 101 | } |
@@ -36,7 +36,8 @@ discard block |
||
| 36 | 36 | // Add person to campaign |
| 37 | 37 | try { |
| 38 | 38 | $campaign = new org_openpsa_directmarketing_campaign_dba($add_to); |
| 39 | - } catch (midcom_error $e) { |
|
| 39 | + } |
|
| 40 | + catch (midcom_error $e) { |
|
| 40 | 41 | // FIXME: More informative error message |
| 41 | 42 | $this->notify('Failed adding person %s to campaign %s', $add_to, 'error'); |
| 42 | 43 | return; |
@@ -86,7 +87,8 @@ discard block |
||
| 86 | 87 | try { |
| 87 | 88 | $campaigns[$membership->campaign] = new org_openpsa_directmarketing_campaign_dba($membership->campaign); |
| 88 | 89 | $campaign_membership_map[$membership->campaign] = $membership; |
| 89 | - } catch (midcom_error $e) { |
|
| 90 | + } |
|
| 91 | + catch (midcom_error $e) { |
|
| 90 | 92 | debug_add('Failed to load campaign ' . $membership->campaign . ', reason: ' . $e->getMessage()); |
| 91 | 93 | } |
| 92 | 94 | } |