search

raduprv / Eternal-Lands

http://www.eternal-lands.com
Other
155 stars 56 forks source link

Last harvest for Xaquelina or Elle quest does not show up in counters #189

Open NetSysFire opened 2 years ago

NetSysFire commented 2 years ago

Steps to reproduce:

  1. Get your daily quest from Xaquelina or do an Elle one, I recommend the former.
  2. Make sure the session counter for that specific thing you are going to harvest is 0 (so it is easier to spot).
  3. She tells you to e.g get 60 Red Snapdragons from WS.
  4. Harvest them.
  5. After you get the popup that you got all of them, check your counters. It will be less than 60.

Expected behavior: The last harvest gets registered in the counters, too.

Spoke with radu about this yesterday, he said I should report it because it might be a client bug. So here it is.

Edit: I do not know how #ach behaves in this case. I am not sure if it registers it or not. I can not test because I harvested over 65001 plants.

pjbroad commented 2 years ago

I can confirm the issue. Looking at the message sequence, it looks to be a server issue. Here's some debug I added:

got new inventory item quantity=54 image_id=27
got new inventory item quantity=56 image_id=27
got new inventory item quantity=58 image_id=27
stopping harvest [You stopped harvesting.]
got new inventory item quantity=60 image_id=27

The "You stopped harvesting" messages, due to you completing the quest, comes before the last items arrive in the inventory. So the client does not recognise the new items were harvested.