WowAce.com

Grid

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, 10 Nov 2009 10:32:16 Changed status from Accepted to Fixed
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

You must login to post a comment. Don't have an account? Register to get one!

  • 6 comments
  • Avatar of Greltok Greltok Tue, 10 Nov 2009 02:44:35

    Addressed in r1232.

  • Avatar of Phanx Phanx Tue, 27 Oct 2009 07:15:33

    Hmm, 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.

  • Avatar of Greltok Greltok Fri, 19 Jun 2009 05:44:28

    Alternately 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/

  • Avatar of enigmacurse enigmacurse Fri, 19 Jun 2009 03:27:18

    Can't work out how to remove old attachments, so use the one with MD5 starting with 1ca89

  • Avatar of enigmacurse enigmacurse Fri, 19 Jun 2009 03:24:18

    Ok 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.

  • Avatar of enigmacurse enigmacurse Fri, 19 Jun 2009 03:10:48

    Just realised, my patch is broken, the localisation isn't working. Will try to fix after this WG :)

  • 6 comments

Facts

Last updated on
10 Nov 2009
Reported on
19 Jun 2009
Status
Fixed - Developer made requested changes. QA should verify.
Type
Enhancement - A change which is intended to better the project in some way
Priority
Medium - Normal priority.

Reported by

Possible assignees