#65: All modules activated in the main.ini are active for each single project.
Issue attachments
New_method_activateModule_added_that_has_the_behavior_of_registerModule_registerModule_is_now_used_to_register_all_modules_configured_in_the_main_ini.patch
Issue revisions
- new by Manuel Pichler at 2009-F-26 20:07
- new by Manuel Pichler at 2009-F-26 20:07
- assigned by Manuel Pichler at 2009-F-27 18:08
- assigned by Kore Nordmann at 2009-F-28 16:08
- assigned by Manuel Pichler at 2009-F-28 17:03
- assigned by Manuel Pichler at 2009-F-28 17:04
- closed by Manuel Pichler at 2009-F-28 17:05
| Type |  bug |
|---|---|
| State |  closed |
| Priority |  low |
| Resolution |  fixed |
| Assigned to | Manuel Pichler |
| Scheduled for | 0.2-alpha |
| Affected versions | 0.1-alpha, Latest SVN |
| Affected components | Core |
| Last change | Saturday 28 February 2009 17:05:25 UTC by Manuel Pichler |
Short description
All modules activated in the main.ini are active signal-slot listeners for all projects, even if the project.ini does not include that module. This is caused by the registerModule() call in the index.php file, where all allowed modules are registered. IMHO arbit should call something like addAllowedModule() here and then the arbitProjectController should register all modules for the current project.
Steps to reproduce
Activate some notifications, e.g. coreUserLogin for your project user account, now deactivate the notification module. Logout and then login, you will get a login notification.
Expected behavior
No login notification.
Actual behavior
No notification module === no login notification.
Kore Nordmann at Saturday 28 February 2009 16:07:46 UTC
Patch looks good, go ahead.
Kore Nordmann at Saturday 28 February 2009 16:08:08 UTC
Scheduled for 0.2-alpha.
Manuel Pichler at Saturday 28 February 2009 17:03:49 UTC
Fixed in svn revision #1022.
Manuel Pichler at Saturday 28 February 2009 17:04:22 UTC
Fixed, missing state change :-/
Manuel Pichler at Saturday 28 February 2009 17:05:25 UTC
Sorry, last missing issue field changed.