fix: catch trying to sync non-user records

SlateFoundation · Sep 6, 2023 · 4c0ae96 · 4c0ae96
1 parent d337960
commit 4c0ae96
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/php-classes/Slate/Connectors/AbstractSpreadsheetConnector.php b/php-classes/Slate/Connectors/AbstractSpreadsheetConnector.php
@@ -278,6 +278,14 @@ public static function pullStudents(IJob $Job, SpreadsheetReader $spreadsheet, $
 
             // apply values from spreadsheet
             try {
+                if (!$Record->isA(User::class)) {
+                    throw new RemoteRecordInvalid(
+                        'not-a-user',
+                        sprintf('Matched student to record %s from row #%u, but record class is not user or student...refusing to promote automatically', $Record->getTitle(), $results['analyzed']),
+                        $row
+                    );
+                }
+
                 static::_applyStudentUserChanges($Job, $Record, $row, $results);
             } catch (RemoteRecordInvalid $e) {
                 if ($e->getValueKey()) {