HelpScout1165153 Name Field issue (#857)

* Check that currentRegistration exists for locked profile
CruGlobal · Jun 7, 2024 · 6b5b8a6 · 6b5b8a6
1 parent cdd5825
commit 6b5b8a6
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/app/scripts/directives/blocks.js b/app/scripts/directives/blocks.js
@@ -30,6 +30,7 @@ angular.module('confRegistrationWebApp').directive('nameQuestion', function () {
         !$scope.adminEditRegistrant &&
           user &&
           user.employeeId &&
+          $scope.currentRegistration &&
           $scope.currentRegistrant ===
             $scope.currentRegistration.primaryRegistrantId &&
           $scope.block.profileType === 'NAME',
@@ -80,6 +81,7 @@ angular
           !$scope.adminEditRegistrant &&
             user &&
             user.employeeId &&
+            $scope.currentRegistration &&
             $scope.currentRegistrant ===
               $scope.currentRegistration.primaryRegistrantId &&
             $scope.block.profileType === 'EMAIL',

diff --git a/test/spec/directives/block.spec.js b/test/spec/directives/block.spec.js
@@ -47,6 +47,16 @@ describe('Directive: blocks', () => {
         expect($scope.lockedStaffProfileBlock).toBe(false);
       });
 
+      it('is false when currentRegistration is null', () => {
+        $scope.currentRegistration = null;
+        globalUserSpy.and.returnValue(null);
+
+        $compile('<name-question></name-question>')($scope);
+        $scope.$digest();
+
+        expect($scope.lockedStaffProfileBlock).toBe(false);
+      });
+
       it('is false for non-staff', () => {
         globalUserSpy.and.returnValue({ employeeId: null });