This is an automated email from the ASF dual-hosted git repository.
klesh pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/incubator-devlake.git
The following commit(s) were added to refs/heads/main by this push:
new 3de984073 fix(tapd): fix overflow when converting lead time minutes
(#8244)
3de984073 is described below
commit 3de98407374f8c30a64af9a76899b2c7e9012686
Author: Lynwee <[email protected]>
AuthorDate: Wed Dec 18 15:07:46 2024 +0800
fix(tapd): fix overflow when converting lead time minutes (#8244)
---
backend/plugins/tapd/tasks/bug_converter.go | 9 +++++++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/backend/plugins/tapd/tasks/bug_converter.go
b/backend/plugins/tapd/tasks/bug_converter.go
index 635b153dc..1ae6eaefe 100644
--- a/backend/plugins/tapd/tasks/bug_converter.go
+++ b/backend/plugins/tapd/tasks/bug_converter.go
@@ -18,6 +18,7 @@ limitations under the License.
package tasks
import (
+ "math"
"reflect"
"strconv"
"time"
@@ -88,8 +89,12 @@ func ConvertBug(taskCtx plugin.SubTaskContext) errors.Error {
results = append(results, issueAssignee)
}
if domainL.ResolutionDate != nil && domainL.CreatedDate
!= nil {
- temp :=
uint(domainL.ResolutionDate.Sub(*domainL.CreatedDate).Minutes())
- domainL.LeadTimeMinutes = &temp
+ durationInMinutes :=
domainL.ResolutionDate.Sub(*domainL.CreatedDate).Minutes()
+ // we have found some issues' ResolutionDate is
earlier than CreatedDate in tapd.
+ if durationInMinutes > 0 && durationInMinutes <
math.MaxUint {
+ temp := uint(durationInMinutes)
+ domainL.LeadTimeMinutes = &temp
+ }
}
boardIssue := &ticket.BoardIssue{
BoardId:
getWorkspaceIdGen().Generate(toolL.ConnectionId, toolL.WorkspaceId),
