Merge branch 'fix-nest-ternary-expressions' into 'main'

Refactored nest ternary expressions

See merge request soapbox-pub/soapbox!3179

.eslintrc.json

@@ -133,7 +133,7 @@
     "no-irregular-whitespace": "error",
     "no-loop-func": "error",
     "no-mixed-spaces-and-tabs": "error",
-    "no-nested-ternary": "warn",
+    "no-nested-ternary": "error",
     "no-restricted-imports": [
       "error",
       {

src/actions/settings.ts

@@ -241,7 +241,8 @@ const saveSettings = (opts?: SettingOpts) =>
 const getLocale = (state: RootState, fallback = 'en') => {
   const localeWithVariant = (getSettings(state).get('locale') as string).replace('_', '-');
   const locale = localeWithVariant.split('-')[0];
-  return Object.keys(messages).includes(localeWithVariant) ? localeWithVariant : Object.keys(messages).includes(locale) ? locale : fallback;
+  const fallbackLocale = Object.keys(messages).includes(locale) ? locale : fallback;
+  return Object.keys(messages).includes(localeWithVariant) ? localeWithVariant : fallbackLocale;
 };
 
 type SettingsAction =

src/features/admin/domains.tsx

@@ -51,7 +51,9 @@ const Domain: React.FC<IDomain> = ({ domain }) => {
     }));
   };
 
-  const domainState = domain.last_checked_at ? (domain.resolves ? 'active' : 'error') : 'pending';
+  const resolveState = domain.resolves ? 'active' : 'error';
+  const domainState = domain.last_checked_at ? resolveState : 'pending';
+
   const domainStateLabel = {
     active: <FormattedMessage id='admin.domains.resolve.success_label' defaultMessage='Resolves correctly' />,
     error: <FormattedMessage id='admin.domains.resolve.fail_label' defaultMessage='Not resolving' />,

src/features/auth-login/components/registration-form.tsx

@@ -141,13 +141,19 @@ const RegistrationForm: React.FC<IRegistrationForm> = ({ inviteToken }) => {
         /></p>}
     </>);
 
+    const confirmationHeading = needsConfirmation
+      ? intl.formatMessage(messages.needsConfirmationHeader)
+      : undefined;
+
+    const approvalHeading = needsApproval
+      ? intl.formatMessage(messages.needsApprovalHeader)
+      : undefined;
+
+    const heading = confirmationHeading || approvalHeading;
+
     dispatch(openModal('CONFIRM', {
       icon: require('@tabler/icons/outline/check.svg'),
-      heading: needsConfirmation
+      heading: heading,
-        ? intl.formatMessage(messages.needsConfirmationHeader)
-        : needsApproval
-          ? intl.formatMessage(messages.needsApprovalHeader)
-          : undefined,
       message,
       confirm: intl.formatMessage(messages.close),
     }));

src/features/backups/index.tsx

@@ -88,12 +88,14 @@ const Backups = () => {
     </Card>
   );
 
