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;
> 
>     }
> 

Reply via email to