Installation process bug -- Can enable "log time" before the module is installed

[carlcrott]

I'm not sure if this has been patched ... if not I could possibly contribute!

https://github.com/delaitre/redmine_time_tracker/#install

Essentially Step 4 can be run before the migrations are added ( thus causing errors in redmine causing the system to 500 ) ... I'm thinking that this would be a really simple fix to just test if the module is defined ... if so: display the option for "log time"

Also I'd like to add a note to the README to tell the user where these permissions can be activated.

[delaitre]

I don't see what is the benefit of doing the step 4 before the migration
and what is "the option for log time". Could you explain a bit more
precisely?

[carlcrott]

There is no advantage to doing step 4 before the migration ... =] it breaks things ... what I'm proposing is to verify that the migration has been run BEFORE the user can perform step 4.

sequentially (1..2..3..4..5..6) everything works ... but a user can activate step 4:
"Setup the ‘log time’ permissions for your roles"

before the migrations and thus they break their redmine installation