| View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | ||||||||
| ID | Project | Category | View Status | Date Submitted | Last Update | ||||
|---|---|---|---|---|---|---|---|---|---|
| 0000642 | v2.3 Release (Closed) | [All Projects] General | public | 2010-11-25 00:52 | 2011-03-24 09:49 | ||||
| Reporter | paul_rogers6 | ||||||||
| Assigned To | caseydk | ||||||||
| Priority | normal | Severity | minor | Reproducibility | always | ||||
| Status | closed | Resolution | fixed | ||||||
| Product Version | |||||||||
| Target Version | Fixed in Version | 2.3 | |||||||
| Summary | 0000642: Reminders - multiple emails sent to one assignee | ||||||||
| Description | Where a task's owner is also an assignee the user in question receives two emails with the same reminder. | ||||||||
| Additional Information | This is because the code in the modules/tasks/tasks.class.php/remind() function is not correctly identifying if an assignee is also the owner. Line 2126 (v2.1.0) should be changed from: $contacts = $q->loadHashList('c.contact_id'); to: $contacts = $q->loadHashList('contact_id'); The amended file is attached (NB: Also contains updates for 0635). | ||||||||
| Tags | No tags attached. | ||||||||
| Attached Files |
| ||||||||
 Relationships |
|
 Relationships |
| Notes |
|
|
paul_rogers6 (reporter) 2010-11-25 01:08 |
NB: The statement: The amended file is attached (NB: Also contains updates for 0635). should read The amended file is attached (NB: Also contains updates for 0623). ie Updates also included for issue 623 NOT 635 Sorry P |
|
paul_rogers6 (reporter) 2010-11-25 04:05 Last edited: 2010-11-25 04:10 |
Have discovered another problem. As is remind function doesn't list owner as assignee if necessary. To fix this amend line 2177 (v2.1.0) from: if ($owner_is_not_assignee || $contact['contact_id'] != $owner_contact) { to: if (!$owner_is_not_assignee || ($owner_is_not_assignee && $contact['contact_id'] != $owner_contact)) { Have uploaded amended file modules/tasks/tasks.class.php (with changes for 0000623) NB: Uploaded as tasks.class.php.latest to avoid duplicate file error |
|
caseydk (administrator) 2011-03-22 23:51 |
Resolved in r1765 |
|
caseydk (administrator) 2011-03-24 09:49 |
Closed in preparation for v2.3 release. |
| Notes |
| Issue History |
|||
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2010-11-25 00:52 | paul_rogers6 | New Issue | |
| 2010-11-25 00:52 | paul_rogers6 | File Added: tasks.class.php | |
| 2010-11-25 01:08 | paul_rogers6 | Note Added: 0001363 | |
| 2010-11-25 04:05 | paul_rogers6 | Note Added: 0001368 | |
| 2010-11-25 04:10 | paul_rogers6 | Note Edited: 0001368 | |
| 2010-11-25 04:11 | paul_rogers6 | File Added: tasks.class.php.latest | |
| 2010-12-03 10:33 | caseydk | Project | v2.0 Release (Closed) => v2.2 Release (Closed) |
| 2010-12-15 21:54 | caseydk | Project | v2.2 Release (Closed) => v2.3 Release (Closed) |
| 2011-03-22 23:51 | caseydk | Note Added: 0001750 | |
| 2011-03-22 23:51 | caseydk | Status | new => resolved |
| 2011-03-22 23:51 | caseydk | Resolution | open => fixed |
| 2011-03-22 23:51 | caseydk | Assigned To | => caseydk |
| 2011-03-24 09:49 | caseydk | Note Added: 0001765 | |
| 2011-03-24 09:49 | caseydk | Status | resolved => closed |
| 2011-03-24 09:49 | caseydk | Fixed in Version | => 2.3 |
| Issue History |