You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@royale.apache.org by pi...@apache.org on 2022/08/12 09:56:09 UTC

[royale-asjs] branch develop updated: Basic: Fix issue where in scenario of providing object in dataProvider contains fields either null or undefined causes display [object Object] instead of empty string

This is an automated email from the ASF dual-hosted git repository.

piotrz pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/royale-asjs.git


The following commit(s) were added to refs/heads/develop by this push:
     new 9cf3840b26 Basic: Fix issue where in scenario of providing object in dataProvider contains fields either null or undefined causes display [object Object] instead of empty string
9cf3840b26 is described below

commit 9cf3840b2646f57a312a121db17a991766307fef
Author: Piotr Zarzycki <pi...@gmail.com>
AuthorDate: Fri Aug 12 11:54:54 2022 +0200

    Basic: Fix issue where in scenario of providing object in dataProvider contains fields either null or undefined causes display [object Object] instead of empty string
    
    Example of dp object: {firstName: "Piotr", lastName: null}
---
 .../royale/org/apache/royale/html/util/getLabelFromData.as     | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as b/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as
index 18aa873040..1581f0c21b 100644
--- a/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as
+++ b/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as
@@ -65,9 +65,17 @@ package org.apache.royale.html.util
         }
 
         var label:String = data["label"];
-        if(label != null){
+        if(label != null)
+        {
             return label;
         }
+
+        if (obj is IHasLabelField && (obj as IHasLabelField).labelField &&
+                (data[(obj as IHasLabelField).labelField] == null || data[(obj as IHasLabelField).labelField] == undefined))
+        {
+            return "";
+        }
+
         return "" + data;
 
     }


Re: [royale-asjs] branch develop updated: Basic: Fix issue where in scenario of providing object in dataProvider contains fields either null or undefined causes display [object Object] instead of empty string

Posted by Harbs <ha...@gmail.com>.
Great. Thanks for letting me know.

