Merge pull request #1480 from SenseNet/Fix-Short-TextField

Short text field no value fix

Author: hassanad94

packages/sn-controls-react/src/fieldcontrols/short-text.tsx

@@ -59,7 +59,9 @@ export const ShortText: React.FC<ReactClientFieldSetting<ShortTextFieldSetting>>
           </Typography>
           <Typography variant="body1" gutterBottom={true}>
             {/* Temporary hot fix */}
-            {typeof props.fieldValue !== 'string' ? '[OBJECT]' : props.fieldValue || localization.noValue}
+            {typeof props.fieldValue === 'object' && props.fieldValue !== null
+              ? '[OBJECT]'
+              : props.fieldValue || localization.noValue}
           </Typography>
         </div>
       )

packages/sn-controls-react/test/__snapshots__/short-text.test.tsx.snap

@@ -1,22 +1,5 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
-exports[`Short text field control in browse view should show [OBJECT] when iccorect type passed 1`] = `
-<div>
-  <WithStyles(ForwardRef(Typography))
-    gutterBottom={true}
-    variant="caption"
-  >
-    Display name
-  </WithStyles(ForwardRef(Typography))>
-  <WithStyles(ForwardRef(Typography))
-    gutterBottom={true}
-    variant="body1"
-  >
-    [OBJECT]
-  </WithStyles(ForwardRef(Typography))>
-</div>
-`;
-
 exports[`Short text field control in browse view should show the displayname and fieldValue when fieldValue is provided 1`] = `
 <div>
   <WithStyles(ForwardRef(Typography))

packages/sn-controls-react/test/short-text.test.tsx

@@ -24,12 +24,11 @@ describe('Short text field control', () => {
       const wrapper = shallow(<ShortText actionName="browse" settings={defaultSettings} />)
       expect(wrapper.find(Typography).last().text()).toBe(defaultLocalization.shortText.noValue)
     })
-    it('should show [OBJECT] when iccorect type passed', () => {
+    it('should show [OBJECT] when iccorect value type passed', () => {
       const value = { url: '' }
       const wrapper = shallow(<ShortText fieldValue={value as any} actionName="browse" settings={defaultSettings} />)
       expect(wrapper.find(Typography).first().text()).toBe(defaultSettings.DisplayName)
       expect(wrapper.find(Typography).last().text()).toBe('[OBJECT]')
-      expect(wrapper).toMatchSnapshot()
     })
   })
   describe('in edit/new view', () => {