Skip to content

[Bug Fix] Evolving Items SayLink#8

Merged
Valorith merged 1 commit intomasterfrom
fix/evolving-saylink-5017
Oct 6, 2025
Merged

[Bug Fix] Evolving Items SayLink#8
Valorith merged 1 commit intomasterfrom
fix/evolving-saylink-5017

Conversation

@Valorith
Copy link
Copy Markdown
Owner

@Valorith Valorith commented Oct 5, 2025

Backports EQEmu#5017 into this fork.
Credit to Nekkola

Description

An outstanding item from PR4992 was item EQEmu#4. An issue where creating a saylink from an evolving item would inject a random character as character EQEmu#1.

Fixes the above noted item from PR4992. Saylinks should work correctly for evolving items.
In testing, noticed that when selling an evolving item with progression, if the player had the merchant window open and clicked inspect, the progression would show incorrectly. This has been fixed to show progression at 0. This was not present if the merchant window was opened after selling the item.
Similar issue with parceling an evolving item with progression. If the player receiving the item had the parcel window open, the inspect would show the incorrect progression. This was not present if the player opens the parcel window after receiving the item.

image image Clients tested: RoF2

Tested ok
@Valorith Valorith merged commit 3000471 into master Oct 6, 2025
@Valorith Valorith deleted the fix/evolving-saylink-5017 branch October 6, 2025 00:01
@Valorith Valorith restored the fix/evolving-saylink-5017 branch October 6, 2025 00:01
Valorith pushed a commit that referenced this pull request Mar 22, 2026
47 test cases across 15 sections covering all evolution types (XP, kill
count, mob race, zone), activation/timer mechanics, evolution completion,
augment handling, XP transfer window, merchant/parcel interactions,
persistence, GM commands, edge cases, and regression tests for PRs EQEmu#4607,
EQEmu#4992, #8, and #98. Includes ready-to-run SQL setup for 5 test item
families with low thresholds for fast manual testing.

https://claude.ai/code/session_017wrKmjuMTicYybGCRiBBFG
@Valorith Valorith deleted the fix/evolving-saylink-5017 branch March 27, 2026 00:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants