151 - Wintergrasp layout functionality
Currently Wintergrasp raids are treated as regular or heroic raids, depending on what the raid leader has set up. Seeing as they default to 10/25 players, while BGs default to 40, I figure (and I prefer) it if raids inside the zone wintergrasp are treated as 40man raids.
This does however cause raid groups forming in Wintergrasp for VoA to show up as 40mans, until they zone inside VoA. I'm not sure if there's a good way around this, but for me at least, this method is preferred.
I've attached the patch file for my changes.
| Name | Size | MD5 |
|---|---|---|
| wintergrasp-layout.patch | 693 B | f9394ad35a38c715c81279de90c1c8e6 |
| wintergrasp-layout.patch | 697 B | 1ca89c12aec6d15317f0123af19023c4 |
| User | When | Change |
|---|---|---|
| Phanx | Tue, 27 Oct 2009 07:15:48 | Changed status from New to Accepted |
| Phanx | Tue, 27 Oct 2009 07:15:39 | Changed assigned to from Pastamancer to Phanx |
| enigmacurse | Fri, 19 Jun 2009 03:24:36 | Added attachment wintergrasp-layout.patch |
| enigmacurse | Fri, 19 Jun 2009 02:34:02 | Added attachment wintergrasp-layout.patch |
| enigmacurse | Fri, 19 Jun 2009 02:33:25 | Create |
- 5 comments
- 5 comments
Facts
- Last updated on
- 27 Oct 2009
- Reported on
- 19 Jun 2009
- Status
- Accepted - Problem reproduced / Need acknowledged.
- Type
- Enhancement - A change which is intended to better the project in some way
- Priority
- Medium - Normal priority.
Reported by
Possible assignees
-
- Maintainer
-
- Manager
- Author
-
- Author
- #5
Phanx Tue, 27 Oct 2009 07:15:33Hmm, I could have sworn Grid already gave Wintergrasp special treatment to use the Battleground layout. Anyway, your patch only performs a check when the group makeup changes, not when changing zones, so it isn't suitable for release as-is. Grid2's approach may be best. Consider the issue accepted though. I'll do something with it.
- #4
Greltok Fri, 19 Jun 2009 05:44:28Alternately could use the same approach Grid 2 does. If you're outside, and the raid has more than 25, treat it as a BG.
http://paste.wowace.com/797/
- #3
enigmacurse Fri, 19 Jun 2009 03:27:18Can't work out how to remove old attachments, so use the one with MD5 starting with 1ca89
- #2
enigmacurse Fri, 19 Jun 2009 03:24:18Ok adding new patch, using LibBabble-Zone-3.0, not sure if that's included in Grid atm though, it's from another of my Ace addons.
- #1
enigmacurse Fri, 19 Jun 2009 03:10:48Just realised, my patch is broken, the localisation isn't working. Will try to fix after this WG :)