> On Aug 24, 2022, at 1:19 PM, Piotr Zarzycki <pi...@gmail.com> wrote:
> 
> Hi Harbs,
> 
> Everything is working fine. I have tested with our application. Thanks for
> your help and advice.
> 
> Thanks,
> Piotr
> 
> pt., 19 sie 2022 o 20:50 Piotr Zarzycki <pi...@gmail.com>
> napisał(a):
> 
>> Harbs,
>> 
>> I’m sorry about that. Not yet, but I will next week.
>> 
>> Thanks for your help,
>> Piotr
>> 
>> On Fri, 19 Aug 2022 at 16:50, Harbs <ha...@gmail.com> wrote:
>> 
>>> Did you double check what I did?
>>> 
>>> I’d feel more comfortable if you did...
>>> 
>>>> On Aug 14, 2022, at 4:10 PM, Harbs <ha...@gmail.com> wrote:
>>>> 
>>>> I just committed what I think will work better.
>>>> 
>>>> Please double check what I did...
>>>> 
>>>>> On Aug 14, 2022, at 12:51 PM, Piotr Zarzycki <
>>> piotrzarzycki21@gmail.com> wrote:
>>>>> 
>>>>> No. In fields there is null or undefined - what do you suggest ?
>>>>> 
>>>>> pt., 12 sie 2022 o 14:38 Gabe Harbs <ha...@gmail.com>
>>> napisał(a):
>>>>> 
>>>>>> No. That’s probably not what you want. If there’s an empty string that
>>>>>> should be used.
>>>>>> 
>>>>>>> On 12 Aug 2022, at 13:57, Piotr Zarzycki <pi...@gmail.com>
>>>>>> wrote:
>>>>>>> 
>>>>>>> So maybe it's enough if I just made:
>>>>>>> 
>>>>>>> If (data[(obj as IHasLabelField).labelField]) ?
>>>>>>> 
>>>>>>> pt., 12 sie 2022 o 12:46 Gabe Harbs <ha...@gmail.com>
>>> napisał(a):
>>>>>>> 
>>>>>>>> FWIW: undefined == null is true. You don’t need to test for both
>>> null
>>>>>> and
>>>>>>>> undefined.
>>>>>>>> 
>>>>>>>> 
>>>>>>>>>> On 12 Aug 2022, at 12:56, piotrz@apache.org wrote:
>>>>>>>>> 
>>>>>>>>> This is an automated email from the ASF dual-hosted git
>>> repository.
>>>>>>>>> 
>>>>>>>>> piotrz pushed a commit to branch develop
>>>>>>>>> in repository https://gitbox.apache.org/repos/asf/royale-asjs.git
>>>>>>>>> 
>>>>>>>>> 
>>>>>>>>> The following commit(s) were added to refs/heads/develop by this
>>> push:
>>>>>>>>> new 9cf3840b26 Basic: Fix issue where in scenario of providing
>>>>>>>> object in dataProvider contains fields either null or undefined
>>> causes
>>>>>>>> display [object Object] instead of empty string
>>>>>>>>> 9cf3840b26 is described below
>>>>>>>>> 
>>>>>>>>> commit 9cf3840b2646f57a312a121db17a991766307fef
>>>>>>>>> Author: Piotr Zarzycki <pi...@gmail.com>
>>>>>>>>> AuthorDate: Fri Aug 12 11:54:54 2022 +0200
>>>>>>>>> 
>>>>>>>>> Basic: Fix issue where in scenario of providing object in
>>>>>>>> dataProvider contains fields either null or undefined causes display
>>>>>>>> [object Object] instead of empty string
>>>>>>>>> 
>>>>>>>>> Example of dp object: {firstName: "Piotr", lastName: null}
>>>>>>>>> ---
>>>>>>>>> .../royale/org/apache/royale/html/util/getLabelFromData.as     | 10
>>>>>>>> +++++++++-
>>>>>>>>> 1 file changed, 9 insertions(+), 1 deletion(-)
>>>>>>>>> 
>>>>>>>>> diff --git
>>>>>>>> 
>>>>>> 
>>> a/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as
>>>>>>>> 
>>>>>> 
>>> b/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as
>>>>>>>>> index 18aa873040..1581f0c21b 100644
>>>>>>>>> ---
>>>>>>>> 
>>>>>> 
>>> a/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as
>>>>>>>>> +++
>>>>>>>> 
>>>>>> 
>>> b/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as
>>>>>>>>> @@ -65,9 +65,17 @@ package org.apache.royale.html.util
>>>>>>>>>     }
>>>>>>>>> 
>>>>>>>>>     var label:String = data["label"];
>>>>>>>>> -        if(label != null){
>>>>>>>>> +        if(label != null)
>>>>>>>>> +        {
>>>>>>>>>         return label;
>>>>>>>>>     }
>>>>>>>>> +
>>>>>>>>> +        if (obj is IHasLabelField && (obj as
>>>>>> IHasLabelField).labelField
>>>>>>>> &&
>>>>>>>>> +                (data[(obj as IHasLabelField).labelField] == null
>>> ||
>>>>>>>> data[(obj as IHasLabelField).labelField] == undefined))
>>>>>>>>> +        {
>>>>>>>>> +            return "";
>>>>>>>>> +        }
>>>>>>>>> +
>>>>>>>>>     return "" + data;
>>>>>>>>> 
>>>>>>>>> }
>>>>>>>>> 
>>>>>>>> 
>>>>>>> 
>>>>>>> 
>>>>>>> --
>>>>>>> 
>>>>>>> Piotr Zarzycki
>>>>>> 
>>>>> 
>>>>> 
>>>>> --
>>>>> 
>>>>> Piotr Zarzycki
>>>> 
>>> 
>>> --
>> 
>> Piotr Zarzycki
>> 
> 
> 
> -- 
> 
> Piotr Zarzycki


Re: [royale-asjs] branch develop updated: Basic: Fix issue where in scenario of providing object in dataProvider contains fields either null or undefined causes display [object Object] instead of empty string

Posted by Piotr Zarzycki <pi...@gmail.com>.
Hi Harbs,

Everything is working fine. I have tested with our application. Thanks for
your help and advice.

Thanks,
Piotr

pt., 19 sie 2022 o 20:50 Piotr Zarzycki <pi...@gmail.com>
napisał(a):

