Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[3.3.5] Share Quest does not work if quest objective is complete #16183

Closed
ghost opened this issue Jan 4, 2016 · 11 comments
Closed

[3.3.5] Share Quest does not work if quest objective is complete #16183

ghost opened this issue Jan 4, 2016 · 11 comments

Comments

@ghost
Copy link

ghost commented Jan 4, 2016

branch: 3.3.5
rev: f377a58

When you try to share a quest while in a group, the quest will only share if the objective is incomplete.

This makes most(if not all) quests where the objective is to speak to another NPC un-shareable, as the objective is complete the moment you accept the quest.

How to replicate:
-Make 2 dwarves on 2 accounts.
-Group up
-Accept very first dwarf quest on 1 character
-At this point, it IS sharable.
-complete the quest, but don't hand it in (.quest complete questID)
-Quest can no longer be shared. (edit: Share button can be clicked, but does nothing)

@Treeston
Copy link
Contributor

Treeston commented Jan 4, 2016

Define "can no longer be shared". The button is greyed out, or the button does nothing?

@ghost
Copy link
Author

ghost commented Jan 4, 2016

Sorry.

The button is clickable, but does nothing when clicked.

@Killyana
Copy link
Contributor

Killyana commented Jan 4, 2016

Confirm, I was thinking it is the retail behavior.

@ghost
Copy link
Author

ghost commented Jan 4, 2016

I can't see it being retail behavior, but maybe I remember wrath wrong... it makes any quest with an auto complete objective (quests that require you to go to another npc) impossible to share, despite them having the shareable flag.

@Treeston
Copy link
Contributor

Treeston commented Jan 4, 2016

Pretty sure it isn't retail behavior (especially since UI indicates the quest should still be sharable).

@ghost
Copy link
Author

ghost commented Jan 6, 2016

CanShareQuest() check inside QuestHandler.cpp seems to be contributing to this issue?

@ghost
Copy link
Author

ghost commented Jan 6, 2016

I agree 100% with you, @Treeston. If a quest is not supposed to be shareable, it should not have the button enabled in the first place. Either the quest flags are wrong, or the sharing functionality is bugged.

@Aokromes
Copy link
Member

Aokromes commented Jan 6, 2016

humm, the "very first dwarf quest" is not sharable for me, nor on tc, nor on retail.

@ghost
Copy link
Author

ghost commented Jan 6, 2016

#179
isn't shareable for you?

@Aokromes
Copy link
Member

Aokromes commented Jan 6, 2016

nope.

@ghost
Copy link
Author

ghost commented Jan 6, 2016

Are you able to accept a quest on one character that you know is shareable, complete the objective, and then share it with another player?

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

No branches or pull requests

4 participants