Merge pull request #52 from Timer2Ticket/duplicating_issue_names

Added patch for issue_names getting duplicated.

Author: hejdamax

src/jobs/remove_obsolete_mappings_job.ts

@@ -158,6 +158,7 @@ export class RemoveObsoleteMappingsJob extends SyncJob {
                 } else {
                     const service = SyncedServiceCreator.create(primaryServiceDefinition)
                     const relatedTimeEntriesFromApi = await service.getTimeEntriesRelatedToMappingObjectForUser(mapping, this._user);
+                    //User doesnt have to have time entries - so this if doesnt really make sense
                     if (!relatedTimeEntriesFromApi) {
                         operationsOk = false;
                         if (service.errors.length > 0) {

src/synced_services/toggl/toggl_synced_service.ts

@@ -373,25 +373,26 @@ export class TogglTrackSyncedService implements SyncedService {
     return entries;
   }
 
+
   async replaceTimeEntryDescription(timeEntry: ServiceTimeEntryObject, tagName: number | string) : Promise<void> {
     const start = new Date();
     start.setMonth(start.getMonth() - 6);
     const timeEntryFromApi = await this.getTimeEntryById(timeEntry.id, start);
 
-    if (timeEntryFromApi === null) {
-       return;
+    if (timeEntryFromApi === null || timeEntryFromApi.text.includes(tagName.toString())) {
+      return;
     }
 
     const body =
       [
         {'op': 'replace',
           'path': '/description',
-          'value': timeEntryFromApi != null ? timeEntryFromApi?.text + ` ${tagName}` : `${tagName}`
+          'value': timeEntryFromApi?.text + ` ${tagName}`
         }
       ]
     ;
-    const teStart = new Date(timeEntryFromApi.start).toISOString();
 
+    const teStart = new Date(timeEntryFromApi.start).toISOString();
     await this._retryAndWaitInCaseOfTooManyRequests(
         superagent
             .patch(`${this._workspacesTimeEntriesUri}/${timeEntry?.id}`)