> Harbs,
>
> I’m sorry about that. Not yet, but I will next week.
>
> Thanks for your help,
> Piotr
>
> On Fri, 19 Aug 2022 at 16:50, Harbs <ha...@gmail.com> wrote:
>
>> Did you double check what I did?
>>
>> I’d feel more comfortable if you did...
>>
>> > On Aug 14, 2022, at 4:10 PM, Harbs <ha...@gmail.com> wrote:
>> >
>> > I just committed what I think will work better.
>> >
>> > Please double check what I did...
>> >
>> >> On Aug 14, 2022, at 12:51 PM, Piotr Zarzycki <
>> piotrzarzycki21@gmail.com> wrote:
>> >>
>> >> No. In fields there is null or undefined - what do you suggest ?
>> >>
>> >> pt., 12 sie 2022 o 14:38 Gabe Harbs <ha...@gmail.com>
>> napisał(a):
>> >>
>> >>> No. That’s probably not what you want. If there’s an empty string that
>> >>> should be used.
>> >>>
>> >>>> On 12 Aug 2022, at 13:57, Piotr Zarzycki <pi...@gmail.com>
>> >>> wrote:
>> >>>>
>> >>>> So maybe it's enough if I just made:
>> >>>>
>> >>>> If (data[(obj as IHasLabelField).labelField]) ?
>> >>>>
>> >>>> pt., 12 sie 2022 o 12:46 Gabe Harbs <ha...@gmail.com>
>> napisał(a):
>> >>>>
>> >>>>> FWIW: undefined == null is true. You don’t need to test for both
>> null
>> >>> and
>> >>>>> undefined.
>> >>>>>
>> >>>>>
>> >>>>>>> On 12 Aug 2022, at 12:56, piotrz@apache.org wrote:
>> >>>>>>
>> >>>>>> This is an automated email from the ASF dual-hosted git
>> repository.
>> >>>>>>
>> >>>>>> piotrz pushed a commit to branch develop
>> >>>>>> in repository https://gitbox.apache.org/repos/asf/royale-asjs.git
>> >>>>>>
>> >>>>>>
>> >>>>>> The following commit(s) were added to refs/heads/develop by this
>> push:
>> >>>>>>  new 9cf3840b26 Basic: Fix issue where in scenario of providing
>> >>>>> object in dataProvider contains fields either null or undefined
>> causes
>> >>>>> display [object Object] instead of empty string
>> >>>>>> 9cf3840b26 is described below
>> >>>>>>
>> >>>>>> commit 9cf3840b2646f57a312a121db17a991766307fef
>> >>>>>> Author: Piotr Zarzycki <pi...@gmail.com>
>> >>>>>> AuthorDate: Fri Aug 12 11:54:54 2022 +0200
>> >>>>>>
>> >>>>>> Basic: Fix issue where in scenario of providing object in
>> >>>>> dataProvider contains fields either null or undefined causes display
>> >>>>> [object Object] instead of empty string
>> >>>>>>
>> >>>>>> Example of dp object: {firstName: "Piotr", lastName: null}
>> >>>>>> ---
>> >>>>>> .../royale/org/apache/royale/html/util/getLabelFromData.as     | 10
>> >>>>> +++++++++-
>> >>>>>> 1 file changed, 9 insertions(+), 1 deletion(-)
>> >>>>>>
>> >>>>>> diff --git
>> >>>>>
>> >>>
>> a/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as
>> >>>>>
>> >>>
>> b/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as
>> >>>>>> index 18aa873040..1581f0c21b 100644
>> >>>>>> ---
>> >>>>>
>> >>>
>> a/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as
>> >>>>>> +++
>> >>>>>
>> >>>
>> b/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as
>> >>>>>> @@ -65,9 +65,17 @@ package org.apache.royale.html.util
>> >>>>>>      }
>> >>>>>>
>> >>>>>>      var label:String = data["label"];
>> >>>>>> -        if(label != null){
>> >>>>>> +        if(label != null)
>> >>>>>> +        {
>> >>>>>>          return label;
>> >>>>>>      }
>> >>>>>> +
>> >>>>>> +        if (obj is IHasLabelField && (obj as
>> >>> IHasLabelField).labelField
>> >>>>> &&
>> >>>>>> +                (data[(obj as IHasLabelField).labelField] == null
>> ||
>> >>>>> data[(obj as IHasLabelField).labelField] == undefined))
>> >>>>>> +        {
>> >>>>>> +            return "";
>> >>>>>> +        }
>> >>>>>> +
>> >>>>>>      return "" + data;
>> >>>>>>
>> >>>>>>  }
>> >>>>>>
>> >>>>>
>> >>>>
>> >>>>
>> >>>> --
>> >>>>
>> >>>> Piotr Zarzycki
>> >>>
>> >>
>> >>
>> >> --
>> >>
>> >> Piotr Zarzycki
>> >
>>
>> --
>
> Piotr Zarzycki
>


-- 

Piotr Zarzycki

Re: [royale-asjs] branch develop updated: Basic: Fix issue where in scenario of providing object in dataProvider contains fields either null or undefined causes display [object Object] instead of empty string

Posted by Piotr Zarzycki <pi...@gmail.com>.
Harbs,

I’m sorry about that. Not yet, but I will next week.

Thanks for your help,
Piotr

On Fri, 19 Aug 2022 at 16:50, Harbs <ha...@gmail.com> wrote:

