computed attribute code improvements

2 files changed, 13 insertions, 11 deletions

diff --git a/modern/src/settings/ComputedAttributePage.js b/modern/src/settings/ComputedAttributePage.js
index f985152..7ff9511 100644
--- a/modern/src/settings/ComputedAttributePage.js
+++ b/modern/src/settings/ComputedAttributePage.js
@@ -25,6 +25,17 @@ const ComputedAttributePage =() => {
     type: value.type,
   }));
 
+  const handleChange = event => {
+    const newValue = event.target.value;
+    setKey(newValue);
+    const positionAttribute = positionAttributes[newValue];
+    if(positionAttribute && positionAttribute.type) {
+      setItem({...item, attribute: newValue, type: positionAttribute.type});
+    } else {
+      setItem({...item, attribute: newValue});
+    }
+  }
+
   return (
     <EditItemView endpoint="/attributes/computed" item={item} setItem={setItem}>
       {item &&
@@ -45,16 +56,7 @@ const ComputedAttributePage =() => {
               <InputLabel>{t('sharedAttribute')}</InputLabel>
               <Select 
                 value={item.attribute || ''} 
-                onChange={(e) => {
-                  const newValue = e.target.value;
-                  const positionAttribute = positionAttributes[newValue];
-                  setKey(newValue);
-                  if(positionAttribute && positionAttribute.type) {
-                    setItem({...item, attribute: newValue, type: positionAttribute.type});
-                  }else {
-                    setItem({...item, attribute: newValue});
-                  }
-                }}>
+                onChange={handleChange}>
                 {options.map((option) => (
                   <MenuItem key={option.key} value={option.key}>{option.name}</MenuItem>
                 ))}
diff --git a/modern/src/settings/ComputedAttributesPage.js b/modern/src/settings/ComputedAttributesPage.js
index 9ca5b69..53d1172 100644
--- a/modern/src/settings/ComputedAttributesPage.js
+++ b/modern/src/settings/ComputedAttributesPage.js
@@ -61,7 +61,7 @@ const ComputedAttributesPage = () => {
   return (
     <>
       <MainToolbar />
-      <EditCollectionView content={ComputedAttributeView} editPath="/settings/attribute/computed" endpoint="attributes/computed" />
+      <EditCollectionView content={ComputedAttributeView} editPath="/settings/attribute" endpoint="attributes/computed" />
     </>
   );
 }