• Visit Rebornbuddy
  • Visit Panda Profiles
  • Visit LLamamMagic
  • [Plugin] TidyBags 3.0 Reloaded!

    Discussion in 'Leveling' started by LiquidAtoR, Sep 15, 2010.

    1. LiquidAtoR

      LiquidAtoR Community Developer

      Joined:
      Jan 15, 2010
      Messages:
      1,430
      Likes Received:
      52
      Trophy Points:
      48
      The duplicate is a remnant when I moved items to the extra sleep version (due to lootlag of some BoP containers).

      As I'm still at work, I'm not running LFR's, so I didn't see it yet.

      Will try to run a few ID checks and update on SVN and attachment when I got most of them.
      Am making my 24 hour break currently on a parking space in Holland.

      Edit: Revision 3.6.3.4: Attachment + SVN - Changed the ID's to match the new 5.2 ID's for LFR bags (Thanks Laria for reporting 1 change).

      The 5.2 patch has had it's item ID's on some containers updated because they now "Possibly" drop more items than before, and as far as I could run the checks only 3 ID's changed (the containers from the pre 5.2 MoP raids).
      The new containers can now contain LFR level epic gear, battlepet stones and other 'valuables' besides the gold it contained before the changes in 5.2.
       
      Last edited: Apr 14, 2013
    2. Zoke20

      Zoke20 New Member

      Joined:
      Jan 4, 2013
      Messages:
      9
      Likes Received:
      0
      Trophy Points:
      0
      I can't open the settings menu... any ideas.
       
    3. finni

      finni Member

      Joined:
      Jun 14, 2012
      Messages:
      248
      Likes Received:
      0
      Trophy Points:
      16
      same, anyone has a clue?
       
    4. tumbum

      tumbum Active Member

      Joined:
      Mar 17, 2011
      Messages:
      3,341
      Likes Received:
      13
      Trophy Points:
      38
      there are no settings
       
    5. LiquidAtoR

      LiquidAtoR Community Developer

      Joined:
      Jan 15, 2010
      Messages:
      1,430
      Likes Received:
      52
      Trophy Points:
      48
      Code:
      87391, // Plundered Treasure (Luck of the Lotus Buff)
      Is already in there (and is in there for some time already)
      Remember, after each next lootevent TidyBags performs it's trick.
      Unless the ID has changed in patch 5.2 for this item?
       
      Last edited: Apr 19, 2013
    6. Valpo

      Valpo Member

      Joined:
      Jun 15, 2012
      Messages:
      205
      Likes Received:
      3
      Trophy Points:
      18
      Hey LiquidAtoR,

      Love the plugin. With my skinner/leatherworker, [Sha-Touched Leather] isn't being made in to [Exotic Leather].

      Peace.
       
    7. ubergoober

      ubergoober New Member

      Joined:
      Nov 19, 2010
      Messages:
      321
      Likes Received:
      0
      Trophy Points:
      0
      Not sure whats going on but WoWhead also lists the itemID# as 87391, but they still don't seem to be getting opened... I'm gonna look around to see if I can find a new ID#
       
    8. ubergoober

      ubergoober New Member

      Joined:
      Nov 19, 2010
      Messages:
      321
      Likes Received:
      0
      Trophy Points:
      0
      Ok I just moved the
      Code:
      87391, // Plundered Treasure (Luck of the Lotus Buff)
      from
      Code:
      private HashSet<uint> _itemRequiresSleep = new HashSet<uint>() {
      to under the
      Code:
      private HashSet<uint> _itemUseOnOne = new HashSet<uint>() {
      and now it works BEAUTIFULLY, not sure whats going on with the itemrequiressleep, but it doesn't seem to be working at all even for the other items listed under it
       
    9. LiquidAtoR

      LiquidAtoR Community Developer

      Joined:
      Jan 15, 2010
      Messages:
      1,430
      Likes Received:
      52
      Trophy Points:
      48
      Probably moved too much around there ^^
      As far as I remember it should be listed on both lists if there's a double sleep required (due to loot issues in the past with certain soulbound items).
      Am home, but not yet settled back in after my working period.
      In the next few days I can start running checks and quick updates if needed.
      For now I have too much to do at home to get to anything else but reading a bit, posting here and there and check my email.

      Edit: Reverted the deleted items from the initial list.
      Revision available from SVN and attached to the first post as well.
      Revision 3.6.3.5: Attachment + SVN - Cleaned up to much without able to test it, thanks ubergoober for pointing it out.
      Edit2: Also added the Stormtouched Cache from Nalak to SVN and Attachment.
      Revision 3.6.3.6: Attachment + SVN - Added Stormtouched Cache (Nalak World Boss Consolidation Loot).
       
      Last edited: May 5, 2013
    10. laria

      laria Well-Known Member

      Joined:
      Jan 15, 2010
      Messages:
      5,386
      Likes Received:
      36
      Trophy Points:
      48
      94296, one more thing to add
       
    11. Proto

      Proto New Member

      Joined:
      Jan 8, 2012
      Messages:
      219
      Likes Received:
      6
      Trophy Points:
      0
      LiquidAtoR,

      After your release of ver v3.6.1.3 I had been editing the plugin on my own, adding some items I found missing when MoP released. Now with 5.3 out, there were a few more items to be added.
      So, I finally checking back into your thread and found that you'd been busy with several updates of your own.

      Here are a few things I added to your old 3.6.1.3, that you may wish to add to your ongoing project..

      First off in your "StackCount" section of code I added support for 3 count stackable items, such as older enchanting shards.
      Code:
                  if (sw.Elapsed.TotalSeconds > 1) { // throttle to 1s
                      foreach (WoWItem item in ObjectManager.GetObjectsOfType<WoWItem>()) { // iterate over every item
                          if (item != null && item.BagSlot != -1) { // check if item exists and is in bag
                              if (_itemUseOnThree.Contains(item.Entry)) { // stacks of 3
                                  if (item.StackCount >= 3) {
                                      this.useItem(item);
                                  }
                              } else if (_itemUseOnFive.Contains(item.Entry)) { // stacks of 5
                                  if (item.StackCount >= 5) {
                                      this.useItem(item);
                                  }
                              } else if (_itemUseOnTen.Contains(item.Entry)) { // stacks of 10
                                  if (item.StackCount >= 10) {
                                      this.useItem(item);
                                  }
                              } else if (_itemUseOnOne.Contains(item.Entry)) { // stacks of 1
                                  if (item.StackCount >= 1) {
                                      this.useItem(item);
      Add for ItemUseOnThree
      Code:
              };
      
              private HashSet<uint> _itemUseOnThree = new HashSet<uint>() {
                  		74252, // Small Ethereal Shard
                  		52720  // Small Heavenly Shard
      Add for ItemUseOnTen (6 new Herbalism scrap items introduced in MoP 5.3. These are dropped when attempting to gather at below the required skill level for the herb node.)
      Code:
              };
      
              private HashSet<uint> _itemUseOnTen = new HashSet<uint>() {
                  		97619, // Torn Green Tea Leaf
                  		97620, // Rain Poppy Petal
                  		97621, // Silkweed Stem
                  		97622, // Snow Lily Petal
                  		97623, // Fool's Cap Spores
                  		97624  // Desecrated Herb Pod
      That's all I've done that you had not already done yourself on subsequent patches. Nice work!

      Thanks,
      Proto
       
    12. Zamphire

      Zamphire Member

      Joined:
      May 16, 2013
      Messages:
      354
      Likes Received:
      11
      Trophy Points:
      18
      You may want to add the following into itemUseOnTen if you're a miner.

      Code:
      			97512, // Ghost Iron Nugget
       
    13. kavlantis

      kavlantis New Member

      Joined:
      Nov 20, 2012
      Messages:
      468
      Likes Received:
      2
      Trophy Points:
      0

      Any chance u update the plugin to add ALL new pandaria petals plus the nugets also?
       
    14. LiquidAtoR

      LiquidAtoR Community Developer

      Joined:
      Jan 15, 2010
      Messages:
      1,430
      Likes Received:
      52
      Trophy Points:
      48
      Just came home after my 3 week working period so I can start looking into changes.
      I didn't even get to see, play or feel the 5.3 patch yet, let alone download it :)

      Will see for changes, concistencies, updates and other stuff as long as it's a neutral item (so it doesn't conflict with not having a certain profession).

      Expect a update in the next few days when I'm settled in again @ home.
       
    15. LiquidAtoR

      LiquidAtoR Community Developer

      Joined:
      Jan 15, 2010
      Messages:
      1,430
      Likes Received:
      52
      Trophy Points:
      48
      You are way behind in updates my friend :)
      I've added your UseOnThree part in the latest release (SVN atm, I have to repack it for the forums), and took the liberty to properly expand on that list (with all the old enchanting items).
      I can't test unfortunately as I ran out of gametime to just check the compiling in HB itself.
      I will get some in the next few days, just don't have the time yet to sit and play.
      Anyways I need to get a new mouse first, the old one magically broke after my son had it for a day ;)
      Added to the last release together with the other item (the Kyparite Fragment).
      Yes, latest release has it all (at least I hope it does have it all).
      Many new items, old items with new ID's and new drops have been introduced in patch 5.3
      I've read trough many patch files and I hope I have most (if not all) the new ID's referenced in the plugin.
      Also catched a few items that are separated in ID's with Alliance and Horde factions (and no horde players here complained, haha).

      Am updating the SVN now, then I do the first post and get a changelog ready here.

      Revision 3.6.3.7: Attachment + SVN - Added 5.3 items to the on-use lists (request by Proto, Zamphire and Kavlantis), Added a UseOnThree part (suggestion by Proto), Added *****ed Primal Egg (request by Laria), Added Kyparite Fragment, a few new ID's for older items and a all new Heroic Cache (Heroic Scenario Reward), Added the Gold container rewards for the Brawler Guild Fights, Separated Alliance and Horde reward ID's and a few more things I forgot to document.

      Please do be aware this release is untested due to my gametime being expired on my account (I will get more time when I have time to play).
      So if there's any bugs please be as clear as possible (if any post the line it occurs).
      Thanks!

      Edit: Presuming noone complained I take it this works :)
       
      Last edited: Jun 3, 2013
    16. Proto

      Proto New Member

      Joined:
      Jan 8, 2012
      Messages:
      219
      Likes Received:
      6
      Trophy Points:
      0
      Brilliant! I had not considered those other 3 stack enchanting items you've added. Works like a charm. And thanks so much for the SVN. No more manual personal updates for me, because I'm too lazy to get on the forum and download the latest zip. I'll leave the updates to you from now on my friend :)
       
    17. klepp0906

      klepp0906 Banned

      Joined:
      Apr 25, 2013
      Messages:
      1,352
      Likes Received:
      8
      Trophy Points:
      38
      So is the directory structure supposed to be HB>Plugins>TidyBags>Trunk>TidyBags>TidyBags.cs ?

      I have a feeling its not and if this is the case either your SVN link is wrong, or you cant update via SVN without having to move a buncha junk afterwards.

      What am I missing?

      Don't have this issue with other svns so im betting its that github crap! >.<
       
    18. LiquidAtoR

      LiquidAtoR Community Developer

      Joined:
      Jan 15, 2010
      Messages:
      1,430
      Likes Received:
      52
      Trophy Points:
      48
      Nope, it's not the github crap.
      Structure is wrong as you already suspected.

      Should be HB > Plugins > TidyBags > TidyBags.cs

      When you checkout, you can browse the structure and checkout what you need (and not the whole thing).
      At least with TortoiseSVN it does for me.
      I have a few SVN's also from other people where I don't want the root, but a subpart of the SVN.
       
    19. tumbum

      tumbum Active Member

      Joined:
      Mar 17, 2011
      Messages:
      3,341
      Likes Received:
      13
      Trophy Points:
      38
      Any chance to get in Option to set On/Off for Skinners/Leatherworker to put the Scraps together while Farming?
       
    20. LiquidAtoR

      LiquidAtoR Community Developer

      Joined:
      Jan 15, 2010
      Messages:
      1,430
      Likes Received:
      52
      Trophy Points:
      48
      Just add them to your local copy?
      A SVN checkout shouldn't delete what you add in there yourself (that is if you have it set to merge ofcourse).
       

    Share This Page