> Did you double check what I did?
>
> I’d feel more comfortable if you did...
>
> > On Aug 14, 2022, at 4:10 PM, Harbs <ha...@gmail.com> wrote:
> >
> > I just committed what I think will work better.
> >
> > Please double check what I did...
> >
> >> On Aug 14, 2022, at 12:51 PM, Piotr Zarzycki <pi...@gmail.com>
> wrote:
> >>
> >> No. In fields there is null or undefined - what do you suggest ?
> >>
> >> pt., 12 sie 2022 o 14:38 Gabe Harbs <ha...@gmail.com> napisał(a):
> >>
> >>> No. That’s probably not what you want. If there’s an empty string that
> >>> should be used.
> >>>
> >>>> On 12 Aug 2022, at 13:57, Piotr Zarzycki <pi...@gmail.com>
> >>> wrote:
> >>>>
> >>>> So maybe it's enough if I just made:
> >>>>
> >>>> If (data[(obj as IHasLabelField).labelField]) ?
> >>>>
> >>>> pt., 12 sie 2022 o 12:46 Gabe Harbs <ha...@gmail.com>
> napisał(a):
> >>>>
> >>>>> FWIW: undefined == null is true. You don’t need to test for both null
> >>> and
> >>>>> undefined.
> >>>>>
> >>>>>
> >>>>>>> On 12 Aug 2022, at 12:56, piotrz@apache.org wrote:
> >>>>>>
> >>>>>> This is an automated email from the ASF dual-hosted git repository.
> >>>>>>
> >>>>>> piotrz pushed a commit to branch develop
> >>>>>> in repository https://gitbox.apache.org/repos/asf/royale-asjs.git
> >>>>>>
> >>>>>>
> >>>>>> The following commit(s) were added to refs/heads/develop by this
> push:
> >>>>>>  new 9cf3840b26 Basic: Fix issue where in scenario of providing
> >>>>> object in dataProvider contains fields either null or undefined
> causes
> >>>>> display [object Object] instead of empty string
> >>>>>> 9cf3840b26 is described below
> >>>>>>
> >>>>>> commit 9cf3840b2646f57a312a121db17a991766307fef
> >>>>>> Author: Piotr Zarzycki <pi...@gmail.com>
> >>>>>> AuthorDate: Fri Aug 12 11:54:54 2022 +0200
> >>>>>>
> >>>>>> Basic: Fix issue where in scenario of providing object in
> >>>>> dataProvider contains fields either null or undefined causes display
> >>>>> [object Object] instead of empty string
> >>>>>>
> >>>>>> Example of dp object: {firstName: "Piotr", lastName: null}
> >>>>>> ---
> >>>>>> .../royale/org/apache/royale/html/util/getLabelFromData.as     | 10
> >>>>> +++++++++-
> >>>>>> 1 file changed, 9 insertions(+), 1 deletion(-)
> >>>>>>
> >>>>>> diff --git
> >>>>>
> >>>
> a/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as
> >>>>>
> >>>
> b/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as
> >>>>>> index 18aa873040..1581f0c21b 100644
> >>>>>> ---
> >>>>>
> >>>
> a/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as
> >>>>>> +++
> >>>>>
> >>>
> b/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as
> >>>>>> @@ -65,9 +65,17 @@ package org.apache.royale.html.util
> >>>>>>      }
> >>>>>>
> >>>>>>      var label:String = data["label"];
> >>>>>> -        if(label != null){
> >>>>>> +        if(label != null)
> >>>>>> +        {
> >>>>>>          return label;
> >>>>>>      }
> >>>>>> +
> >>>>>> +        if (obj is IHasLabelField && (obj as
> >>> IHasLabelField).labelField
> >>>>> &&
> >>>>>> +                (data[(obj as IHasLabelField).labelField] == null
> ||
> >>>>> data[(obj as IHasLabelField).labelField] == undefined))
> >>>>>> +        {
> >>>>>> +            return "";
> >>>>>> +        }
> >>>>>> +
> >>>>>>      return "" + data;
> >>>>>>
> >>>>>>  }
> >>>>>>
> >>>>>
> >>>>
> >>>>
> >>>> --
> >>>>
> >>>> Piotr Zarzycki
> >>>
> >>
> >>
> >> --
> >>
> >> Piotr Zarzycki
> >
>
> --

Piotr Zarzycki

Re: [royale-asjs] branch develop updated: Basic: Fix issue where in scenario of providing object in dataProvider contains fields either null or undefined causes display [object Object] instead of empty string

Posted by Harbs <ha...@gmail.com>.
Did you double check what I did?

I’d feel more comfortable if you did...

