WerWolv / ImHex

🔍 A Hex Editor for Reverse Engineers, Programmers and people who value their retinas when working at 3 AM.
https://imhex.werwolv.net
GNU General Public License v2.0
45.04k stars 1.95k forks source link

[Bug] "Edit the hex" achievement doesn't unlock #1846

Open BobSmun opened 3 months ago

BobSmun commented 3 months ago

Operating System

Windows

What's the issue you encountered?

Modifying a byte by double-clicking on it and then entering the new value does not unlock the "Edit the hex" achievement

How can the issue be reproduced?

Alternatively, use enter (not num-pad enter) to start or finish the edit

ImHex Version

1.35.4

ImHex Build Type

Installation type

MSI

Additional context?

In the code, it looks like EventPatchCreated is subscribed to, to unlock the achievement, but a search does not show the event being posted / raised anywhere

paxcut commented 3 months ago

I can confirm issue on dev branch 1.36.0-wip as well, but I think EventPatchCreated may be for the ROM hacks achievement. If so and it is not posted maybe that one doesn't work either.

BobSmun commented 3 months ago

EventPatchCreated certainly sounds like it would be intended for the 'ROM hacks' achievement, but it unlocks "hex.builtin.achievement.hex_editor.modify_byte.name", whereas the exportIPSPatch and exportIPS32Patch functions call unlockAchievement (with "hex.builtin.achievement.hex_editor.create_patch.name") directly