Merge pull request #189 from lucassabreu/bugfix/clone-wrong-date

fix: clone was using start time of the cloned

CHANGELOG.md

@@ -11,6 +11,11 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Fixed
+
+- `clone` command was using the start time of the copied time entry to close the current one
+  instead of the start time of new one being created.
+
 ## [v0.36.0] - 2022-05-09
 
 ### Added

cmd/clone.go

@@ -64,25 +64,25 @@ var cloneCmd = &cobra.Command{
 		if tec, err = manageEntry(
 			tec,
 			fillTimeEntryWithFlags(cmd.Flags()),
-			func(tei dto.TimeEntryImpl) (dto.TimeEntryImpl, error) {
+			func(tec dto.TimeEntryImpl) (dto.TimeEntryImpl, error) {
 				if noClosing {
-					return tei, nil
+					return tec, nil
 				}
 
-				return tei, validateClosingTimeEntry(
+				return tec, validateClosingTimeEntry(
 					c, tec.WorkspaceID, tec.UserID,
 				)
 			},
 			getAllowNameForIDsFn(c),
 			getPropsInteractiveFn(c, dc),
 			getDatesInteractiveFn(),
 			getValidateTimeEntryFn(c),
-			func(tei dto.TimeEntryImpl) (dto.TimeEntryImpl, error) {
+			func(tec dto.TimeEntryImpl) (dto.TimeEntryImpl, error) {
 				if noClosing {
-					return tei, nil
+					return tec, nil
 				}
 
-				return tei, out(tec, c)
+				return tec, out(tec, c)
 			},
 		); err != nil {
 			return err