> On Aug 14, 2022, at 4:10 PM, Harbs <ha...@gmail.com> wrote:
> 
> I just committed what I think will work better.
> 
> Please double check what I did...
> 
>> On Aug 14, 2022, at 12:51 PM, Piotr Zarzycki <pi...@gmail.com> wrote:
>> 
>> No. In fields there is null or undefined - what do you suggest ?
>> 
>> pt., 12 sie 2022 o 14:38 Gabe Harbs <ha...@gmail.com> napisał(a):
>> 
>>> No. That’s probably not what you want. If there’s an empty string that
>>> should be used.
>>> 
>>>> On 12 Aug 2022, at 13:57, Piotr Zarzycki <pi...@gmail.com>
>>> wrote:
>>>> 
>>>> So maybe it's enough if I just made:
>>>> 
>>>> If (data[(obj as IHasLabelField).labelField]) ?
>>>> 
>>>> pt., 12 sie 2022 o 12:46 Gabe Harbs <ha...@gmail.com> napisał(a):
>>>> 
>>>>> FWIW: undefined == null is true. You don’t need to test for both null
>>> and
>>>>> undefined.
>>>>> 
>>>>> 
>>>>>>> On 12 Aug 2022, at 12:56, piotrz@apache.org wrote:
>>>>>> 
>>>>>> This is an automated email from the ASF dual-hosted git repository.
>>>>>> 
>>>>>> piotrz pushed a commit to branch develop
>>>>>> in repository https://gitbox.apache.org/repos/asf/royale-asjs.git
>>>>>> 
>>>>>> 
>>>>>> The following commit(s) were added to refs/heads/develop by this push:
>>>>>>  new 9cf3840b26 Basic: Fix issue where in scenario of providing
>>>>> object in dataProvider contains fields either null or undefined causes
>>>>> display [object Object] instead of empty string
>>>>>> 9cf3840b26 is described below
>>>>>> 
>>>>>> commit 9cf3840b2646f57a312a121db17a991766307fef
>>>>>> Author: Piotr Zarzycki <pi...@gmail.com>
>>>>>> AuthorDate: Fri Aug 12 11:54:54 2022 +0200
>>>>>> 
>>>>>> Basic: Fix issue where in scenario of providing object in
>>>>> dataProvider contains fields either null or undefined causes display
>>>>> [object Object] instead of empty string
>>>>>> 
>>>>>> Example of dp object: {firstName: "Piotr", lastName: null}
>>>>>> ---
>>>>>> .../royale/org/apache/royale/html/util/getLabelFromData.as     | 10
>>>>> +++++++++-
>>>>>> 1 file changed, 9 insertions(+), 1 deletion(-)
>>>>>> 
>>>>>> diff --git
>>>>> 
>>> a/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as
>>>>> 
>>> b/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as
>>>>>> index 18aa873040..1581f0c21b 100644
>>>>>> ---
>>>>> 
>>> a/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as
>>>>>> +++
>>>>> 
>>> b/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as
>>>>>> @@ -65,9 +65,17 @@ package org.apache.royale.html.util
>>>>>>      }
>>>>>> 
>>>>>>      var label:String = data["label"];
>>>>>> -        if(label != null){
>>>>>> +        if(label != null)
>>>>>> +        {
>>>>>>          return label;
>>>>>>      }
>>>>>> +
>>>>>> +        if (obj is IHasLabelField && (obj as
>>> IHasLabelField).labelField
>>>>> &&
>>>>>> +                (data[(obj as IHasLabelField).labelField] == null ||
>>>>> data[(obj as IHasLabelField).labelField] == undefined))
>>>>>> +        {
>>>>>> +            return "";
>>>>>> +        }
>>>>>> +
>>>>>>      return "" + data;
>>>>>> 
>>>>>>  }
>>>>>> 
>>>>> 
>>>> 
>>>> 
>>>> --
>>>> 
>>>> Piotr Zarzycki
>>> 
>> 
>> 
>> -- 
>> 
>> Piotr Zarzycki
> 


Re: [royale-asjs] branch develop updated: Basic: Fix issue where in scenario of providing object in dataProvider contains fields either null or undefined causes display [object Object] instead of empty string

Posted by Harbs <ha...@gmail.com>.
I just committed what I think will work better.

Please double check what I did...

