View Issue Details

IDProjectCategoryView StatusLast Update
0003766mantisbtemailpublic2004-08-29 01:39
Reporterpermutations Assigned Tothraxisp  
PrioritynormalSeverityblockReproducibilityalways
Status closedResolutionfixed 
Fixed in Version0.19.0rc1 
Summary0003766: status notifications are confused and confusing
Description

There is no complete and accurate list of what events trigger notifications. The documentation is contradictory and confusing. For example, is 'close' an event or a status (represented as 'close' or 'status_close')? It's shown BOTH ways in the config file comments.

This confusion extends into how the program works, causing bugs. For example, I set the threshold for closing projects to "manager", and this removes the "Close" button from bug reports when you're logged in as a developer. But developers can still close projects. How? A 'close' option is also in the Status pulldown menu. This should not be there.

The Status pulldown menu also contains a "resolved" option. Why is this there, where there is a Resolved button that triggers a nice form where you can enter final comments?

The impression I have is that 12 differnt people worked on the status and event notification piece without ever talking to each other, so it's implemented in multiple ways in multiple places, and you can turn it off in one place while it's still on in another.

If someone could go through and clean this up, it would be great, because the notification feature is a very important part of the program.

Thank you.

TagsNo tags attached.

Relationships

related to 0003772 closedthraxisp Status updating should be different from bug updating 
has duplicate 0003458 closedvboctor email notification notification 
related to 0003447 closedjlatour New bug entries are not emailed to non-admin users 
related to 0003496 closedjlatour Reported not receiving "Email on New " 
related to 0003474 closedjlatour $g_notify_flags['new'] settings ignored 
child of 0003987 closedvboctor Mantis 0.19.0 Release 

Activities

permutations

permutations

2004-04-26 01:36

reporter   ~0005434

Last edited: 2004-04-26 10:07

Since no one volunteered to explain the notification rules, I had to read the source code to find out since the redundant notifications were driving everyone crazy. I'll post my findings for others. It does appear that two different individuals implemented notifications because they are in two separate modules: bug_api.php and email_api.php.

There are three notification categories in bug_api.php that are always sent out:

bug assignment change, status change, and updated (any field).

I have commented these out. Our projects are one-person projects so assignments don't change.

The status change and update notifications are duplicate, and also usually occur when someone

reporting a bug forgets to make a selection. The developer doesn't need to be bothered with

administrative fixes to the form; the notes are what matter.

These are the roles eligible for notification (from email_api.php):

#

'reporter': the reporter of the bug

'handler': the handler of the bug

'monitor': users who are monitoring a bug

'bugnotes': users who have added a bugnote to the bug

'threshold_max': all users with access <= max

'threshold_min': ..and with access >= min

These are the events eligible for notification (from email_api.php):

#

new

bugnote

resolved (was "status_resolved") -- to be fixed in 19.0

closed (was "status_closed") -- to be fixed in 19.0

reopened

assigned

deleted <-- This is the one item that is not on the Preferences menu

Also, a bug causes delete notifications to be sent to every project contributor

when the project is deleted, so I will disable deletes.

#

Note: the Status and Priority checkboxes in Preferences don't do anything.

$g_notify_flags['deleted']  = array('bugnotes'  => OFF,
                                    'monitor'   => OFF,
                                    'handler'   => OFF,
                                    'reporter'  => OFF);    

This override causes updaters (aka beta testers) to receive notifications of all new

bug reports and bugnotes, regardless of whether they are in one of the four notification roles.

In other words, it gives notifications for all projects. Developers still only receive notifications

for their own projects (since this is what they wanted).

$g_notify_flags['new']['threshold_min'] = UPDATER;
$g_notify_flags['new']['threshold_max'] = UPDATER;

$g_notify_flags['bugnote']['threshold_min'] = UPDATER;
$g_notify_flags['bugnote']['threshold_max'] = UPDATER;

#

Manager receives notifications using the $g_to_email address.

edited on: 04-26-04 10:07

jlatour

jlatour

2004-08-07 09:24

reporter   ~0006758

If this is still so unclear, the documentation will have to be improved on this. I've seen a lot of people have trouble with this.

thraxisp

thraxisp

2004-08-16 18:26

reporter   ~0007021

The items in the second and third paragraphs have been fixed with 0003772 (in CVS).

The documentation for email actions ($gnotify*flags) is being updated for the next release.

This leaves the control (on/off) of messages on assignment change and non-status update to be looked into.

thraxisp

thraxisp

2004-08-18 11:54

reporter   ~0007057

Last edited: 2004-08-22 08:21

Some cleanup is still needed. I will change the following:

1) There will be a separate entry for each status (corresponding to $g_status_enumstring, with spaces replaced by underscores). Emails will be sent on the status chenge event. "new" is a required status. The "status<state>" entries will be eliminated.

2) There will be an entry for "updated" used when the bug control information is changed (e.g, ETA, ...).

3) There will be an entry for "owner" used when the person the bug is assigned to changes. This replaces the current "assigned" which is also a status.

4) There will be a "sponsor" entry used for all sponsorship changes. This replaces the "sponsorship_added", "sponsorship_updated", and "sponsorship_deleted" entries (which are new).

5) Relationship changes will be sent using the "relation" action. In the case of a child bug being closed, the parent "handler" will also be notified.

edited on: 08-22-04 08:21

thraxisp

thraxisp

2004-08-23 09:45

reporter   ~0007116

fixes implemented in CVS