doadin / Baggins

zlib License
6 stars 7 forks source link

Equipment sets not sorting #76

Closed Weetbix813 closed 1 year ago

Weetbix813 commented 1 year ago

I'm using the most current retail version, Equipment Sets are not sorting into designated sections, there is no error message, the category just doesn't appear, all armor/weapons remain in the general armor/weapons sections

doadin commented 1 year ago

As much as I hate to blame something else I think this is a second instance of the game api not returning proper info. Running for bag=0,NUM_BAG_SLOTS do for slot=1,C_Container.GetContainerNumSlots(bag) do local item = C_Container.GetContainerItemID(bag,slot) if not item then break end local inset, setstring = C_Container.GetContainerItemEquipmentSetInfo(bag, slot) local name = GetItemInfo(item) print(name, " ", inset) end end

Basically if you don't know code if I loop through all bag items and check for if it is in a set(inset from above code) it is returning false for all items even though I know I have items that are part of a equipment set in my bags.

Weetbix813 commented 1 year ago

Hmm I'm not a coder so I don't fully understand, but I know AdiBags that i switched from for other issues was still able to sort equipment sets so i'm not sure if they have it coded differently or maybe I somehow have something else thats interfering?

doadin commented 1 year ago

I added a fix at least for now. This fix is included in v4.5.8 . https://github.com/doadin/Baggins/commit/5a13827f7767c8e91e4f388fc9233eccc156e0ee

Weetbix813 commented 1 year ago

Thank you so much 😊

Dr. Rochelle M. Burnaford She/her pronounshttp://www.transstudent.org/pronouns101/ Psychology Area Convener 2022-2023 Professor of Psychology Diablo Valley College

"I am not free while any woman is unfree, even when her shackles are very different from my own." -Audre Lorde

From: @.> Sent: Saturday, November 19, 2022 1:02 AM To: @.> Cc: Burnaford, @.>; @.> Subject: Re: [doadin/Baggins] Equipment sets not sorting (Issue #76)

CAUTION: This email originated from outside of 4CD. Do not click links or open attachments unless you recognize the sender and know the content is safe.

I added a fix at least for now. This fix is included in v4.5.8 . 5a13827https://nam10.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdoadin%2FBaggins%2Fcommit%2F5a13827f7767c8e91e4f388fc9233eccc156e0ee&data=05%7C01%7Crburnaford%40dvc.edu%7C167a5d8d2ac94410da9f08daca0cd912%7Cc354694acce5489fb2a3a2a9d26e0c3f%7C0%7C0%7C638044453779854887%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=DcLQSRsiE7xdi59tdlPhAQ%2B%2BJaikrGU2m4u5hTkpOIg%3D&reserved=0

— Reply to this email directly, view it on GitHubhttps://nam10.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdoadin%2FBaggins%2Fissues%2F76%23issuecomment-1320839716&data=05%7C01%7Crburnaford%40dvc.edu%7C167a5d8d2ac94410da9f08daca0cd912%7Cc354694acce5489fb2a3a2a9d26e0c3f%7C0%7C0%7C638044453779854887%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=vrvVLWho5NsUW4QcJYK9Z7%2FoS9jcLgjl0cMBf4oqiiE%3D&reserved=0, or unsubscribehttps://nam10.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAWB7QD5FAGYWTGXBY76LZPTWJCJT5ANCNFSM6AAAAAASE6BIPM&data=05%7C01%7Crburnaford%40dvc.edu%7C167a5d8d2ac94410da9f08daca0cd912%7Cc354694acce5489fb2a3a2a9d26e0c3f%7C0%7C0%7C638044453779854887%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=mVwRgXpcR8mbRrRVyKrnVUhgjOOhJJHp9twowZ8GVjk%3D&reserved=0. You are receiving this because you authored the thread.Message ID: @.***>