> On Aug 14, 2022, at 12:51 PM, Piotr Zarzycki <pi...@gmail.com> wrote:
> 
> No. In fields there is null or undefined - what do you suggest ?
> 
> pt., 12 sie 2022 o 14:38 Gabe Harbs <ha...@gmail.com> napisał(a):
> 
>> No. That’s probably not what you want. If there’s an empty string that
>> should be used.
>> 
>>> On 12 Aug 2022, at 13:57, Piotr Zarzycki <pi...@gmail.com>
>> wrote:
>>> 
>>> So maybe it's enough if I just made:
>>> 
>>> If (data[(obj as IHasLabelField).labelField]) ?
>>> 
>>> pt., 12 sie 2022 o 12:46 Gabe Harbs <ha...@gmail.com> napisał(a):
>>> 
>>>> FWIW: undefined == null is true. You don’t need to test for both null
>> and
>>>> undefined.
>>>> 
>>>> 
>>>>>> On 12 Aug 2022, at 12:56, piotrz@apache.org wrote:
>>>>> 
>>>>> This is an automated email from the ASF dual-hosted git repository.
>>>>> 
>>>>> piotrz pushed a commit to branch develop
>>>>> in repository https://gitbox.apache.org/repos/asf/royale-asjs.git
>>>>> 
>>>>> 
>>>>> The following commit(s) were added to refs/heads/develop by this push:
>>>>>   new 9cf3840b26 Basic: Fix issue where in scenario of providing
>>>> object in dataProvider contains fields either null or undefined causes
>>>> display [object Object] instead of empty string
>>>>> 9cf3840b26 is described below
>>>>> 
>>>>> commit 9cf3840b2646f57a312a121db17a991766307fef
>>>>> Author: Piotr Zarzycki <pi...@gmail.com>
>>>>> AuthorDate: Fri Aug 12 11:54:54 2022 +0200
>>>>> 
>>>>>  Basic: Fix issue where in scenario of providing object in
>>>> dataProvider contains fields either null or undefined causes display
>>>> [object Object] instead of empty string
>>>>> 
>>>>>  Example of dp object: {firstName: "Piotr", lastName: null}
>>>>> ---
>>>>> .../royale/org/apache/royale/html/util/getLabelFromData.as     | 10
>>>> +++++++++-
>>>>> 1 file changed, 9 insertions(+), 1 deletion(-)
>>>>> 
>>>>> diff --git
>>>> 
>> a/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as
>>>> 
>> b/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as
>>>>> index 18aa873040..1581f0c21b 100644
>>>>> ---
>>>> 
>> a/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as
>>>>> +++
>>>> 
>> b/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as
>>>>> @@ -65,9 +65,17 @@ package org.apache.royale.html.util
>>>>>       }
>>>>> 
>>>>>       var label:String = data["label"];
>>>>> -        if(label != null){
>>>>> +        if(label != null)
>>>>> +        {
>>>>>           return label;
>>>>>       }
>>>>> +
>>>>> +        if (obj is IHasLabelField && (obj as
>> IHasLabelField).labelField
>>>> &&
>>>>> +                (data[(obj as IHasLabelField).labelField] == null ||
>>>> data[(obj as IHasLabelField).labelField] == undefined))
>>>>> +        {
>>>>> +            return "";
>>>>> +        }
>>>>> +
>>>>>       return "" + data;
>>>>> 
>>>>>   }
>>>>> 
>>>> 
>>> 
>>> 
>>> --
>>> 
>>> Piotr Zarzycki
>> 
> 
> 
> -- 
> 
> Piotr Zarzycki


Re: [royale-asjs] branch develop updated: Basic: Fix issue where in scenario of providing object in dataProvider contains fields either null or undefined causes display [object Object] instead of empty string

Posted by Piotr Zarzycki <pi...@gmail.com>.
No. In fields there is null or undefined - what do you suggest ?

pt., 12 sie 2022 o 14:38 Gabe Harbs <ha...@gmail.com> napisał(a):

