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

Jump to bottom

[6.x] ZoneId in phase_area #17025

Closed
Lopfest opened this issue Apr 24, 2016 · 0 comments
Closed

[6.x] ZoneId in phase_area #17025

Lopfest opened this issue Apr 24, 2016 · 0 comments
Assignees
@MitchesD
Labels
Branch-master Comp-Core

Comments

@Lopfest
Copy link
Contributor

Lopfest commented Apr 24, 2016

Description: missing option to use phases for entire zone

Current behaviour: currently zone id cannot be used in phase_area. nothing happens.

Expected behaviour: apply specified phase to the whole zone. regarding phase conditions, when there is condition for zone, and also condition(s) for specific area in that zone -> apply the stricter condition, which is the one for area.

Branch(es): 6x

TC hash/commit: 3ac3316

TDB version: 6.04

Operating system: Windows 7 x64

Linking model: static/dynamic
msoky referenced this issue May 17, 2016
@MitchesD
Core/ConditionMgr: condition CONDITION_SOURCE_TYPE_PHASE should not b…
192c428 
…reak on first phaseID which is equal to condition phase Id, it should iterate over all and check if more areas are trying to apply the same phase

* it's required for example for Wandering Isle quests

Closes #17054
@MitchesD MitchesD self-assigned this May 17, 2016
This was referenced May 18, 2016
Closed
Closed
@Lopfest Lopfest mentioned this issue May 19, 2016
Draft
51 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@MitchesD MitchesD

Labels
Branch-master Comp-Core
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Lopfest @MitchesD