feat(core-connections): CB-1037 remove unnecessary optional chaining

webapp/packages/core-connections/src/ConnectionForm/ConnectionAccess/ConnectionAccessGrantedList.tsx

@@ -69,7 +69,9 @@ export const ConnectionAccessGrantedList: React.FC<Props> = observer(function Co
   const translate = useTranslate();
   const [selectedSubjects] = useState<Map<any, boolean>>(() => observable(new Map()));
   const [filterState] = useState<IFilterState>(() => observable({ filterValue: '' }));
-  const subjectsSelected = useMemo(() => computed(() => Array.from(selectedSubjects.values()).some(Boolean)), []);
+  const subjectsSelected = useMemo(() => computed(
+    () => Array.from(selectedSubjects.values()).some(Boolean)
+  ), [selectedSubjects]);
 
   const revoke = useCallback(() => {
     const subjectsToRevoke = [];

webapp/packages/core-connections/src/ConnectionForm/ConnectionAccess/getFilteredSubjects.ts

@@ -14,7 +14,7 @@ import type { AdminRoleInfo, AdminUserInfoFragment } from '@cloudbeaver/core-sdk
  */
 export function getFilteredUsers(users: AdminUserInfoFragment[], filter: string): AdminUserInfoFragment[] {
   return users
-    .filter(user => user.userId?.toLowerCase().includes(filter.toLowerCase()))
+    .filter(user => user.userId.toLowerCase().includes(filter.toLowerCase()))
     .sort((a, b) => (a.userId).localeCompare(b.userId));
 }