You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flex.apache.org by "Andrii Radistao (JIRA)" <ji...@apache.org> on 2014/10/30 17:03:34 UTC
[jira] [Comment Edited] (FLEX-34630)
spark.components.Alert#partRemoved() implementation makes wrong
super.partAdded() call
[ https://issues.apache.org/jira/browse/FLEX-34630?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14190273#comment-14190273 ]
Andrii Radistao edited comment on FLEX-34630 at 10/30/14 4:03 PM:
------------------------------------------------------------------
Spark Alert bug demonstration sources attached
was (Author: radistao):
Spark Alert bug demonstration
> spark.components.Alert#partRemoved() implementation makes wrong super.partAdded() call
> --------------------------------------------------------------------------------------
>
> Key: FLEX-34630
> URL: https://issues.apache.org/jira/browse/FLEX-34630
> Project: Apache Flex
> Issue Type: Bug
> Affects Versions: Apache Flex 4.13.0
> Environment: Flash Player 15.0.0.189 (debug version), Opera, Windows 7, SDK 4.13.0 build 20140701 and WIN_flex4.13.0-air15.0.0.249
> Reporter: Andrii Radistao
> Labels: Alert, Skin, partAdded, partRemoved, spark
> Attachments: TestSparkAlert.zip
>
>
> _partRemoved_ implementation for {{spark.components.Alert}} probably has wrong super method call: _partAdded_ instead of _partRemoved_.
> {code:title=Alert.as: 642|borderStyle=solid}
> override protected function partRemoved(partName:String, instance:Object):void {
> super.partAdded(partName, instance);
> if (partName == "buttonGroup") {
> destroyButtons(Group(instance));
> }
> }
> {code}
> *This implementation cause an exception if you change component's skin at runtime* (see the attachment):
> [Fault] exception, information=ReferenceError: Error #1069: Property autoLayout not found on Number and there is no default value.
> at spark.components::SkinnableContainer/partAdded()[/Users/aharui/release4.13.0/frameworks/projects/spark/src/spark/components/SkinnableContainer.as:970]
> at spark.components::Panel/partAdded()[/Users/aharui/release4.13.0/frameworks/projects/spark/src/spark/components/Panel.as:554]
> at spark.components::Alert/partRemoved()[/Users/aharui/release4.13.0/frameworks/projects/experimental/src/spark/components/Alert.as:643]
> at spark.components.supportClasses::SkinnableComponent/clearSkinParts()[/Users/aharui/release4.13.0/frameworks/projects/spark/src/spark/components/supportClasses/SkinnableComponent.as:779]
> at spark.components.supportClasses::SkinnableComponent/detachSkin()[/Users/aharui/release4.13.0/frameworks/projects/spark/src/spark/components/supportClasses/SkinnableComponent.as:813]
> at spark.components.supportClasses::SkinnableComponent/validateSkinChange()[/Users/aharui/release4.13.0/frameworks/projects/spark/src/spark/components/supportClasses/SkinnableComponent.as:453]
> at spark.components.supportClasses::SkinnableComponent/commitProperties()[/Users/aharui/release4.13.0/frameworks/projects/spark/src/spark/components/supportClasses/SkinnableComponent.as:472]
> at mx.core::UIComponent/validateProperties()[/Users/aharui/release4.13.0/frameworks/projects/framework/src/mx/core/UIComponent.as:8751]
> at mx.managers::LayoutManager/validateProperties()[/Users/aharui/release4.13.0/frameworks/projects/framework/src/mx/managers/LayoutManager.as:605]
> at mx.managers::LayoutManager/doPhasedInstantiation()[/Users/aharui/release4.13.0/frameworks/projects/framework/src/mx/managers/LayoutManager.as:821]
> at mx.managers::LayoutManager/doPhasedInstantiationCallback()[/Users/aharui/release4.13.0/frameworks/projects/framework/src/mx/managers/LayoutManager.as:1188]
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)