This is an automated email from the ASF dual-hosted git repository.

carlosrovira 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 79a1aa2  Clean names and separate add item from List and Table
79a1aa2 is described below

commit 79a1aa2a9ac7afdb3edc2b82435bab2be9b2bd01
Author: Carlos Rovira <carlosrov...@apache.org>
AuthorDate: Thu Aug 23 20:54:53 2018 +0200

    Clean names and separate add item from List and Table
---
 .../JewelExample/src/main/royale/ListPlayGround.mxml   |  4 ++--
 .../JewelExample/src/main/royale/TablePlayGround.mxml  |  2 +-
 .../Jewel/src/main/resources/jewel-manifest.xml        |  4 ++--
 ...tData.as => AddListItemRendererForArrayListData.as} | 18 +++++++++---------
 .../AddTableItemRendererForArrayListData.as            | 12 +++++-------
 ...ata.as => RemoveAllItemRendererForArrayListData.as} | 18 +++++++++---------
 6 files changed, 28 insertions(+), 30 deletions(-)

diff --git a/examples/royale/JewelExample/src/main/royale/ListPlayGround.mxml 
b/examples/royale/JewelExample/src/main/royale/ListPlayGround.mxml
index ec24b0d..6b60d03 100644
--- a/examples/royale/JewelExample/src/main/royale/ListPlayGround.mxml
+++ b/examples/royale/JewelExample/src/main/royale/ListPlayGround.mxml
@@ -112,10 +112,10 @@ limitations under the License.
                                                        sourceID="listModel"
                                                        
sourcePropertyName="iconListData"
                                                        
destinationPropertyName="dataProvider" />
-                                               
<j:DynamicAddItemRendererForArrayListData/>
+                                               
<j:AddListItemRendererForArrayListData/>
                                                
<j:DynamicRemoveItemRendererForArrayListData/>
                                                
<j:DynamicUpdateItemRendererForArrayListData/>
-                                               
<j:DynamicRemoveAllItemRendererForArrayListData/>
+                                               
<j:RemoveAllItemRendererForArrayListData/>
                                        </j:beads>
                                </j:List>
                                <j:Button text="Add item" click="addItem()"/>
diff --git a/examples/royale/JewelExample/src/main/royale/TablePlayGround.mxml 
b/examples/royale/JewelExample/src/main/royale/TablePlayGround.mxml
index de9c76a..6dd838a 100644
--- a/examples/royale/JewelExample/src/main/royale/TablePlayGround.mxml
+++ b/examples/royale/JewelExample/src/main/royale/TablePlayGround.mxml
@@ -188,7 +188,7 @@ limitations under the License.
                                <j:AddTableItemRendererForArrayListData/>
                                <j:DynamicRemoveItemRendererForArrayListData/>
                                <j:DynamicUpdateItemRendererForArrayListData/>
-                               
<j:DynamicRemoveAllItemRendererForArrayListData/>
+                               <j:RemoveAllItemRendererForArrayListData/>
                        </j:beads>
                </j:Table>
 
diff --git a/frameworks/projects/Jewel/src/main/resources/jewel-manifest.xml 
b/frameworks/projects/Jewel/src/main/resources/jewel-manifest.xml
index 7820f6f..e5691b9 100644
--- a/frameworks/projects/Jewel/src/main/resources/jewel-manifest.xml
+++ b/frameworks/projects/Jewel/src/main/resources/jewel-manifest.xml
@@ -120,10 +120,10 @@
     <component id="TableItemRendererFactoryForCollectionView" 
class="org.apache.royale.jewel.beads.itemRenderers.TableItemRendererFactoryForCollectionView"/>
     
 
-    <component id="DynamicAddItemRendererForArrayListData" 
class="org.apache.royale.jewel.beads.itemRenderers.DynamicAddItemRendererForArrayListData"
 />
+    <component id="AddListItemRendererForArrayListData" 
class="org.apache.royale.jewel.beads.itemRenderers.AddListItemRendererForArrayListData"
 />
     <component id="DynamicRemoveItemRendererForArrayListData" 
class="org.apache.royale.jewel.beads.itemRenderers.DynamicRemoveItemRendererForArrayListData"
 />
     <component id="DynamicUpdateItemRendererForArrayListData" 
class="org.apache.royale.jewel.beads.itemRenderers.DynamicUpdateItemRendererForArrayListData"
 />
-    <component id="DynamicRemoveAllItemRendererForArrayListData" 
class="org.apache.royale.jewel.beads.itemRenderers.DynamicRemoveAllItemRendererForArrayListData"
 />
+    <component id="RemoveAllItemRendererForArrayListData" 
class="org.apache.royale.jewel.beads.itemRenderers.RemoveAllItemRendererForArrayListData"
 />
     
     <component id="AddTableItemRendererForArrayListData" 
