flex-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alex Harui (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (FLEX-33987) FocusManager NPE
Date Tue, 10 Dec 2013 17:58:08 GMT

    [ https://issues.apache.org/jira/browse/FLEX-33987?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13844471#comment-13844471
] 

Alex Harui commented on FLEX-33987:
-----------------------------------

You have raised this NPE issue as "critical" so it must be reproducible in some way, but I
want to understand whether there is some other root problem.  I would think it is "impossible"
for this method to be called in a container with no focusable objects.  There are plenty of
places we could add more null checks and clean up the framework code, but doing so makes the
framework bigger and slower, so we want to make sure we spend our time and your time wisely,
that it actually makes a "critical" difference.

Otherwise, we should downgrade it to minor.

> FocusManager NPE
> ----------------
>
>                 Key: FLEX-33987
>                 URL: https://issues.apache.org/jira/browse/FLEX-33987
>             Project: Apache Flex
>          Issue Type: Bug
>    Affects Versions: Apache Flex 4.11.0
>            Reporter: Alexander Mazuruk
>            Priority: Critical
>
> FocusManager Null pointer exception occurs.
> This:
>     public function getNextFocusManagerComponent(
>                             backward:Boolean = false):IFocusManagerComponent
> 	{
> 		return getNextFocusManagerComponent2(backward, fauxFocus).displayObject as IFocusManagerComponent;
> 	}
> Must be rewritten as:
> public function getNextFocusManagerComponent(
>                             backward:Boolean = false):IFocusManagerComponent
> 	{
>         const focusInfo:FocusInfo = getNextFocusManagerComponent2(backward, fauxFocus);
>         return focusInfo ? focusInfo.displayObject as IFocusManagerComponent : null;
> 	}



--
This message was sent by Atlassian JIRA
(v6.1.4#6159)

Mime
View raw message