-
Bug
-
Resolution: Not a bug
-
Minor
-
None
-
5.0
-
None
-
MOODLE_500_STABLE
Moodle is designed to work with other open source plugins. Many of these plugins will extend moodle core classes.
This has broken php unit tests in the Snap theme because it utilises a class which extends mod_assign\base_test
1) A class that is named "base_" implies that it is something that *should be extended.
2) There's no benefit to making classes final in open-source projects. It will just prevent third-party developers from being able to fully exploit the benefits of core code. To fix the issue with the Snap theme (I'm not a maintainer, BTW) I would have to create a copy of all that core code to get the PHP unit tests to pass.
Please, let's keep core code as reusable as possible unless there are some definite reasons for needing to use the final class. Personally I can't think of any in an open source project.
- has a non-specific relationship to
-
MDL-81520 Update existing unit tests to be final
-
- Closed
-