MantisBT
Main | My View | View Issues | Change Log | Roadmap | Wiki | ManTweet | Repositories
  

View Issue Details [ Jump to Notes ] [ Wiki ] [ Related Changesets ] [ Issue History ] [ Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0008849mantisbttime trackingpublic2008-02-01 07:522008-08-11 09:41
Reporterkynx 
Assigned Todaryn 
PrioritynormalSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformLinuxOSRHELOS Version5
Product Version1.1.1 
Target VersionFixed in Version1.2.0a2 
Summary0008849: Emails ignore time tracking view threshold
DescriptionThe emailing script contains time tracking information, even if the user they are sent to is below the timetracking view threshold.

I notice that the check for the threshold is being made in bugnote_get_all_visible_bugnotes(), but then ignored email_format_bug_message(). The attached patch (against 1.1.1) corrects that.
TagsNo tags attached.
Attached Filesdiff file icon email_timetracking.diff [^] (904 bytes) 2008-02-01 07:52 [Show Content]

- Relationships
parent of 0009184closedvboctor Port 0008849: Emails ignore time tracking view threshold 

-  Notes
User avatar (0017639)
daryn (developer)
2008-04-21 10:55

 Looked over changes in patch. Looks good so applied to trunk.
User avatar (0017775)
lolotrav (reporter)
2008-05-05 09:25
edited on: 2008-05-05 09:25

 Hum, this patch does does not work here (mantis 1.1.1), and probably
also not in trunk.

In case user hasn't enough rights to see time tracking info, $t_bugnote->time_tracking is set to 0 (bugnote_api.php, L283)
thus the patch test is correct.

However, if the user has enough rights, the problem is that $t_bugnote->time_tracking is already formatted as hh:mm
(same file, L356).
Thus email contains only time tracking info if time is more than
1 hour, and is wrongly displayed as 00:hh...

I would suggest :

if ( $t_bugnote->time_tracking !== 0 && $t_bugnote->time_tracking != '00:00') {
    $t_time_tracking = ' ' . lang_get( 'time_tracking' ) . ' ' . $t_bugnote->time_tracking ) . "\n";
} else {
...

It works, but it does not look very nice.

In any case, I think bug should be reopened.
User avatar (0017778)
daryn (developer)
2008-05-06 09:29

 reopening to check note 0008849:0017775.
User avatar (0017950)
daryn (developer)
2008-05-29 15:53

 lolotrav can you check this with svn-revision 5303? I think another change slipped in between my original fix and your testing that broke the display and calculation of time tracking. I've removed the extra formatting and it seems to work correctly for me. This only applies to trunk. I did not patch the 1.1.x branch (I think victor did that earlier though). Thanks.
User avatar (0017962)
daryn (developer)
2008-05-30 12:14

 This works correctly for me now. I'm looking for confirmation from others, specifically lolotrav, that this is fixed in trunk.
User avatar (0017972)
lolotrav (reporter)
2008-05-31 14:36

 I received your email. Unfortunately I can't test your patch, our Mantis 1.1.1 is now in production.
Looking at your patch, I suppose it solves the problem (provided nobody else expects the ->time_tracking field to be formatted ; who added the call to db_minutes_to_hhmm(), and why ??)
User avatar (0018034)
daryn (developer)
2008-06-06 17:27

 I'm going to resolve this as my testing indicates that it is fixed.

lolotrav to present we haven't really supported outside use of the apis other than through the soap interface. All internal calls to this function already format the string as needed so i'm not too worried that someone is expecting a formatted string. I'm not sure when or why the db_minutes_to_hhmm call was added to the bugnotes... function but returning the actual value and allowing the caller to format it is the correct way to do this under the current api. Providing an additional function in the nature of bugnote->time_tracking->to_string is being discussed as a future possibility.

- Related Changesets
MantisBT: master e6c44ba1
Timestamp: 2008-04-21 14:52:57
Author: daryn
[ Details ] [ Diff ] 		Apply patch for bug 8849 - Emails ignore time tracking view threshold

git-svn-id: http://mantisbt.svn.sourceforge.net/svnroot/mantisbt/trunk@5185 [^] /?p=mantisbt.git;a=object;h=f5dc347c-c33d-0410-90a0-b07cc1902cb9
mod - core/email_api.php [ Diff ] [ File ]

- Issue History
Date Modified Username Field Change
2008-02-01 07:52 kynx New Issue
2008-02-01 07:52 kynx File Added: email_timetracking.diff
2008-02-02 01:20 daryn Assigned To => daryn
2008-02-02 01:20 daryn Status new => acknowledged
2008-04-21 10:55 daryn Status acknowledged => resolved
2008-04-21 10:55 daryn Fixed in Version => 1.2.0a2
2008-04-21 10:55 daryn Resolution open => fixed
2008-04-21 10:55 daryn Note Added: 0017639
2008-05-05 09:25 lolotrav Note Added: 0017775
2008-05-05 09:25 lolotrav Note Edited: 0017775
2008-05-06 09:29 daryn Status resolved => feedback
2008-05-06 09:29 daryn Resolution fixed => reopened
2008-05-06 09:29 daryn Note Added: 0017778
2008-05-06 09:29 daryn Status feedback => assigned
2008-05-21 02:38 vboctor Issue cloned: 0009184
2008-05-21 02:38 vboctor Relationship added parent of 0009184
2008-05-29 15:53 daryn Note Added: 0017950
2008-05-30 12:14 daryn Note Added: 0017962
2008-05-30 12:14 daryn Status assigned => feedback
2008-05-31 14:36 lolotrav Note Added: 0017972
2008-06-06 17:27 daryn Note Added: 0018034
2008-06-06 17:27 daryn Status @0@ => resolved
2008-06-06 17:27 daryn Resolution @0@ => fixed
2008-08-11 09:41 giallu Status resolved => closed
2008-10-20 16:50 Changeset attached master bcd70226 =>
2008-11-11 08:33 daryn Changeset attached master e6c44ba1 =>
2008-11-11 08:46 daryn Changeset attached master e6c44ba1 =>

MantisBT 1.2.16dev master-1.2.x-8c2bd07 [^]
Copyright © 2000 - 2013 MantisBT Team
Time: 0.1271 seconds.
memory usage: 2,841 KB
Powered by Mantis Bugtracker