Issue Details (XML | Word | Printable)

Key: CONTRIB-1359
Type: Improvement Improvement
Status: Resolved Resolved
Resolution: Fixed
Priority: Minor Minor
Assignee: Mike Worth
Reporter: Mark Johnson
Votes: 0
Watchers: 1
Operations

Add/Edit UI Mockup to this issue
If you were logged in you would be able to see more operations.
Non-core contributed modules

Prevent clash notification emails sending multiple times

Created: 19/Jun/09 06:17 PM   Updated: 19/Jun/09 06:26 PM
Return to search
Component/s: Block: Mrbs
Affects Version/s: 1.9.4
Fix Version/s: None

File Attachments: 1. Text File mrbs_importemails.patch (12 kB)

Issue Links:
Relates
 

Participants: Mark Johnson and Mike Worth
Security Level: None
Resolved date: 19/Jun/09
Affected Branches: MOODLE_19_STABLE


 Description  « Hide
This changes the import script to check if a booking was already there yesterday. If it was and it clashes with another booking, no clash notification e-mail is sent as one would already have been sent the first time the import script detected this clash.
Also improves performance not deleting and re-inserting records that haven't changed.

 All   Comments   Change History   Version Control      Sort Order: Ascending order - Click to sort in descending order
Mark Johnson made changes - 19/Jun/09 06:19 PM
Field Original Value New Value
Link This issue has a non-specific relationship to CONTRIB-1189 [ CONTRIB-1189 ]
Mike Worth made changes - 19/Jun/09 06:20 PM
Assignee Anthony Borrow [ aborrow ] Mike Worth [ mike1989 ]
Mike Worth committed 3 files to 'Contrib CVS' - 19/Jun/09 06:25 PM
CONTRIB-1359 Import script changed to recognise repeat clashes and only send emails once. Also improves performance
MODIFY plugins/blocks/mrbs/web/config.inc.php   Rev. 1.17    (+2 -1 lines)
MODIFY plugins/blocks/mrbs/lang/en_utf8/block_mrbs.php   Rev. 1.23    (+2 -2 lines)
MODIFY plugins/blocks/mrbs/import.php   Rev. 1.4    (+37 -16 lines)
Mike Worth added a comment - 19/Jun/09 06:26 PM
Committed to CVS

Mike Worth made changes - 19/Jun/09 06:26 PM
Status Open [ 1 ] Resolved [ 5 ]
Resolution Fixed [ 1 ]