Bug - Fixed Item amounts not parsed correctly when transferring items to the closet/inventory

slyz

Developer
It looks like the transfers happen correctly, but Mafia updates the inventory/closet as if all of the items have been transfered, instead of the amount that has been actually transfered.
Code:
> closet list skeleton key

> inv skeleton key

skeleton key (8)

> closet put 1 skeleton key

Placing items into closet...
Requests complete.

> closet list skeleton key

skeleton key (8)

> inv skeleton key

> refresh all

Refreshing session data...
Loading character status...
Synchronizing moon data...
Retrieving character data...
Refreshing closet...
Examining consumables in closet...
Examining equipment in closet...
Examining miscellaneous items in closet...
Updating inventory...
Retrieving familiar data...
Familiar data retrieved.
Retrieving campground data...
Session data refreshed.

> closet list skeleton key

skeleton key

> inv skeleton key

skeleton key (7)

It looks like TransferItemRequest.transferItems() is called using HOWMANY_PATTERN by default. It should be called using QTY_PATTERN when "qty" is present in the urlString, or HOWMANY_PATTERN could be changed to:
Code:
(?:howmany|qty)\\d*=(\\d+)

Here is a patch changing HOWMANY_PATTERN which seems to work for closet/inventory transfers. I also tested sending an item by kmail.
 

Attachments

  • HOWMANY.patch
    795 bytes · Views: 31
Top