You don't lose current XP if you die while you have XP gain disabled #1021

Closed
opened 2024-09-27 23:32:33 +00:00 by loysaranne · 3 comments
loysaranne commented 2024-09-27 23:32:33 +00:00 (Migrated from github.com)

Reproduction steps

1. Fill some of your current XP bar
2. Disable XP gain in options
3. Die

Expected results

The current XP should move to pool of unsaved experience

Actual results

Your current XP moves to the pool of unsaved experience and gets applied back to the current XP.

Bug prevalence

Every time

Description

According to the double message of "added to pool of unsaved experience" that the XP is correctly added to pool at first, but then it is applied back to current XP for some reason.

https://pastebin.com/yc7UU8wA

### Reproduction steps ```bash 1. Fill some of your current XP bar 2. Disable XP gain in options 3. Die ``` ### Expected results ```bash The current XP should move to pool of unsaved experience ``` ### Actual results ```bash Your current XP moves to the pool of unsaved experience and gets applied back to the current XP. ``` ### Bug prevalence Every time ### Description ```bash According to the double message of "added to pool of unsaved experience" that the XP is correctly added to pool at first, but then it is applied back to current XP for some reason. ``` ### Report link https://pastebin.com/yc7UU8wA
kajoriEU commented 2024-09-30 08:17:50 +00:00 (Migrated from github.com)

Confirmed.

{B71B1A79-819B-4E4F-A1AB-036DBD1EEDD9}

Once the XP gains are disabled, XP moves to pool and back to current XP on death.

Confirmed. ![{B71B1A79-819B-4E4F-A1AB-036DBD1EEDD9}](https://github.com/user-attachments/assets/2999597e-877c-4f21-8130-54ca7b024baa) Once the XP gains are disabled, XP moves to pool and back to current XP on death.
kajori added this to the Season 2 Launch project 2024-12-15 18:21:00 +00:00
drake added the
state
fixed
label 2024-12-20 16:33:52 +00:00

This should now be fixed

This should now be fixed
drake self-assigned this 2024-12-20 16:34:25 +00:00
Member

Can confirm that this is now fixed and works as expected :)

With Xp Gains Disabled:

Can confirm that this is now fixed and works as expected :) With Xp Gains Disabled: <video src="/attachments/50189975-5b22-430d-869e-b52394295157" title="2024-12-20 19-33-45.mp4" controls></video>
Sign in to join this conversation.
No milestone
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: prk/issues#1021
No description provided.