History | Log In     View a printable version of the current page.  
Issue Details (XML)

Key: SDK-16742
Type: Patch Submission Patch Submission
Status: Rejected Rejected
Resolution: Incomplete
Priority: None None
Assignee: Hans Muller
Reporter: Brian Curtis
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Flex SDK
ComboBase::selectedIndex does not handle ItemPendingError (SDK-16741)

Handle ItemPendingError in ComboBase::selectedIndex

Created: 09/06/08 08:57 PM   Updated: 04/03/09 06:51 PM
Security Level: Public (All JIRA Users )

File Attachments: 1. Text File avoiderror_cb_ssi.patch (3 kb)
2. Text File avoiderror_cb_ssi.patch (3 kb)
3. Text File avoiderror_cb_ssi.patch (3 kb)
4. Text File avoiderror_cb_ssi.patch (1.0 kb)
5. Text File sdk16742.patch (3 kb)


Resolved by: Deepa Subramaniam
Parent Component: mx: ComboBox


 Description  « Hide
This patch replaces the use of the ComboBase::iterator with a local IViewCursor instance to get the first item. By using this method, we don't need to worry about the state of the ComboBase::iterator if an ItemPendingError is received (which could result in an endless loop of ItemPendingError depending on the size of the collections page cache). Further, I have added a try-catch block for ItemPendingError. When the item is available, the another attempt to set the index is made.

 All   Comments      Sort Order:
Brian Curtis - [09/07/08 02:58 PM ]
Fix parameter passing not caught by the compiler.

Lauren Park - [09/10/08 07:09 PM ]
To Deepa for review.

Brian Curtis - [09/15/08 11:29 PM ]
Patch updated:

Moved setting of _selectedIndex to inside the if-else only so that when an ItemPendingError is caught, the selectedIndex is not already set to the value desired when the error was encountered. This plays a particularly important role with validation on change.

Deepa Subramaniam - [01/21/09 12:55 PM ]
To Jason for review and integration into the 3.4 release.

Brian Curtis - [02/06/09 07:40 PM ]
Patch updated to allow setting selectedIndex = -1.

Deepa Subramaniam - [03/09/09 12:54 PM ]
To Hans for patch review, hopefully during i11 bugfixing week.

Deepa Subramaniam - [03/27/09 06:11 PM ]
Patch under active review.

Deepa Subramaniam - [04/03/09 06:51 PM ]
We've retired the associated bug and as such will not be accepting the patch submission since the implementation will be different in the Spark components.