Skip to content

Core/Packets: implement GAMEOBJECT_TYPE_UI_LINK#20919

Merged
joschiwald merged 1 commit intoTrinityCore:masterfrom
DoctorKraft:gob-ui
Jan 4, 2018
Merged

Core/Packets: implement GAMEOBJECT_TYPE_UI_LINK#20919
joschiwald merged 1 commit intoTrinityCore:masterfrom
DoctorKraft:gob-ui

Conversation

@DoctorKraft
Copy link
Copy Markdown
Contributor

Changes proposed:

  • Implement GAMEOBJECT_TYPE_UI_LINK for gob with type 48
    Get UI for Obliterium Forge or Hero's Call Board with Adventure Journal

Target branch(es):

  • master

Issues addressed: Closes #20913

Tests performed: Build & tested in-game

Known issues and TODO list:

  • Radius not implemented (if anyone have an idea)

Copy link
Copy Markdown
Member

@Shauren Shauren left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

And radius - isnt that simply the interaction distance limit?

WorldPacket const* Write() override;

ObjectGuid ObjectGUID;
uint32 UILink = 0;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This one should be int32

WorldPacket const* WorldPackets::GameObject::GameObjectUIAction::Write()
{
_worldPacket << ObjectGUID;
_worldPacket << UILink;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

always write the type for ints/floats (_worldPacket << int32(UILink);)

@DoctorKraft
Copy link
Copy Markdown
Contributor Author

@Shauren yeah i think same, but i don't have any idea where to implement this. It's in GridNotifier?

@ghost ghost changed the title Core/Packets: Implement GAMEOBJECT_TYPE_UI_LINK Core/Packets: implement GAMEOBJECT_TYPE_UI_LINK Nov 18, 2017
@joschiwald
Copy link
Copy Markdown
Contributor

take a look at GameObject::GetInteractionDistance

@DoctorKraft
Copy link
Copy Markdown
Contributor Author

I have tried one but not working. For example i set 50 yards, when icon for interact are grayed you can do everything it's always same range (only when icon are not grayed). The structure contains also a spellfocus...

@joschiwald
Copy link
Copy Markdown
Contributor

it looks like that all gameobjects have the default interaction range

@joschiwald
Copy link
Copy Markdown
Contributor

merging in the next days

@Killyana
Copy link
Copy Markdown
Contributor

Gobs interaction range will be fixed by #18917

@joschiwald joschiwald merged commit 2ee6ce6 into TrinityCore:master Jan 4, 2018
zgn1988 pushed a commit to ProjectLegion/ProjectLegion that referenced this pull request Jan 4, 2018
@DoctorKraft DoctorKraft deleted the gob-ui branch February 22, 2018 17:36
Aszune pushed a commit to Aszune/TrinityCoreRP that referenced this pull request Mar 20, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants