Skip to content

[sql] Fixed Greater Pugil Aggro#10880

Open
PocketDev89 wants to merge 1 commit into
LandSandBoat:basefrom
PocketDev89:fix-greater-pugil-aggro
Open

[sql] Fixed Greater Pugil Aggro#10880
PocketDev89 wants to merge 1 commit into
LandSandBoat:basefrom
PocketDev89:fix-greater-pugil-aggro

Conversation

@PocketDev89

Copy link
Copy Markdown
Contributor

I affirm:

  • I understand that if I do not agree to the following points by completing the checkboxes my PR will be ignored.
  • I understand I should leave resolving conversations to the LandSandBoat team so that reviewers won't miss what was said.
  • I have read and understood the Contributing Guide and the Code of Conduct.
  • I have tested my code and the things my code has changed since the last commit in the PR and will test after any later commits.

What does this pull request do?

Corrects the aggro and linking behavior of Greater Pugils.

Pool 1807 (Greater_Pugil) had aggro=0 and links=1. With aggro=0, CMobController::CanAggroTarget returns early, so these Pugils could not aggro by any means. Retail behavior is aggressive, aggros by sound, and does not link.

This pool is shared by the Greater Pugils in Gusgen Mines, Korroloka Tunnel, and Qufim Island, so all three are corrected. Their species (38, Pugil) already has detects=2 (hearing), so once aggro is enabled they aggro to sound as expected.

Set pool 1807 to aggro=1, links=0.

Sources / reasoning:

  • BG-Wiki: Greater Pugil lists the mob as aggressive and detecting by Sound.
  • The fished Greater Pugil pools already in the database (5868, 7311) already carry aggro=1, links=0, so this change brings the dungeon pool in line with the existing data.
  • Link status is not explicitly documented on the wiki; links=0 matches the other Greater Pugil pools and the reported retail "Aggro, No Link" behavior.

Steps to test these changes

Apply the updated mob_pools data and enter a zone that uses pool 1807 (Qufim Island shown here):

!zone 126
!gotoname Greater_Pugil 1

Lower level to Pugil so it is not Too Weak (or use /heal, which aggros by sound regardless of level), and make sure you are not in GM/hidden status.

  • Before: the Greater Pugil ignores you completely, even resting (/heal) right next to it.
  • After: the Greater Pugil aggros to sound (resting beside it, or being within sound range at an appropriate level).

Gusgen Mines (!zone 196) and Korroloka Tunnel (!zone 173) use the same pool and behave identically.

Pool 1807 (Greater_Pugil), used in Gusgen Mines, Korroloka Tunnel and
Qufim Island, had aggro=0 and links=1. Retail behaviour is "Aggro, No
Link", and the fished Greater Pugil pools already carry aggro=1, links=0.
Species 38 already detects by sound, so with aggro enabled they aggro to
sound as on retail. Set pool 1807 to aggro=1, links=0.
@Xaver-DaRed Xaver-DaRed added the hold On hold, pending further action/info label Jul 24, 2026
@Hatberg

Hatberg commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

This does not match retail behaviour:

Qufim: https://www.youtube.com/live/Muphwi4L_Wc?si=3AeCMZ6KWN32T7gm&t=87 Doesn't Aggro, Does Link
Korroloka: https://www.youtube.com/live/Muphwi4L_Wc?si=1omCOe4uFpkYcd_i&t=324 Aggros, Does not Link
Gusgen: https://www.youtube.com/live/Muphwi4L_Wc?si=dIFKFh3kOMQFoY0B&t=747 Aggros, Does not Link

Please verify on retail with documented proof, BGWiki is not a reliable source.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hold On hold, pending further action/info

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants