Love this addon. I've made RidinDirty compatibile with it by enabling auto meticulous disassembly at enchanting stations because Mass Decon throws out warnings on them. Meticulous disassembly doesn't provide any benefit to enchanting according to the informaiton provided.

+1 to this. I'd like to set a custom quantity to be refined.

Also when it says "refine full stacks" (the setting turned ON) is that refining a complete stack of 200 raw materials, or is it the Max refine button when at a crafting station?

e.g. I have 800 platinum dust (4 x full stacks). Would it do 4x refine batches or refine all 800 at once?

Addon sounded interesting in concept, but effectively unusable in gamepad mode.

can you add a configuration to set the maximum default to be refined? With the endeavors refining the default off ALL results in having no mats to refine when that endeavor shows up - too easy to get all when only needing a few. I try to keep a large number of raw mats for guild vendors, so only refine the minimum as needed for the endeavor.
thanks for a fantastic and highly useful addon.

Hey don't worry bud, I've seen that problem for a few days and now I don't see it anymore...I think I did something wrong with settings back then. Now it's working perfectly since when I posted that issue

Are you on the most recent version? There's no reason why there should be a string there, nor is operator < used at that point.

Heya. Sorry for the late reply on this. I've not experienced anything like that, but I will look into it. Is it a constant issue or does it resolve if you press the button/keybind a couple times or relog?

Any idea why many items aren't being deconstructed? I can show you the options I have.
I'm confused because so far it's been working alright.

I have a problem with jewellery in particular, I can't deconstruct it. The error that I get every time I open jewellery station:

user:/AddOns/MassDeconstructor/MassDeconstructor.lua:696: operator < is not supported for string < number
stack traceback:
user:/AddOns/MassDeconstructor/MassDeconstructor.lua:696: in function 'ShouldDeconstructItem'
|caaaaaa<Locals> bagId = 1, slotIndex = 0, itemLink = "|H0:item:69167:363:50:0:0:0:0:...", sIcon = "/esoui/art/icons/gear_breton_r...", iStack = 1, iSellPrice = 51, bMeetsUsageRequirement = T, isLocked = F, iEquipType = 12, iItemStyle = 9, quality = 4, qualityOverride = 4, CraftingSkillType = 7, itemType = 2, specialisedItemType = 0, boundType = 2, isSetPc = T </Locals>|r
user:/AddOns/MassDeconstructor/MassDeconstructor.lua:724: in function 'AddItemsToDeconstructionQueue'
|caaaaaa<Locals> bagId = 1, bagSlots = 160, slotIndex = 0, itemLink = "|H1:item:69167:363:50:0:0:0:0:..." </Locals>|r
user:/AddOns/MassDeconstructor/MassDeconstructor.lua:752: in function 'BuildDeconstructionQueue'
user:/AddOns/MassDeconstructor/MassDeconstructor.lua:1077: in function 'MD.OnCrafting'
|caaaaaa<Locals> craftSkill = 131534, sameStation = 7, craftMode = T </Locals>|r

Hey Troodon80,

your addon is leacking some variables to the global table _G.
Especially this one here is critical "isMarked" as FCOIS is using same variable names and you overwrite it that way!


Please fix this asap and add the local up in front:
local isMarked, markedArray = FCOIS.IsMarked(bagId, slotId, {9}, nil)


Other global leaking variables:
x !!! very critical (function AddItemsToDeconstructionQueue)
itemString (function ListItemsInQueue)
itemToDeconstruct (function MD.ContinueWork())
slotIndex !!!!!! very critical! (function AddCraftingBagItemsToRefineQueue)

Please fix those asap and check your other code files for leaking to global variables, thanks.

If the API of FCOIS is used properly ANY FCOIS marker icon is protected against deconstruction unless it was not explcitly allwowed to be deconstructed (for normal marker icons, check the submenu anti-* protection to e.g. allow deconstruction of items marked as intrictae or marked for decon -> even this is protected by default / for dynamic icons: each icon itsself defines if allowed to be deconstructed or not at the icon settings).

