wiln / flexlib

Automatically exported from code.google.com/p/flexlib
0 stars 0 forks source link

AbstractEntryLayout saveItemWithRow function sets wrong values to rowItem #153

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Just use the function with relevant arguments
2.
3.

What is the expected output? What do you see instead?
Expected: RowLocatorItem will be created with passed arguments row and rowItem.

Observed: RowLocatorItem uses the row argument for row and for rowItem.

What version of the product are you using? On what operating system?
Version 2.4 on WinXP

Please provide any additional information below.

so, instead of:
protected function saveItemWithRow( item:EntryLayoutItem, row:Number,
rowItem : Number ) : void
               {
                       var rowLocatorItem : RowLocatorItem = new
RowLocatorItem();
                       rowLocatorItem.row = row;
                       rowLocatorItem.rowItem = row;
                       setRowLocatorItem( item, rowLocatorItem );
               }

it should be:
protected function saveItemWithRow( item : EntryLayoutItem, row:Number,
rowItem : Number ) : void
               {
                       var rowLocatorItem : RowLocatorItem = new
RowLocatorItem();
                       rowLocatorItem.row = row;
                       rowLocatorItem.rowItem = rowItem;
                       setRowLocatorItem( item, rowLocatorItem );
               }

Original issue reported on code.google.com by sefi.ni...@gmail.com on 19 Aug 2008 at 6:51

GoogleCodeExporter commented 9 years ago

Original comment by dmcc...@gmail.com on 8 Jan 2009 at 5:51

GoogleCodeExporter commented 9 years ago
Fixed @Rev. 158

Original comment by sefi.ni...@gmail.com on 25 Jan 2009 at 7:59