Skip to content

Commit 77e6b8e

Browse files
committed
Update code and tests
1 parent 3bf38fd commit 77e6b8e

3 files changed

Lines changed: 11 additions & 20 deletions

File tree

migrations/v10x/m1_initial_data.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ class m1_initial_data extends \phpbb\db\migration\migration
2222
* @static
2323
* @access public
2424
*/
25-
static public function depends_on()
25+
public static function depends_on()
2626
{
2727
return array('\phpbb\db\migration\data\v310\gold');
2828
}

migrations/v10x/m2_anonymize_ip.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ class m2_anonymize_ip extends \phpbb\db\migration\migration
1818
/**
1919
* {@inheritdoc}
2020
*/
21-
static public function depends_on()
21+
public static function depends_on()
2222
{
2323
return array('\phpbb\googleanalytics\migrations\v10x\m1_initial_data');
2424
}

tests/event/listener_test.php

Lines changed: 9 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ class listener_test extends \phpbb_test_case
2020
/** @var \phpbb\config\config */
2121
protected $config;
2222

23-
/** @var \PHPUnit_Framework_MockObject_MockObject|\phpbb\template\template */
23+
/** @var \PHPUnit\Framework\MockObject\MockObject|\phpbb\template\template */
2424
protected $template;
2525

2626
/** @var \phpbb\user */
@@ -98,9 +98,9 @@ public function test_load_google_analytics()
9898
'S_ANONYMIZE_IP' => $this->config['ga_anonymize_ip'],
9999
));
100100

101-
$dispatcher = new \Symfony\Component\EventDispatcher\EventDispatcher();
101+
$dispatcher = new \phpbb\event\dispatcher();
102102
$dispatcher->addListener('core.page_header', array($this->listener, 'load_google_analytics'));
103-
$dispatcher->dispatch('core.page_header');
103+
$dispatcher->trigger_event('core.page_header');
104104
}
105105

106106
/**
@@ -143,19 +143,12 @@ public function test_add_googleanalytics_configs($mode, $display_vars, $expected
143143
{
144144
$this->set_listener();
145145

146-
$dispatcher = new \Symfony\Component\EventDispatcher\EventDispatcher();
146+
$dispatcher = new \phpbb\event\dispatcher();
147147
$dispatcher->addListener('core.acp_board_config_edit_add', array($this->listener, 'add_googleanalytics_configs'));
148148

149149
$event_data = array('display_vars', 'mode');
150-
$event = new \phpbb\event\data(compact($event_data));
151-
$dispatcher->dispatch('core.acp_board_config_edit_add', $event);
152-
153-
$event_data_after = $event->get_data_filtered($event_data);
154-
foreach ($event_data as $expected)
155-
{
156-
self::assertArrayHasKey($expected, $event_data_after);
157-
}
158-
extract($event_data_after);
150+
$event_data_after = $dispatcher->trigger_event('core.acp_board_config_edit_add', compact($event_data));
151+
extract($event_data_after, EXTR_OVERWRITE);
159152

160153
$keys = array_keys($display_vars['vars']);
161154

@@ -236,19 +229,17 @@ public function test_validate_googleanalytics_id($cfg_array, $expected_error)
236229
$config_definition = array('validate' => 'googleanalytics_id');
237230
$error = array();
238231

239-
$dispatcher = new \Symfony\Component\EventDispatcher\EventDispatcher();
232+
$dispatcher = new \phpbb\event\dispatcher();
240233
$dispatcher->addListener('core.validate_config_variable', array($this->listener, 'validate_googleanalytics_id'));
241234

242235
$event_data = array('cfg_array', 'config_name', 'config_definition', 'error');
243-
$event = new \phpbb\event\data(compact($event_data));
244-
$dispatcher->dispatch('core.validate_config_variable', $event);
236+
$event_data_after = $dispatcher->trigger_event('core.validate_config_variable', compact($event_data));
245237

246-
$event_data_after = $event->get_data_filtered($event_data);
247238
foreach ($event_data as $expected)
248239
{
249240
self::assertArrayHasKey($expected, $event_data_after);
250241
}
251-
extract($event_data_after);
242+
extract($event_data_after, EXTR_OVERWRITE);
252243

253244
self::assertEquals($expected_error, $error);
254245
}

0 commit comments

Comments
 (0)