-  const body = showLoading ? <Spinner /> : backups.isEmpty() ? emptyMessage : (
+  const backupsContent = backups.isEmpty() ? emptyMessage : (
     <div className='mb-4 grid grid-cols-1 gap-4 sm:grid-cols-2'>
       {backups.map((backup) => <Backup key={backup.id} backup={backup} />)}
     </div>
   );
 
+  const body = showLoading ? <Spinner /> : backupsContent;
+
   return (
     <Column label={intl.formatMessage(messages.heading)}>
       {body}

src/features/filters/index.tsx

@@ -70,50 +70,56 @@ const Filters = () => {
         emptyMessage={emptyMessage}
         itemClassName='pb-4 last:pb-0'
       >
-        {filters.map((filter) => (
+        {filters.map((filter) => {
-          <div key={filter.id} className='rounded-lg bg-gray-100 p-4 dark:bg-primary-800'>
+
-            <Stack space={2}>
+          const truthMessageFilter = filter.filter_action === 'hide' ?
-              <Stack className='grow' space={1}>
+            <FormattedMessage id='filters.filters_list_hide_completely' defaultMessage='Hide content' /> :
-                <Text weight='medium'>
+            <FormattedMessage id='filters.filters_list_warn' defaultMessage='Display warning' />;
-                  <FormattedMessage id='filters.filters_list_phrases_label' defaultMessage='Keywords or phrases:' />
+
-                  {' '}
+          const falseMessageFilter = (filter.filter_action === 'hide' ?
-                  <Text theme='muted' tag='span'>{filter.keywords.map(keyword => keyword.keyword).join(', ')}</Text>
+            <FormattedMessage id='filters.filters_list_drop' defaultMessage='Drop' /> :
-                </Text>
+            <FormattedMessage id='filters.filters_list_hide' defaultMessage='Hide' />);
-                <Text weight='medium'>
+
-                  <FormattedMessage id='filters.filters_list_context_label' defaultMessage='Filter contexts:' />
+          return (
-                  {' '}
+            <div key={filter.id} className='rounded-lg bg-gray-100 p-4 dark:bg-primary-800'>
-                  <Text theme='muted' tag='span'>{filter.context.map(context => contexts[context] ? intl.formatMessage(contexts[context]) : context).join(', ')}</Text>
+              <Stack space={2}>
-                </Text>
+                <Stack className='grow' space={1}>
-                <HStack space={4} wrap>
                   <Text weight='medium'>
-                    {filtersV2 ? (
+                    <FormattedMessage id='filters.filters_list_phrases_label' defaultMessage='Keywords or phrases:' />
-                      filter.filter_action === 'hide' ?
+                    {' '}
-                        <FormattedMessage id='filters.filters_list_hide_completely' defaultMessage='Hide content' /> :
+                    <Text theme='muted' tag='span'>{filter.keywords.map(keyword => keyword.keyword).join(', ')}</Text>
-                        <FormattedMessage id='filters.filters_list_warn' defaultMessage='Display warning' />
-                    ) : (filter.filter_action === 'hide' ?
-                      <FormattedMessage id='filters.filters_list_drop' defaultMessage='Drop' /> :
-                      <FormattedMessage id='filters.filters_list_hide' defaultMessage='Hide' />)}
                   </Text>
-                  {filter.expires_at && (
+                  <Text weight='medium'>
+                    <FormattedMessage id='filters.filters_list_context_label' defaultMessage='Filter contexts:' />
+                    {' '}
+                    <Text theme='muted' tag='span'>{filter.context.map(context => contexts[context] ? intl.formatMessage(contexts[context]) : context).join(', ')}</Text>
+                  </Text>
+                  <HStack space={4} wrap>
                     <Text weight='medium'>
-                      {new Date(filter.expires_at).getTime() <= Date.now()
+                      {filtersV2 ? truthMessageFilter : falseMessageFilter}
-                        ? <FormattedMessage id='filters.filters_list_expired' defaultMessage='Expired' />
-                        : <RelativeTimestamp timestamp={filter.expires_at} className='whitespace-nowrap' futureDate />}
                     </Text>
-                  )}
+                    {filter.expires_at && (
+                      <Text weight='medium'>
+                        {new Date(filter.expires_at).getTime() <= Date.now()
+                          ? <FormattedMessage id='filters.filters_list_expired' defaultMessage='Expired' />
+                          : <RelativeTimestamp timestamp={filter.expires_at} className='whitespace-nowrap' futureDate />}
+                      </Text>
+                    )}
+                  </HStack>
+                </Stack>
+                <HStack space={2} justifyContent='end'>
+                  <Button theme='primary' onClick={handleFilterEdit(filter.id)}>
+                    {intl.formatMessage(messages.edit)}
+                  </Button>
+                  <Button theme='danger' onClick={handleFilterDelete(filter.id)}>
+                    {intl.formatMessage(messages.delete)}
+                  </Button>
                 </HStack>
               </Stack>
-              <HStack space={2} justifyContent='end'>
+            </div>
-                <Button theme='primary' onClick={handleFilterEdit(filter.id)}>
+          );
-                  {intl.formatMessage(messages.edit)}
+        },
-                </Button>
+        )}
-                <Button theme='danger' onClick={handleFilterDelete(filter.id)}>
-                  {intl.formatMessage(messages.delete)}
-                </Button>
-              </HStack>
-            </Stack>
-          </div>
-        ))}
       </ScrollableList>
     </Column>
   );

src/features/ui/components/modals/captcha-modal/captcha-modal.tsx

@@ -22,6 +22,8 @@ const CaptchaModal: React.FC<ICaptchaModal> = ({ onClose }) => {
     xPosition,
   } = useCaptcha();
 
+  const messageButton = tryAgain ? <FormattedMessage id='nostr_signup.captcha_try_again_button' defaultMessage='Try again' /> : <FormattedMessage id='nostr_signup.captcha_check_button' defaultMessage='Check' />;
+
   return (
     <Modal
       title={<FormattedMessage id='nostr_signup.captcha_title' defaultMessage='Human Verification' />} width='sm'
@@ -46,10 +48,7 @@ const CaptchaModal: React.FC<ICaptchaModal> = ({ onClose }) => {
             >
               {isSubmitting ? (
                 <FormattedMessage id='nostr_signup.captcha_check_button.checking' defaultMessage='Checking…' />
-              ) : (tryAgain ?
+              ) : messageButton}
-                <FormattedMessage id='nostr_signup.captcha_try_again_button' defaultMessage='Try again' /> :
-                <FormattedMessage id='nostr_signup.captcha_check_button' defaultMessage='Check' />
-              )}
             </Button>
             <Button onClick={loadCaptcha}>
               <FormattedMessage id='nostr_signup.captcha_reset_button' defaultMessage='Reset puzzle' />

src/normalizers/admin-account.ts

@@ -33,12 +33,18 @@ export const AdminAccountRecord = ImmutableRecord({
 
 const normalizePleromaAccount = (account: ImmutableMap<string, any>) => {
   if (!account.get('account')) {
+
+    const isAdmin = account.getIn(['roles', 'admin']);
+    const isModerator = account.getIn(['roles', 'moderator']) ? 'moderator' : null;
+
+    const accountRole = isAdmin ? 'admin' : isModerator;
+
     return account.withMutations(account => {
       account.set('approved', account.get('is_approved'));
       account.set('confirmed', account.get('is_confirmed'));
       account.set('disabled', !account.get('is_active'));
       account.set('invite_request', account.get('registration_reason'));
-      account.set('role', account.getIn(['roles', 'admin']) ? 'admin' : (account.getIn(['roles', 'moderator']) ? 'moderator' : null));
+      account.set('role', accountRole);
     });
   }