Lukas W.
Well-known member
- Affected version
- 2.0.0 RC 3
Just a bunch of non-critical reports/suggestions:
- For event app_pub_render_page, $renderer would probably be better suited as name for the renderer object in the arguments list.
- For event bb_code_rules, $subContext would probably better be suited as name for the third argument.
- For dispatcher_match there's a discrepancy between signature and argument naming. Seems okay, but maybe want to keep this consistent.
- For dispatcher_post_render, the same problem as for app_pub_renderer_page.
- For entity_defaults, isn't $em your abbreviation for 'entity manager' and not an entity?
- For inline_mod_actions there is an inconsistent naming for the app object.
- For app_admin_render_page, the same as for app_pub_render_page
