Issue Details (XML | Word | Printable)

Key: MDL-17442
Type: Sub-task Sub-task
Status: In Progress In Progress
Priority: Minor Minor
Assignee: Penny Leach
Reporter: Penny Leach
Votes: 3
Watchers: 5
Operations

Add/Edit UI Mockup to this issue
If you were logged in you would be able to see more operations.
Moodle
MDL-17376

LOCAL: event subscriptions

Created: 28/Nov/08 10:56 PM   Updated: 05/Mar/10 01:06 AM
Component/s: Events API
Affects Version/s: 1.9.3, 2.0
Fix Version/s: None

File Attachments: 1. Text File 0013--UPSTREAM-add-support-for-local-event-handlers.patch (0.7 kB)
2. Text File 0017--UPSTREAM-run-the-event-queue-in-cron.patch (0.7 kB)


Participants: Eloy Lafuente (stronk7), Penny Leach and Petr Škoda (skodak)
Security Level: None
Affected Branches: MOODLE_19_STABLE, MOODLE_20_STABLE


 Description  « Hide
Got a patch, just need to review, commit & document.

 All   Comments   Change History   Version Control      Sort Order: Ascending order - Click to sort in descending order
Penny Leach added a comment - 19/Dec/08 12:05 AM
Petr - I think this one could go in stable, what do you think ?

Attaching 2 patches.


Penny Leach added a comment - 19/Dec/08 12:08 AM
two patches:

1. install event definitions in lib/locallib
2. run event queue in cron (hard to believe that isn't there!)


Eloy Lafuente (stronk7) added a comment - 24/Dec/08 10:51 PM
Looks ok for me (strange event queue wasn't in cron)... uhm..

Penny Leach added a comment - 05/Mar/10 12:06 AM
ok to go in stable ?

petr I guess you've changed this in HEAD anyway with your local plugin stuff but I'd like to put it in 1.9


Petr Škoda (skodak) added a comment - 05/Mar/10 12:16 AM
looks ok to me for 1.9.x,
the HEAD has already different plugin infrastructure, please just make sure the events are processed in cron for all plugins in local

Penny Leach added a comment - 05/Mar/10 01:06 AM
okie, committed to 1.9. will test head in a bit.