Skip to content

Commit 257654b

Browse files
committed
Fix observers
1 parent 659a824 commit 257654b

File tree

4 files changed

+14
-12
lines changed

4 files changed

+14
-12
lines changed

classes/local/util/manager.php

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1522,11 +1522,13 @@ protected function prepare_observers() {
15221522
list($observername, $callback) = explode('::', $observer['callback']);
15231523
}
15241524

1525-
if (strpos($observername, $this->recipe['component']) !== false) {
1526-
$observername = substr($observername, strlen($this->recipe['component'].'_'));
1525+
if ($isinsidenamespace !== false && (strpos($observername, $this->recipe['component']) !== false)) {
1526+
$observerfilename = substr($observername, strlen($this->recipe['component'].'_'));
1527+
} else {
1528+
$observerfilename = $observername;
15271529
}
15281530

1529-
$observerfile = 'classes/'.$observername.'.php';
1531+
$observerfile = 'classes/'.$observerfilename.'.php';
15301532

15311533
if (empty($this->files[$observerfile])) {
15321534
$this->prepare_file_skeleton($observerfile, 'observer_file', 'classes_observer', $observerrecipe);

tests/examples/auth.yaml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -94,17 +94,17 @@ cli_scripts:
9494

9595
observers:
9696
- eventname: \core\event\something_happened
97-
callback: \local_test\another\observer_one::something_happened
97+
callback: \auth_skeltest\another\observer_one::something_happened
9898
includefile: "/path/to/file/relative/to/moodle/dir/root"
9999
priority: 200
100100
internal: true
101101

102102
- eventname: \core\event\something_else_happened
103-
callback: local_test_locallib_function
103+
callback: auth_skeltest_locallib_function
104104
internal: false
105105

106106
- eventname: \core\event\something_else_happened
107-
callback: local_test_observer_two::do_something
107+
callback: auth_skeltest_observer_two::do_something
108108

109109
events:
110110
- eventname: first_event

tests/examples/format.yaml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -88,17 +88,17 @@ cli_scripts:
8888

8989
observers:
9090
- eventname: \core\event\something_happened
91-
callback: \local_test\another\observer_one::something_happened
91+
callback: \format_skeltest\another\observer_one::something_happened
9292
includefile: "/path/to/file/relative/to/moodle/dir/root"
9393
priority: 200
9494
internal: true
9595

9696
- eventname: \core\event\something_else_happened
97-
callback: local_test_locallib_function
97+
callback: format_skeltest_locallib_function
9898
internal: false
9999

100100
- eventname: \core\event\something_else_happened
101-
callback: local_test_observer_two::do_something
101+
callback: format_skeltest_observer_two::do_something
102102

103103
events:
104104
- eventname: first_event

tests/examples/mod.yaml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -97,17 +97,17 @@ cli_scripts:
9797

9898
observers:
9999
- eventname: \core\event\something_happened
100-
callback: \local_test\another\observer_one::something_happened
100+
callback: \mod_skeltest\another\observer_one::something_happened
101101
includefile: "/path/to/file/relative/to/moodle/dir/root"
102102
priority: 200
103103
internal: true
104104

105105
- eventname: \core\event\something_else_happened
106-
callback: local_test_locallib_function
106+
callback: skeltest_locallib_function
107107
internal: false
108108

109109
- eventname: \core\event\something_else_happened
110-
callback: local_test_observer_two::do_something
110+
callback: skeltest_observer_two::do_something
111111

112112
events:
113113
- eventname: course_module_viewed

0 commit comments

Comments
 (0)