|Anonymous | Login||2022-10-07 02:16 UTC|
|My View | View Issues | Change Log | Roadmap|
|View Issue Details|
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0001945||VCMI||Mechanics - Objects||public||2014-11-11 11:41||2016-01-17 12:35|
|Platform||Tablet ASUS TF201||OS||Android||OS Version||4.1.1.|
|Target Version||Fixed in Version||0.97c|
|Summary||0001945: Crash when learning skill from University or Magic Univercity|
|Description||Game crash when trying to learnqa new skill from Univercity on adventure map or from Conflux's Magic Univercity. Any hero, any map, any time, always.|
After clicking buy, you exit chosen skill window and see Univercity window again, but skill doesn't turn "gold", shortly after buying skill crash happens.
|Steps To Reproduce||Click to buy skill in certain building.|
|Tags||No tags attached.|
|Confirmed, I get this crash as well|
@Warmonger if you not looking into this one I'll fix this one. :-)
At moment problem is pretty easy:
1 - First of all server-side always get SecondarySkill::PATHFINDING when player attempt to buy skill.
2 - There also possible some issue with "availableItemsIds" check.
Funny, but it's exactly same bug like with 0001899.
There is lambda function that use local variable which end up being garbage, but likely in process of network request it's become 0. Also once fixed it's looks like availableItemsIds check start to work as expected too.
|At some point GUI was reworked, possibly more such lambdas were introduced.|
Though I'm checked source code and it's looks like there no other dangerous lambdas like this one. As far as I understand it's should be safe to use non-local variables by reference in lambdas, but would be great if someone more experienced can check exist usages.
Will send pull request once check everything work as expected.
So I'm not completely sure what possible caused crash, so I'm decide mark this one as related to this one as related to 0001991 and not as duplicate. Though I suppose Android may be more sensitive to such corruption.
Anyway this one should be fixed in git now:
|2014-11-11 11:41||Sebt||New Issue|
|2014-11-11 11:41||Sebt||Status||new => assigned|
|2014-11-11 11:41||Sebt||Assigned To||=> Warmonger|
|2014-11-17 11:15||Noui||Note Added: 0005118|
|2015-01-07 11:11||SXX||Relationship added||has duplicate 0001991|
|2015-01-07 11:13||SXX||Note Added: 0005356|
|2015-01-07 11:14||SXX||Assigned To||Warmonger => SXX|
|2015-01-07 11:50||SXX||Note Added: 0005357|
|2015-01-07 11:59||Warmonger||Note Added: 0005358|
|2015-01-07 12:06||SXX||Note Added: 0005359|
|2015-01-07 15:15||SXX||Relationship added||related to 0001899|
|2015-01-07 15:41||SXX||Relationship replaced||related to 0001991|
|2015-01-07 15:48||SXX||Note Added: 0005363|
|2015-01-07 15:48||SXX||Status||assigned => resolved|
|2015-01-07 15:48||SXX||Fixed in Version||=> 0.97c|
|2015-01-07 15:48||SXX||Resolution||open => fixed|
|2016-01-17 12:35||SXX||Status||resolved => closed|
|Copyright © 2000 - 2022 MantisBT Team|