MDL-84632 mod_resource: deprecate index.php related event

This commit is contained in:
ferran
2025-08-25 15:03:24 +02:00
parent b82b297954
commit df82be756e
3 changed files with 24 additions and 35 deletions
@@ -0,0 +1,7 @@
issueNumber: MDL-84632
notes:
mod_resource:
- message: >-
The \mod_resource\event\course_module_instance_list_viewed event is now
deprecated. Use \core\event\course_resources_list_viewed instead.
type: deprecated
@@ -14,14 +14,6 @@
// You should have received a copy of the GNU General Public License
// along with Moodle. If not, see <http://www.gnu.org/licenses/>.
/**
* The mod_resource instance list viewed event.
*
* @package mod_resource
* @copyright 2014 Rajesh Taneja <[email protected]>
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
*/
namespace mod_resource\event;
defined('MOODLE_INTERNAL') || die();
@@ -33,7 +25,23 @@ defined('MOODLE_INTERNAL') || die();
* @since Moodle 2.7
* @copyright 2014 Rajesh Taneja <[email protected]>
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
* @deprecated This event is deprecated and will be removed in Moodle 6.0.
* @todo Remove class in 6.0 (MDL-86384)
*/
#[\core\attribute\deprecated(
replacement: core\event\course_resources_list_viewed::class,
since: '5.1',
mdl: 'MDL-84632',
)]
class course_module_instance_list_viewed extends \core\event\course_module_instance_list_viewed {
// No need for any code here as everything is handled by the parent class.
#[\Override]
#[\core\attribute\deprecated(
replacement: core\event\course_resources_list_viewed::class,
since: '5.1',
mdl: 'MDL-84632',
)]
protected function init() {
\core\deprecation::emit_deprecation([self::class, __FUNCTION__]);
parent::init();
}
}
@@ -44,32 +44,6 @@ final class events_test extends \advanced_testcase {
$this->setAdminUser();
}
/**
* Test course_module_instance_list_viewed event.
*/
public function test_course_module_instance_list_viewed(): void {
// There is no proper API to call to trigger this event, so what we are
// doing here is simply making sure that the events returns the right information.
$course = $this->getDataGenerator()->create_course();
$params = array(
'context' => \context_course::instance($course->id)
);
$event = \mod_resource\event\course_module_instance_list_viewed::create($params);
// Triggering and capturing the event.
$sink = $this->redirectEvents();
$event->trigger();
$events = $sink->get_events();
$this->assertCount(1, $events);
$event = reset($events);
// Checking that the event contains the expected values.
$this->assertInstanceOf('\mod_resource\event\course_module_instance_list_viewed', $event);
$this->assertEquals(\context_course::instance($course->id), $event->get_context());
$this->assertEventContextNotUsed($event);
}
/**
* Test course_module_viewed event.
*/