class="org.apache.royale.jewel.beads.itemRenderers.AddTableItemRendererForArrayListData"
 />
     <!-- <component id="JewelLabelViewBead" 
class="org.apache.royale.jewel.beads.views.JewelLabelViewBead"/> -->
diff --git 
a/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/DynamicAddItemRendererForArrayListData.as
 
b/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/AddListItemRendererForArrayListData.as
similarity index 94%
rename from 
frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/DynamicAddItemRendererForArrayListData.as
rename to 
frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/AddListItemRendererForArrayListData.as
index e311587..c4bc590 100644
--- 
a/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/DynamicAddItemRendererForArrayListData.as
+++ 
b/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/AddListItemRendererForArrayListData.as
@@ -35,15 +35,15 @@ package org.apache.royale.jewel.beads.itemRenderers
        import org.apache.royale.utils.loadBeadFromValuesManager;
 
     /**
-        * Handles the adding of an itemRenderer once the corresponding datum 
has been added
+        * Handles the adding of an itemRenderer in a List component once the 
corresponding datum has been added
         * from the IDataProviderModel.
         *
         *  @langversion 3.0
         *  @playerversion Flash 10.2
         *  @playerversion AIR 2.6
-        *  @productversion Royale 0.9.0
+        *  @productversion Royale 0.9.3
         */
