Could you please describe some more details about this
patch. Because in my opinion this is wrong.
1.
if (item.getOwner() != null && item.getOwner()!=this) {
doesn't throw exception when item already has the owner of
this form and according to the specification have to:
"IllegalStateException - if the item is already owned by a
container" - from Form.append(Item)
2.
if (itemNum!=0 && (itemNum < 0 || itemNum >= numOfItems)) {
did you changed that one for situation when numOfItems==0?
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
patch for microedition form
Logged In: YES
user_id=248825
Could you please describe some more details about this
patch. Because in my opinion this is wrong.
1.
if (item.getOwner() != null && item.getOwner()!=this) {
doesn't throw exception when item already has the owner of
this form and according to the specification have to:
"IllegalStateException - if the item is already owned by a
container" - from Form.append(Item)
2.
if (itemNum!=0 && (itemNum < 0 || itemNum >= numOfItems)) {
did you changed that one for situation when numOfItems==0?