Notify time is universal, not by course and activity

[Urpokarhu1]

Hello,

I have noticed a bug, notify time is universal for whole system and activity. It should be per activity, not per system.

[danmarsden]

this could be related to #149 - but we really need a bit more info to be able to reproduce this ... It would be good if you could try to reproduce with the latest code now that #149 has landed.

[Urpokarhu1]

Hello,

Is tested again with version mod_reengagement 2023020804 and problem persists.

1. Create course1 and course2
2. Create re-engagement on course1 with settings:

• notify user: after delay 5 minutes
• reminder count: 1
• activity completion
• Reengagement duration: 20 minutes.

3. Notice the date and time in "notify time"-column. for example 24/04/24, 10:15

4. Create re-engagement on course2 with settings:

• notify user: after delay 15 minutes
• reminder count: 1
• activity completion
• Reengagement duration: 20 minutes.

5. Notice how "notify time" is identical to a course1 re-engagement-activity for example 24/04/24, 10:15 and not what it should be according to re-engagement-activity settings on this course.

course1

course2

[Urpokarhu1]

You should also check #151