> No. That’s probably not what you want. If there’s an empty string that
> should be used.
>
> > On 12 Aug 2022, at 13:57, Piotr Zarzycki <pi...@gmail.com>
> wrote:
> >
> > So maybe it's enough if I just made:
> >
> > If (data[(obj as IHasLabelField).labelField]) ?
> >
> > pt., 12 sie 2022 o 12:46 Gabe Harbs <ha...@gmail.com> napisał(a):
> >
> >> FWIW: undefined == null is true. You don’t need to test for both null
> and
> >> undefined.
> >>
> >>
> >>>> On 12 Aug 2022, at 12:56, piotrz@apache.org wrote:
> >>>
> >>> This is an automated email from the ASF dual-hosted git repository.
> >>>
> >>> piotrz pushed a commit to branch develop
> >>> in repository https://gitbox.apache.org/repos/asf/royale-asjs.git
> >>>
> >>>
> >>> The following commit(s) were added to refs/heads/develop by this push:
> >>>    new 9cf3840b26 Basic: Fix issue where in scenario of providing
> >> object in dataProvider contains fields either null or undefined causes
> >> display [object Object] instead of empty string
> >>> 9cf3840b26 is described below
> >>>
> >>> commit 9cf3840b2646f57a312a121db17a991766307fef
> >>> Author: Piotr Zarzycki <pi...@gmail.com>
> >>> AuthorDate: Fri Aug 12 11:54:54 2022 +0200
> >>>
> >>>   Basic: Fix issue where in scenario of providing object in
> >> dataProvider contains fields either null or undefined causes display
> >> [object Object] instead of empty string
> >>>
> >>>   Example of dp object: {firstName: "Piotr", lastName: null}
> >>> ---
> >>> .../royale/org/apache/royale/html/util/getLabelFromData.as     | 10
> >> +++++++++-
> >>> 1 file changed, 9 insertions(+), 1 deletion(-)
> >>>
> >>> diff --git
> >>
> a/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as
> >>
> b/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as
> >>> index 18aa873040..1581f0c21b 100644
> >>> ---
> >>
> a/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as
> >>> +++
> >>
> b/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as
> >>> @@ -65,9 +65,17 @@ package org.apache.royale.html.util
> >>>        }
> >>>
> >>>        var label:String = data["label"];
> >>> -        if(label != null){
> >>> +        if(label != null)
> >>> +        {
> >>>            return label;
> >>>        }
> >>> +
> >>> +        if (obj is IHasLabelField && (obj as
> IHasLabelField).labelField
> >> &&
> >>> +                (data[(obj as IHasLabelField).labelField] == null ||
> >> data[(obj as IHasLabelField).labelField] == undefined))
> >>> +        {
> >>> +            return "";
> >>> +        }
> >>> +
> >>>        return "" + data;
> >>>
> >>>    }
> >>>
> >>
> >
> >
> > --
> >
> > Piotr Zarzycki
>


-- 

Piotr Zarzycki

Re: [royale-asjs] branch develop updated: Basic: Fix issue where in scenario of providing object in dataProvider contains fields either null or undefined causes display [object Object] instead of empty string

Posted by Gabe Harbs <ha...@gmail.com>.
No. That’s probably not what you want. If there’s an empty string that should be used.

> On 12 Aug 2022, at 13:57, Piotr Zarzycki <pi...@gmail.com> wrote:
> 
> So maybe it's enough if I just made:
> 
> If (data[(obj as IHasLabelField).labelField]) ?
> 
> pt., 12 sie 2022 o 12:46 Gabe Harbs <ha...@gmail.com> napisał(a):
> 
>> FWIW: undefined == null is true. You don’t need to test for both null and
>> undefined.
>> 
>> 
>>>> On 12 Aug 2022, at 12:56, piotrz@apache.org wrote:
>>> 
>>> This is an automated email from the ASF dual-hosted git repository.
>>> 
>>> piotrz pushed a commit to branch develop
>>> in repository https://gitbox.apache.org/repos/asf/royale-asjs.git
>>> 
>>> 
>>> The following commit(s) were added to refs/heads/develop by this push:
>>>    new 9cf3840b26 Basic: Fix issue where in scenario of providing
>> object in dataProvider contains fields either null or undefined causes
>> display [object Object] instead of empty string
>>> 9cf3840b26 is described below
>>> 
>>> commit 9cf3840b2646f57a312a121db17a991766307fef
>>> Author: Piotr Zarzycki <pi...@gmail.com>
>>> AuthorDate: Fri Aug 12 11:54:54 2022 +0200
>>> 
>>>   Basic: Fix issue where in scenario of providing object in
>> dataProvider contains fields either null or undefined causes display
>> [object Object] instead of empty string
>>> 
>>>   Example of dp object: {firstName: "Piotr", lastName: null}
>>> ---
>>> .../royale/org/apache/royale/html/util/getLabelFromData.as     | 10
>> +++++++++-
>>> 1 file changed, 9 insertions(+), 1 deletion(-)
>>> 
>>> diff --git
>> a/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as
>> b/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as
>>> index 18aa873040..1581f0c21b 100644
>>> ---
>> a/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as
>>> +++
>> b/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as
>>> @@ -65,9 +65,17 @@ package org.apache.royale.html.util
>>>        }
>>> 
>>>        var label:String = data["label"];
>>> -        if(label != null){
>>> +        if(label != null)
>>> +        {
>>>            return label;
>>>        }
>>> +
>>> +        if (obj is IHasLabelField && (obj as IHasLabelField).labelField
>> &&
>>> +                (data[(obj as IHasLabelField).labelField] == null ||
>> data[(obj as IHasLabelField).labelField] == undefined))
>>> +        {
>>> +            return "";
>>> +        }
>>> +
>>>        return "" + data;
>>> 
>>>    }
>>> 
>> 
> 
> 
> -- 
> 
> Piotr Zarzycki

Re: [royale-asjs] branch develop updated: Basic: Fix issue where in scenario of providing object in dataProvider contains fields either null or undefined causes display [object Object] instead of empty string

Posted by Piotr Zarzycki <pi...@gmail.com>.
So maybe it's enough if I just made:

If (data[(obj as IHasLabelField).labelField]) ?

pt., 12 sie 2022 o 12:46 Gabe Harbs <ha...@gmail.com> napisał(a):

> FWIW: undefined == null is true. You don’t need to test for both null and
> undefined.
>
>
> > On 12 Aug 2022, at 12:56, piotrz@apache.org wrote:
> >
> > This is an automated email from the ASF dual-hosted git repository.
> >
> > piotrz pushed a commit to branch develop
> > in repository https://gitbox.apache.org/repos/asf/royale-asjs.git
> >
> >
> > The following commit(s) were added to refs/heads/develop by this push:
> >     new 9cf3840b26 Basic: Fix issue where in scenario of providing
> object in dataProvider contains fields either null or undefined causes
> display [object Object] instead of empty string
> > 9cf3840b26 is described below
> >
> > commit 9cf3840b2646f57a312a121db17a991766307fef
> > Author: Piotr Zarzycki <pi...@gmail.com>
> > AuthorDate: Fri Aug 12 11:54:54 2022 +0200
> >
> >    Basic: Fix issue where in scenario of providing object in
> dataProvider contains fields either null or undefined causes display
> [object Object] instead of empty string
> >
> >    Example of dp object: {firstName: "Piotr", lastName: null}
> > ---
> > .../royale/org/apache/royale/html/util/getLabelFromData.as     | 10
> +++++++++-
> > 1 file changed, 9 insertions(+), 1 deletion(-)
> >
> > diff --git
> a/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as
> b/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as
> > index 18aa873040..1581f0c21b 100644
> > ---
> a/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as
> > +++
> b/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as
> > @@ -65,9 +65,17 @@ package org.apache.royale.html.util
> >         }
> >
> >         var label:String = data["label"];
> > -        if(label != null){
> > +        if(label != null)
> > +        {
> >             return label;
> >         }
> > +
> > +        if (obj is IHasLabelField && (obj as IHasLabelField).labelField
> &&
> > +                (data[(obj as IHasLabelField).labelField] == null ||
> data[(obj as IHasLabelField).labelField] == undefined))
> > +        {
> > +            return "";
> > +        }
> > +
> >         return "" + data;
> >
> >     }
> >
>


-- 

Piotr Zarzycki

Re: [royale-asjs] branch develop updated: Basic: Fix issue where in scenario of providing object in dataProvider contains fields either null or undefined causes display [object Object] instead of empty string

Posted by Gabe Harbs <ha...@gmail.com>.
FWIW: undefined == null is true. You don’t need to test for both null and undefined.


> On 12 Aug 2022, at 12:56, piotrz@apache.org wrote:
> 
> This is an automated email from the ASF dual-hosted git repository.
> 
> piotrz pushed a commit to branch develop
> in repository https://gitbox.apache.org/repos/asf/royale-asjs.git
> 
> 
> The following commit(s) were added to refs/heads/develop by this push:
>     new 9cf3840b26 Basic: Fix issue where in scenario of providing object in dataProvider contains fields either null or undefined causes display [object Object] instead of empty string
> 9cf3840b26 is described below
> 
> commit 9cf3840b2646f57a312a121db17a991766307fef
> Author: Piotr Zarzycki <pi...@gmail.com>
> AuthorDate: Fri Aug 12 11:54:54 2022 +0200
> 
>    Basic: Fix issue where in scenario of providing object in dataProvider contains fields either null or undefined causes display [object Object] instead of empty string
> 
>    Example of dp object: {firstName: "Piotr", lastName: null}
> ---
> .../royale/org/apache/royale/html/util/getLabelFromData.as     | 10 +++++++++-
> 1 file changed, 9 insertions(+), 1 deletion(-)
> 
> diff --git a/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as b/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as
> index 18aa873040..1581f0c21b 100644
> --- a/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as
> +++ b/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/util/getLabelFromData.as
> @@ -65,9 +65,17 @@ package org.apache.royale.html.util
>         }
> 
>         var label:String = data["label"];
> -        if(label != null){
> +        if(label != null)
> +        {
>             return label;
>         }
> +
> +        if (obj is IHasLabelField && (obj as IHasLabelField).labelField &&
> +                (data[(obj as IHasLabelField).labelField] == null || data[(obj as IHasLabelField).labelField] == undefined))
> +        {
> +            return "";
> +        }
> +
>         return "" + data;
> 
>     }
>