Mailing-List: contact issues-help@flex.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@flex.apache.org
Date: Wed, 16 Apr 2014 10:20:15 +0000 (UTC)
From: "Mihai Chira (JIRA)" <jira@apache.org>
To: issues@flex.apache.org
Message-ID: <JIRA.12708726.1397643563750.123975.1397643615913@arcas>
In-Reply-To: <JIRA.12708726.1397643563750@arcas>
References: <JIRA.12708726.1397643563750@arcas>
Subject: [jira] [Created] (FLEX-34229) DataGridEditor doesn't remove the
 Event.DEACTIVATE listener on destroyItemEditor()
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit

Mihai Chira created FLEX-34229:
----------------------------------

             Summary: DataGridEditor doesn't remove the Event.DEACTIVATE listener on destroyItemEditor()
                 Key: FLEX-34229
                 URL: https://issues.apache.org/jira/browse/FLEX-34229
             Project: Apache Flex
          Issue Type: Bug
          Components: Spark: DataGrid
    Affects Versions: Apache Flex 4.12.0
            Reporter: Mihai Chira
            Priority: Minor
             Fix For: Apache Flex 4.13.0
         Attachments: _editorDeactivateHandler.fxp

Steps to reproduce:
1. Import and run attached project.
2. Set a breakpoint in DataGridEditor.deactivateHandler().
3. Double click on 'hello' to show the editor.
4. Click in the grid under 'hello' to remove the editor.
5. Click on the Remove Grid button.
6. Click anywhere outside the AIR app to de-focus it.
				
Actual result: the breakpoint is triggered, which means that the DataGridEditor is still responding to Event.DEACTIVATE even after the editor was closed, and even after the grid was removed.
Desired result: DataGridEditor.deactivateHandler() should only be called while an editor is open, as long as the grid itself is still on stage (NB the grid can be removed while the editor is still open, see FLEX-34216 for the problems that causes)

Expected result: the breakpoint is not triggered.

Notes:
-the second line of DataGridEditor.destroyItemEditor() looks like a typo: instead of addEventListener shouldn't it say removeEventListener(...)?


--
This message was sent by Atlassian JIRA
(v6.2#6252)