Skip to content

Commit 8bc313c

Browse files
Fix bug with repeated calls
Otherwise was adding every time since we reset stuff for `job` not `self` different. Felt smoother to align with all being job object to help maintain context since the array `attached_directories` was already reset above Prior output example: Directory ID: 9693 Schema ID: 3665 {'attached_directories_list': [{'directory_id': 9693, 'selected': 'sync'}]} Directory ID: 9695 Schema ID: 3666 {'attached_directories_list': [{'directory_id': 9693, 'selected': 'sync'}, {'directory_id': 9695, 'selected': 'sync'}]} Directory ID: 9697 Schema ID: 3667 {'attached_directories_list': [{'directory_id': 9693, 'selected': 'sync'}, {'directory_id': 9695, 'selected': 'sync'}, {'directory_id': 9697, 'selected': 'sync'}]} New example: Directory ID: 58 Schema ID: 12 {'attached_directories_list': [{'directory_id': 58, 'selected': 'sync'}]} Directory ID: 60 Schema ID: 13 {'attached_directories_list': [{'directory_id': 60, 'selected': 'sync'}]} Directory ID: 62 Schema ID: 14 {'attached_directories_list': [{'directory_id': 62, 'selected': 'sync'}]}
1 parent fa2a4fd commit 8bc313c

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

sdk/diffgram/job/job.py

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -201,12 +201,14 @@ def new(self,
201201
'Please provide at least one attached directory to the job in either sync_directories param or single_copy_directories')
202202

203203
for dir in sync_directories:
204-
self.__add_directory_to_job(directory=dir, mode='sync')
204+
job.__add_directory_to_job(directory=dir, mode='sync')
205205

206206
for dir in single_copy_directories:
207-
self.__add_directory_to_job(directory=dir, mode='sync')
207+
job.__add_directory_to_job(directory=dir, mode='sync')
208208

209-
job.attached_directories_dict = {'attached_directories_list': self.attached_directories}
209+
job.attached_directories_dict = {
210+
'attached_directories_list': job.attached_directories
211+
}
210212

211213
endpoint = "/api/v1/project/{}/job/new".format(self.client.project_string_id)
212214

0 commit comments

Comments
 (0)