View Issue Details

IDProjectCategoryView StatusLast Update
0004043mantisbtemailpublic2004-08-29 02:01
Reportersmhanson Assigned Tobpfennig  
PrioritynormalSeverityfeatureReproducibilityhave not tried
Status closedResolutionfixed 
Fixed in Version0.19.0rc1 
Summary0004043: Preference to exclude old bugnotes from notification
Description

A user has requested not include all previous bugnotes in his notifications, since he doesn't like scrolling when an issue has many comments. This might be a good user preference.

TagsNo tags attached.

Relationships

has duplicate 0003044 closedvboctor option to leave out previous bugnotes 
related to 0004278 new Summary email vs detailed email 

Activities

sgrund

sgrund

2004-07-09 01:04

reporter   ~0005949

My User like this idea too. As 'workaround' change ordering of bugnotes.
($g_bugnote_order = "DESC";)

vboctor

vboctor

2004-07-11 03:01

manager   ~0005982

I don't agree with excluding old notes. However, I will leave this open so that in the future the ordering can be a user preference rather than a general configuration.

sgrund

sgrund

2004-07-12 01:06

reporter   ~0006002

Wouldn't it be OK to customize the number of bugnotes? Something like n'th newest.
We have long tickets with 20+ notes, so the mail is really long.
I would prefer both: Ordering and Number.

goemon1

goemon1

2004-07-15 15:46

reporter   ~0006069

we have tickets with enormously long notes.

there really should be a user option to limit to the last N notes of bugs, it's really annoying to get emails over and over with eg 40 notes, each of the notes being several pages long.

bpfennig

bpfennig

2004-08-07 16:17

reporter   ~0006779

I've created a patch for this, it does the following things:

  1. Let the users define their own bugnote order in preferences.
  2. Let the users define the limit of bugnotes in email.

I will send the patch to the Mantis team for the next release.

jlatour

jlatour

2004-08-07 17:23

reporter   ~0006782

Thank you, Bastian.

The coding guidelines apparantly aren't all that clear on this, but the line

if ($t_email_bugnote_limit == 0)
{

should be formatted as

if ( 0 == $t_email_bugnote_limit ) {

(just for future reference - the patch is small enough that I can clean this up myself)

I'll try to commit the functionality tomorrow.

bpfennig

bpfennig

2004-08-08 02:38

reporter   ~0006792

:)

Ok, I will take a look at the coding guidelines next time!

jlatour

jlatour

2004-08-08 06:32

reporter   ~0006799

OK, I've applied the patch.

Changes:

  • I think you misreferenced a preference field somewhere
  • Formatting :-)
  • Created SQL upgrade scripts
  • Added a default value to that query that adds bugnote_order, because existing users were getting two unselected radio boxes. The default value is set to the configured system default.

I haven't tested the mail, so I hope you did :-)