Skip to content

Commit 743d332

Browse files
authored
Merge pull request #1105 from uw-it-aca/bug/sis-import-none
ensure sis_import var exists
2 parents 46b2a7a + 63415ee commit 743d332

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

sis_provisioner/management/commands/import_users.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,8 @@ def handle(self, *args, **options):
4242
try:
4343
sis_import = imp.import_csv()
4444

45-
if imp.priority == User.PRIORITY_IMMEDIATE:
45+
if (sis_import is not None and
46+
imp.priority == User.PRIORITY_IMMEDIATE):
4647
logger.info(f'SIS Import URL: {sis_import.post_url}, '
4748
f'Headers: {sis_import.post_headers}')
4849

sis_provisioner/models/__init__.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -142,6 +142,7 @@ def import_csv(self):
142142
if not self.csv_path:
143143
raise MissingImportPathException()
144144

145+
sis_import = None
145146
try:
146147
sis_import = sis_import_by_path(self.csv_path,
147148
self.override_sis_stickiness)

0 commit comments

Comments
 (0)