fixing attribute form

TIER · Jun 23, 2021 · 3a968af · 3a968af
1 parent 6b5ef7a
commit 3a968af
Show file tree

Hide file tree

Showing 7 changed files with 17 additions and 21 deletions.
diff --git a/ui/public/assets/schema/attribute/attribute.schema.json b/ui/public/assets/schema/attribute/attribute.schema.json
@@ -93,19 +93,11 @@
                                 "BOOLEAN"
                             ]
                         },
-                        "defaultValue": {
+                        "defaultValueBoolean": {
                             "title": "label.entity-attribute-default",
                             "description": "tooltip.entity-attribute-default",
-                            "type": "string",
-                            "default": "true",
-                            "enum": [
-                                "true",
-                                "false"
-                            ],
-                            "enumNames": [
-                                "True",
-                                "False"
-                            ]
+                            "type": "boolean",
+                            "default": true
                         },
                         "persistValue": {
                             "type": "string",

diff --git a/ui/src/app/form/component/widgets/CheckboxWidget.js b/ui/src/app/form/component/widgets/CheckboxWidget.js
@@ -41,7 +41,7 @@ const CheckboxWidget = (props) => {
                     </span>
                     {schema.description && <InfoIcon value={schema.description} className="ml-2" />}
                 </span>}
-                checked={typeof value === "undefined" ? false : value}
+                checked={typeof value === "undefined" ? false : typeof value === 'boolean' ? value : value === 'true' ? true : false}
                 required={required}
                 disabled={disabled || readonly}
                 autoFocus={autofocus}

diff --git a/ui/src/app/form/component/widgets/RadioWidget.js b/ui/src/app/form/component/widgets/RadioWidget.js
@@ -46,7 +46,7 @@ const RadioWidget = ({
                 const itemDisabled =
                     Array.isArray(enumDisabled) &&
                     enumDisabled.indexOf(option.value) !== -1;
-                const checked = option.value === value;
+                const checked = option.value.toString() === value.toString();
 
                 const radio = (
                     <Form.Check

diff --git a/ui/src/app/metadata/domain/attribute/CustomAttributeDefinition.js b/ui/src/app/metadata/domain/attribute/CustomAttributeDefinition.js
@@ -18,8 +18,6 @@ export const CustomAttributeDefinition = {
                         'attributeName',
                         'attributeFriendlyName',
                         'displayName',
-                        'persistValue',
-                        'persistType',
                         'helpText'
                     ]
                 },
@@ -30,14 +28,20 @@ export const CustomAttributeDefinition = {
                         'defaultValue',
                         'defaultValueBoolean',
                         'defaultValueString',
-                        'customAttrListDefinitions'
+                        'customAttrListDefinitions',
+                        'persistValue',
+                        'persistType',
+                        'invert'
                     ]
                 }
             ]
         },
         defaultValueBoolean: {
             'ui:widget': 'radio'
         },
+        persistType: {
+            'ui:widget': 'hidden'
+        },
         customAttrListDefinitions: {
             'ui:field': 'StringListWithDefaultField',
             'ui:title': 'label.entity-attribute-list-options',
@@ -100,7 +104,8 @@ export const CustomAttributeDefinition = {
         if (attributeType === 'BOOLEAN') {
             formatted = {
                 ...formatted,
-                defaultValueBoolean: formatted.defaultValue === 'true' ? true : false
+                defaultValueBoolean: formatted.defaultValue === 'true' ? true : false,
+                invert: formatted.invert === 'true' ? true : false
             }
         }
 

diff --git a/ui/src/app/metadata/editor/MetadataAttributeEditor.js b/ui/src/app/metadata/editor/MetadataAttributeEditor.js
@@ -51,7 +51,6 @@ export function MetadataAttributeEditor({ children }) {
                     </Form>
                 </div>
             </div>
-            <pre>{JSON.stringify(errors, null, 4)}</pre>
         </div>
     );
 }
diff --git a/ui/src/app/metadata/view/MetadataAttributeEdit.js b/ui/src/app/metadata/view/MetadataAttributeEdit.js
@@ -35,7 +35,7 @@ export function MetadataAttributeEdit() {
     }
 
     async function save(metadata) {
-        const resp = await put(``, definition.parser(metadata));
+        const resp = await put(`/${id}`, definition.parser(metadata));
         if (response.ok) {
             gotoDetail({ refresh: true });
         } else {

diff --git a/ui/src/app/metadata/view/MetadataAttributeList.js b/ui/src/app/metadata/view/MetadataAttributeList.js
@@ -63,13 +63,13 @@ export function MetadataAttributeList ({entities, onDelete}) {
                                                 <td>{attr.helpText}</td>
                                                 <td>{attr.defaultValue?.toString()}</td>
                                                 <td className="text-right">
-                                                    <Link to={`../attributes/${attr.name}/edit`} className="btn btn-link text-primary">
+                                                    <Link to={`../attributes/${attr.resourceId}/edit`} className="btn btn-link text-primary">
                                                         <FontAwesomeIcon icon={faEdit} size="lg" />
                                                         <span className="sr-only">
                                                             <Translate value="action.edit">Edit</Translate>
                                                         </span>
                                                     </Link>
-                                                    <Button variant="link" className="text-danger" onClick={() => block(() => remove(attr.name))}>
+                                                    <Button variant="link" className="text-danger" onClick={() => block(() => remove(attr.resourceId))}>
                                                         <FontAwesomeIcon icon={faTrash} size="lg" />
                                                         <span className="sr-only">
                                                             <Translate value="action.delete">Delete</Translate>