I've tested this with a self crafted item marked for decon (bagId 1, slotindex 44)
-Marked for decon
/tb FCOIS.IsDeconstructionLocked(1,44) -> false
-Marked as sell in guild store
/tb FCOIS.IsDeconstructionLocked(1,44) -> true
-Marked for decon & sell at guildstore (only works if you have not enabled to remove automatically other marker icons if you apply the decon/sell at guild store!)
/tb FCOIS.IsDeconstructionLocked(1,44) -> true

Also tested this with other set items etc, all was okay.
Did you maybe enable at the anti-* protection settings submenu -> deconstruction -> Allow decon (ignore others)? This would allow the item to be deconstructed if it's marked for decon AND sell at guildstore as the decon icon alone would be respected then!



Edit:
Seems the below described "open decon panle first" was added already as I can see at function MD.StartDeconstruction(). Even looks like my old provided code ;-) So decon panel seems to be properly opened meanwhile, alright.

If you are still able to deon items marked for "sell at guil store" that way, please write me a PM and I'll have a look with you guys how to fix this.




If items marked as sell at guildstore got deconstructed either the API was not used properly, or there is an "allow decon of marked at guild store" setting at the anti-* protection settings whch was enabled, or the API is not working properly (which might be the case if the addon using the API is e.g. trying to decostruct items AT THE SAME SECOND the crafting tables open AND it is not properly activating the deconstruction tab at the crafting stations -> e.g. try to decon all the second the crafting table is interacted with and we are still at the refinement tab!)

The game allows decon that way but even default vanilla UI is not working properly if you just do it that way and I have told each addon doing this to change that please. If they do not change it and want to stay at refine to do a deconstuction, not my problem.
Either use the game's API and UI as expected or live with the iterferences.

I even had added an explicity pass-in of the LF_SMITHING_DECONSTRUCTION filterId to circumvent that problem and only check this (instead of checking the active scene, fragment, controls showing which obviously fails if you expect it to show the decon inventory but refine inv. is shown!) one e.g. by using the API FCOIS.IsDeconstructionLocked(bagId, slotIndex, doNotCheckJewelry)
But I cannot say it is working each time, depends on the addons using it and what they do before (as described above).

I'm open to improve this if there is a way but I'd rather let the addons open decon panel properly before starting a decon, instead of doing all at the refine UI tab. This should fix a lot of problems, especially with other addons that apply filters and protections for decon AT the deconstuction UI (but not at refine or any other).

And there is a pretty easy way using the descriptor of the decon tab menu button to open that menu easily!
ZO_MenuBar_SelectDescriptor(self.tabs, descriptorNameOfDeconTab)
where self-tabs should be SMITHING.deconstructionPanel.tabs or similar and the descriptorNameOfDeconTab would be the SI_* string constant used for that tab button. Inspect the tab button with merTorchbug/zgoo e.g. and check for the data table or the descriptor table to see the text. Search for that text at merTorchbug Strings tab then to find the SI constant.

I've got items marked for sale at a guild store. Since that is a default icon it doesn't have the ability to block deconstruction. Can you enhance the FCOIS integration so I can block some of the default icons from being deconstructed?

Yes, absolutely. I'll get that added soon.

Hey! Could there be an option, when turned on the addon would warn when Meticulous Disassembly isn't slotted? Or something similar

Thanks for your work

I actually removed this option and made it use the in-game option, which more easily allows you to toggle it from within the deconstruction menu itself instead of needing to leave the decon panel and go find the addon settings.



The base game checkbox is plainly visible outside of using gamepad mode. I don't imagine I'll change this unless I get more feedback saying a change would be the best direction.

As I do not personally use FCOIS, I do not know. The options for it are from before I took over maintenance. Though all I can say is that the options for FCOIS and FilterIt are run before any of the other filters.

What other options do you have enabled/disabled? Do you have CraftStore, and the optional CraftStore compatibility for account-wide trait knowledge enabled?