-       public class DynamicAddItemRendererForArrayListData implements IBead
+       public class AddListItemRendererForArrayListData implements IBead
        {
                /**
                 * Constructor
@@ -51,9 +51,9 @@ package org.apache.royale.jewel.beads.itemRenderers
                 *  @langversion 3.0
                 *  @playerversion Flash 10.2
                 *  @playerversion AIR 2.6
-                *  @productversion Royale 0.9.0
+                *  @productversion Royale 0.9.3
                 */
-               public function DynamicAddItemRendererForArrayListData()
+               public function AddListItemRendererForArrayListData()
                {
                }
 
@@ -67,7 +67,7 @@ package org.apache.royale.jewel.beads.itemRenderers
                 *  @langversion 3.0
                 *  @playerversion Flash 10.2
                 *  @playerversion AIR 2.6
-                *  @productversion Royale 0.9.0
+                *  @productversion Royale 0.9.3
                 */
                public function set strand(value:IStrand):void
                {
@@ -115,7 +115,7 @@ package org.apache.royale.jewel.beads.itemRenderers
                 *  @langversion 3.0
                 *  @playerversion Flash 10.2
                 *  @playerversion AIR 2.6
-                *  @productversion Royale 0.9.0
+                *  @productversion Royale 0.9.3
                 */
                protected function handleItemAdded(event:CollectionEvent):void
                {
@@ -145,7 +145,7 @@ package org.apache.royale.jewel.beads.itemRenderers
                 *  @langversion 3.0
                 *  @playerversion Flash 10.2
                 *  @playerversion AIR 2.6
-                *  @productversion Royale 0.9.0
+                *  @productversion Royale 0.9.3
                 */
                public function get dataProviderModel(): IDataProviderModel
                {
@@ -164,7 +164,7 @@ package org.apache.royale.jewel.beads.itemRenderers
                 *  @langversion 3.0
                 *  @playerversion Flash 10.2
                 *  @playerversion AIR 2.6
-                *  @productversion Royale 0.9.0
+                *  @productversion Royale 0.9.3
                 */
                public function get itemRendererParent():IItemRendererParent
                {
diff --git 
a/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/AddTableItemRendererForArrayListData.as
 
b/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/AddTableItemRendererForArrayListData.as
index eaee0cd..d7c29ad 100644
--- 
a/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/AddTableItemRendererForArrayListData.as
+++ 
b/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/AddTableItemRendererForArrayListData.as
@@ -47,7 +47,7 @@ package org.apache.royale.jewel.beads.itemRenderers
         *  @langversion 3.0
         *  @playerversion Flash 10.2
         *  @playerversion AIR 2.6
-        *  @productversion Royale 0.9.0
+        *  @productversion Royale 0.9.3
         */
        public class AddTableItemRendererForArrayListData implements IBead
        {
@@ -57,7 +57,7 @@ package org.apache.royale.jewel.beads.itemRenderers
                 *  @langversion 3.0
                 *  @playerversion Flash 10.2
                 *  @playerversion AIR 2.6
-                *  @productversion Royale 0.9.0
+                *  @productversion Royale 0.9.3
                 */
                public function AddTableItemRendererForArrayListData()
                {
@@ -73,7 +73,7 @@ package org.apache.royale.jewel.beads.itemRenderers
                 *  @langversion 3.0
                 *  @playerversion Flash 10.2
                 *  @playerversion AIR 2.6
-                *  @productversion Royale 0.9.0
+                *  @productversion Royale 0.9.3
                 */
                public function set strand(value:IStrand):void
                {
@@ -121,7 +121,7 @@ package org.apache.royale.jewel.beads.itemRenderers
                 *  @langversion 3.0
                 *  @playerversion Flash 10.2
                 *  @playerversion AIR 2.6
-                *  @productversion Royale 0.9.0
+                *  @productversion Royale 0.9.3
                 */
                protected function handleItemAdded(event:CollectionEvent):void
                {
@@ -131,7 +131,6 @@ package org.apache.royale.jewel.beads.itemRenderers
             var ir:ITextItemRenderer;
 
                        var index:int = event.index * model.columns.length;
-                       var rowIndex:int = index;
                        for(var j:int = 0; j < model.columns.length; j++)
                        {
                                column = model.columns[j] as TableColumn;
@@ -161,7 +160,6 @@ package org.apache.royale.jewel.beads.itemRenderers
                        // update the index values in the itemRenderers to 
correspond to their shifted positions.
                        var n:int = tbody.numElements;
                        var d:DataItemRenderer;
-                       index = event.index * model.columns.length;
                        for (var i:int = event.index; i < n; i++)
                        {
                                d = tbody.getItemRendererForIndex(i) as 
DataItemRenderer;
@@ -180,7 +178,7 @@ package org.apache.royale.jewel.beads.itemRenderers
                 *  @langversion 3.0
                 *  @playerversion Flash 10.2
                 *  @playerversion AIR 2.6
-                *  @productversion Royale 0.9.0
+                *  @productversion Royale 0.9.3
                 */
                public function get itemRendererParent():IItemRendererParent
                {
diff --git 
a/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/DynamicRemoveAllItemRendererForArrayListData.as
 
b/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/RemoveAllItemRendererForArrayListData.as
similarity index 91%
rename from 
frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/DynamicRemoveAllItemRendererForArrayListData.as
rename to 
frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/RemoveAllItemRendererForArrayListData.as
index 409ddf0..1707f4b 100644
--- 
a/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/DynamicRemoveAllItemRendererForArrayListData.as
+++ 
b/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/RemoveAllItemRendererForArrayListData.as
@@ -30,14 +30,14 @@ package org.apache.royale.jewel.beads.itemRenderers
 
        /**
         * Handles the removal of all itemRenderers once the all items has been 
removed
-        * from the IDataProviderModel.
+        * from the IDataProviderModel. This works the same for List and Table 
components
         *
         *  @langversion 3.0
         *  @playerversion Flash 10.2
         *  @playerversion AIR 2.6
-        *  @productversion Royale 0.9.0
+        *  @productversion Royale 0.9.3
         */
-       public class DynamicRemoveAllItemRendererForArrayListData implements 
IBead
+       public class RemoveAllItemRendererForArrayListData implements IBead
        {
                /**
                 * Constructor
@@ -45,9 +45,9 @@ package org.apache.royale.jewel.beads.itemRenderers
                 *  @langversion 3.0
                 *  @playerversion Flash 10.2
                 *  @playerversion AIR 2.6
-                *  @productversion Royale 0.9.0
+                *  @productversion Royale 0.9.3
                 */
-               public function DynamicRemoveAllItemRendererForArrayListData()
+               public function RemoveAllItemRendererForArrayListData()
                {
                }
 
@@ -59,7 +59,7 @@ package org.apache.royale.jewel.beads.itemRenderers
                 *  @langversion 3.0
                 *  @playerversion Flash 10.2
                 *  @playerversion AIR 2.6
-                *  @productversion Royale 0.9.0
+                *  @productversion Royale 0.9.3
                 */
                public function set strand(value:IStrand):void
                {
@@ -105,7 +105,7 @@ package org.apache.royale.jewel.beads.itemRenderers
                 *  @langversion 3.0
                 *  @playerversion Flash 10.2
                 *  @playerversion AIR 2.6
-                *  @productversion Royale 0.9.0
+                *  @productversion Royale 0.9.3
                 */
                protected function 
handleAllItemsRemoved(event:CollectionEvent):void
                {
@@ -129,7 +129,7 @@ package org.apache.royale.jewel.beads.itemRenderers
                 *  @langversion 3.0
                 *  @playerversion Flash 10.2
                 *  @playerversion AIR 2.6
-                *  @productversion Royale 0.9.0
+                *  @productversion Royale 0.9.3
                 */
                public function get dataProviderModel(): IDataProviderModel
                {
@@ -148,7 +148,7 @@ package org.apache.royale.jewel.beads.itemRenderers
                 *  @langversion 3.0
                 *  @playerversion Flash 10.2
                 *  @playerversion AIR 2.6
-                *  @productversion Royale 0.9.0
+                *  @productversion Royale 0.9.3
                 */
                public function get itemRendererParent():IItemRendererParent
                {

Reply via email to