#518 DoF client mender / repair

Closed
opened 11 months ago by image · 1 comments
image commented 11 months ago

Right now sending the merchant packet for the repair/mender window alone in DoF does not work, displays a blank window.

Workaround is to send buy window (type 0) first, then send a identical packet (type 16) which populates the repair panel with the items.

Repair All works, Repair a single item does not.

This workaround however avoids crashes which was the main goal, going from a normal merchant to mender would cause the merchants list to display, clicking the UI would previously crash the client.

redo how the setmerchant pointer is handled we don't want loose spawn ptrs.

Right now sending the merchant packet for the repair/mender window alone in DoF does not work, displays a blank window. Workaround is to send buy window (type 0) first, then send a identical packet (type 16) which populates the repair panel with the items. Repair All works, Repair a single item does not. This workaround however avoids crashes which was the main goal, going from a normal merchant to mender would cause the merchants list to display, clicking the UI would previously crash the client. redo how the setmerchant pointer is handled we don't want loose spawn ptrs.
image commented 6 months ago
Collaborator

the issue with repairing a single item was due to the no repair flag being improperly checked so that is against both clients..

the merchant window "behaving" correctly will also be patched next... was due to the fact the "type" of merchant has changed, where-as it was (0x60)96 in newer clients (AoM and such) "classic" needs it to be (0x70)112, 0x10(16) is also set.

the issue with repairing a single item was due to the no repair flag being improperly checked so that is against both clients.. the merchant window "behaving" correctly will also be patched next... was due to the fact the "type" of merchant has changed, where-as it was (0x60)96 in newer clients (AoM and such) "classic" needs it to be (0x70)112, 0x10(16) is also set.
Sign in to join this conversation.
Loading...
Cancel
Save
There is no content yet.