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 <piotrzarzyck...@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 <harbs.li...@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 <harbs.li...@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 <
>> piotrzarzyck...@gmail.com> wrote:
>> >>
>> >> No. In fields there is null or undefined - what do you suggest ?
>> >>
>> >> pt., 12 sie 2022 o 14:38 Gabe Harbs <harbs.li...@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 <piotrzarzyck...@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 <harbs.li...@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, pio...@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 <piotrzarzyck...@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

Reply via email to