refactor: clean up unused code and improve null checks in settings components

Author: Chisomchima

src/metadata-settings/metadataSettings.component.jsx

@@ -207,16 +207,6 @@ class MetadataSettings extends Component {
             td: { border: '1px solid #ddd', padding: '10px' },
         }
 
-        // const fieldGetter =
-        //     (field, filter = (x) => x) =>
-        //     // eslint-disable-next-line react/display-name, react/prop-types
-        //     ({ rowIndex, ...props }) =>
-        //         (
-        //             <Cell {...props}>
-        //                 {filter(this.state.metadataVersions[rowIndex][field])}
-        //             </Cell>
-        //         )
-
         const dateFmt = (str) => new Date(str).toLocaleString()
 
         return (

src/settingsFields.component.jsx

@@ -334,19 +334,18 @@ class SettingsFields extends React.Component {
 
                 // Base config, common for all component types
                 const validators = []
-                if (mapping.validators) {
+                if (mapping && mapping.validators) {
                     mapping.validators.forEach((name) => {
                         if (wordToValidatorMap.has(name)) {
                             const validator = wordToValidatorMap.get(name)
                             validators.push({
                                 validator,
-                                message: translateValidatorMessage(
-                                    validator.message
-                                ),
+                                message: translateValidatorMessage(validator.message),
                             })
                         }
                     })
                 }
+                
 
                 const fieldBase = {
                     name: key,