Go to Page... |
Compatibility: | Lost Depths (8.1.5) High Isle (8.0.0) Ascending Tide (7.3.5) Greymoor (6.0.5) |
Updated: | 12/15/22 05:20 PM |
Created: | 06/09/15 03:54 PM |
Monthly downloads: | 795 |
Total downloads: | 85,068 |
Favorites: | 89 |
MD5: | |
Categories: | TradeSkill Mods, Bags, Bank, Inventory |
File Name |
Version |
Size |
Uploader |
Date |
6.4 |
17kB |
Troodon80 |
08/15/22 08:50 AM |
|
6.3 |
17kB |
Troodon80 |
08/04/22 03:48 AM |
|
6.2 |
17kB |
Troodon80 |
08/03/22 05:25 PM |
|
6.1 |
16kB |
Troodon80 |
04/26/22 05:32 AM |
|
6.0 |
16kB |
Troodon80 |
04/20/22 06:03 PM |
|
5.5 |
12kB |
Troodon80 |
04/09/22 08:38 PM |
|
5.4 |
11kB |
Troodon80 |
04/02/22 03:01 PM |
|
5.3b |
9kB |
Troodon80 |
03/30/22 01:36 PM |
|
5.3 |
9kB |
Troodon80 |
03/30/22 12:56 PM |
|
5.2 |
8kB |
ahmetertem |
07/09/20 06:33 PM |
|
5.1 |
8kB |
MaralieRindu |
03/05/20 03:08 AM |
|
5.0 |
8kB |
MaralieRindu |
03/02/20 07:40 AM |
|
4.5 |
34kB |
MaralieRindu |
02/28/20 05:35 AM |
|
4.4d |
38kB |
MaralieRindu |
08/26/19 05:50 AM |
|
4.4b |
34kB |
MaralieRindu |
08/19/19 03:58 PM |
|
4.4a |
38kB |
MaralieRindu |
08/19/19 06:56 AM |
|
4.4 |
38kB |
MaralieRindu |
08/18/19 03:24 AM |
|
4.3 |
38kB |
MaralieRindu |
06/22/19 01:43 AM |
|
4.2 |
38kB |
MaralieRindu |
03/29/19 02:26 PM |
![]() |
Comment Options |
jayman10000 |
View Public Profile |
Send a private message to jayman10000 |
Find More Posts by jayman10000 |
Add jayman10000 to Your Buddy List |
![]() |
||
Forum posts: 0
File comments: 5
Uploads: 0
|
Re: Here is temporary fix
I could not first figure out which lua file that was... and what those "+" and "-" mean... but then i did work it out. It's a MassDeconstructor.lua (file) in AddOns folder, - means what is now typed in the file (wrong line that makes the addon fails), and + is what you have to change that broken line for. Thanks !!!! |
|
![]() |
![]() |
Mzark |
View Public Profile |
Send a private message to Mzark |
Find More Posts by Mzark |
Add Mzark to Your Buddy List |
![]() |
|
|
Multi-pass
While you are working on an update to use the new settings could I request a feature? It'd be great to see multi-pass capability when refining, giving the addon the ability to refine until complete rather than having to hit refine multiple times when refining gives you more refining to do in jewelry.
|
![]() |
![]() |
MethosFrost |
View Public Profile |
Send a private message to MethosFrost |
Find More Posts by MethosFrost |
Add MethosFrost to Your Buddy List |
![]() |
||
Forum posts: 0
File comments: 67
Uploads: 0
|
![]()
I just tried out the new in-game mass refine, what a timesaveer that will be. Combined with this great add-on to handle all the stacks, it will be amazing. |
|
![]() |
![]() |
BringerOfOmens |
View Public Profile |
Send a private message to BringerOfOmens |
Find More Posts by BringerOfOmens |
Add BringerOfOmens to Your Buddy List |
![]() |
||
Forum posts: 7
File comments: 74
Uploads: 0
|
Re: Here is temporary fix
![]() I'll be switching MD over to simply select all the items it wants to deconstruct, then use "extract all". I spent most of yesterday and today's coding time trying to figure out how to switch to Deconstruct tab cleanly. |
|
![]() |
![]() |
MaralieRindu |
View Public Profile |
Send a private message to MaralieRindu |
Find More Posts by MaralieRindu |
Add MaralieRindu to Your Buddy List |
![]() |
|||
Forum posts: 7
File comments: 74
Uploads: 0
|
Re: Re: Multi-pass
|
||
![]() |
![]() |
MaralieRindu |
View Public Profile |
Send a private message to MaralieRindu |
Find More Posts by MaralieRindu |
Add MaralieRindu to Your Buddy List |
![]() |
|
Forum posts: 7
File comments: 74
Uploads: 0
|
Okay, this was an emergency release to get things working again. Let me know how much new breakage has occurred.
Easiest way to get my attention is the Maralie's Manse server on Discord, or submit an issue on GitHub. Coming Real Soon Now™ will be a version that uses the bulk actions that were introduced with Scalecaller. Real Soon is probably next weekend. |
![]() |
![]() |
MaralieRindu |
View Public Profile |
Send a private message to MaralieRindu |
Find More Posts by MaralieRindu |
Add MaralieRindu to Your Buddy List |
![]() |
|
|
Please do not overwrite the ENCHANTING table values like the enchantingMode within your addon!
It will maybe destroy other addons as they rely on the "save way" of getting the enchantingMode. Code:
L:333 - ENCHANTING:SetEnchantingMode(ENCHANTING_MODE_EXTRACTION) + ENCHANTING.enchantingMode = ENCHANTING_MODE_EXTRACTION Same with the Smithing mode! You definately will break protective functions of FCOItemSaver this way. It relies on the mode and enchantingMode of SMITHING and ENCHANTING. Your code looks like it will even extract if you are at the "Create glyph" enchantingMode? Maybe some kind of auto extract as you open the crafting tables? Lua Code:
This will make other addons think they are at the deconstruction/extraction panel but the controlNames of these panels like the extract slots are missing and thus the protective functions will just allow the deconstruction of maybe protected items. At best this will work properly as LibFilters is maybe recognizing the SMITHING.mode or ENCHANTING.enchantingMode and think it's at the deconstruction/extract panel. But you cannot be sure. Keep that in mind and please don't blame the other protective addons if your addon is messing with the crafting tables this way and protected items get deconstructed ![]() Thanks for your understanding.
Last edited by Baertram : 08/18/19 at 07:07 AM.
|
![]() |
![]() |
Baertram |
View Public Profile |
Send a private message to Baertram |
Find More Posts by Baertram |
Add Baertram to Your Buddy List |
![]() |
|
|
And here we go (related to my post directly below):
Some user complaiend in the past at my addon FCOCraftFilter that something is broken. He tested and found out it's MasssDeconstructor which is changing SMITHING variables (I bet you do the same like with the ENCHANTING described below, changing the smithingMod here...) and this will make my addon's buttons don't work properly. https://www.esoui.com/downloads/file...=1104#comments PLEASE do not do this. Let the user swith to the extraction/deconstruction tab properly and THEN start the deon/extraction. Or DO NOT change the enchantingMopde or smithing mode and just extract the stufgf. But if you keep it running like this is will break my and other addons for sure ![]() |
![]() |
![]() |
Baertram |
View Public Profile |
Send a private message to Baertram |
Find More Posts by Baertram |
Add Baertram to Your Buddy List |
![]() |
|
Forum posts: 2
File comments: 160
Uploads: 0
|
Getting below error when at crafting tables when trying to mass deconstruct.
EDIT: I found out how to trigger it. It will trigger if there is a gear piece already added before starting to mass deconstruct. Clearing it manually before using Mass Deconstructor will prevent the error from happening.
Warning: Spoiler
user:/AddOns/MassDeconstructor/MassDeconstructor.lua:313: function expected instead of nil stack traceback: user:/AddOns/MassDeconstructor/MassDeconstructor.lua:313: in function 'MD.ContinueWork' user:/AddOns/MassDeconstructor/MassDeconstructor.lua:346: in function 'MD.StartDeconstruction' user:/AddOns/MassDeconstructor/MassDeconstructor.lua:577: in function 'callback' EsoUI/Libraries/ZO_KeybindStrip/ZO_KeybindStrip.lua:32: in function 'OnButtonClicked' |caaaaaa<Locals> button = ud, keybindButtonDescriptor = tbl </Locals>|r EsoUI/Libraries/ZO_KeybindButton/ZO_KeybindButton.lua:194: in function 'ZO_KeybindButtonMixin:OnClicked' |caaaaaa<Locals> self = ud, visible = true </Locals>|r EsoUI/Libraries/ZO_KeybindButton/ZO_KeybindButton.lua:265: in function 'ZO_KeybindButtonTemplate_OnMouseUp' |caaaaaa<Locals> self = ud, button = 1, upInside = true </Locals>|r ZO_Dialog1Button2_MouseUp:3: in function '(main chunk)' |caaaaaa<Locals> self = ud, button = 1, upInside = true, ctrl = false, alt = false, shift = false, command = false </Locals>|r EDIT2: and found another error. When exiting the crafting table before Mass Deconstructor is done produces this error:
Warning: Spoiler
user:/AddOns/MassDeconstructor/MassDeconstructor.lua:313: function expected instead of nil stack traceback: user:/AddOns/MassDeconstructor/MassDeconstructor.lua:313: in function 'MD.ContinueWork' EDIT3: Found another error. If accidentally trying to mass deconstruct again while MD is already working to deconstruct will produce this error and halt the work (and place a ?-icon where the gear to be deconned is normally displayed):
Warning: Spoiler
user:/AddOns/MassDeconstructor/MassDeconstructor.lua:313: function expected instead of nil stack traceback: user:/AddOns/MassDeconstructor/MassDeconstructor.lua:313: in function 'MD.ContinueWork' user:/AddOns/MassDeconstructor/MassDeconstructor.lua:346: in function 'MD.StartDeconstruction' :1: in function '(main chunk)' |caaaaaa<Locals> keybind = "MD_DECONSTRUCTOR_DECON_ALL" </Locals>|r I realize that all these errors can be prevented by the user. But still, should probably be fixed?
Last edited by jayman10000 : 08/18/19 at 05:17 PM.
|
![]() |
![]() |
jayman10000 |
View Public Profile |
Send a private message to jayman10000 |
Find More Posts by jayman10000 |
Add jayman10000 to Your Buddy List |
![]() |
||
Forum posts: 7
File comments: 74
Uploads: 0
|
How is what I am doing any different to what the panel buttons do? And how is Mass Deconstructor being invoked when the player hasn't opened the crafting table scene?
Last edited by MaralieRindu : 08/19/19 at 06:10 AM.
|
|
![]() |
![]() |
MaralieRindu |
View Public Profile |
Send a private message to MaralieRindu |
Find More Posts by MaralieRindu |
Add MaralieRindu to Your Buddy List |
![]() |
|
Forum posts: 7
File comments: 74
Uploads: 0
|
I think I found it:
ZO_MenuBar_SelectDescriptor(SMITHING.modeBar, SMITHING_MODE_DECONSTRUCTION) So this will be interacting with the mode bar without simply performing the function that the button performs. Now to see if I can replicate the error the FCO Craft Filter user was experiencing (though it still doesn't explain why Mass Deconstructor is being invoked without actually interacting with a crafting table).
Last edited by MaralieRindu : 08/19/19 at 07:30 AM.
|
![]() |
![]() |
MaralieRindu |
View Public Profile |
Send a private message to MaralieRindu |
Find More Posts by MaralieRindu |
Add MaralieRindu to Your Buddy List |
![]() |
|||
Forum posts: 7
File comments: 74
Uploads: 0
|
|
||
![]() |
![]() |
MaralieRindu |
View Public Profile |
Send a private message to MaralieRindu |
Find More Posts by MaralieRindu |
Add MaralieRindu to Your Buddy List |
![]() |
|||
|
ZOs only sets the mode AT THE BEGINNING as you change the filter tabs like from "Create" to "Deconstruct". Do you have any option which automatically deconstructs if you enter a crafting station or similar to this? Like you approach the woodworking table, are at the "Refine" panel (SMITHING.mode is refine then). Now you start the deconstruction of items and set the smithing mode to deconstruction, which will run other addon codes, which fail as they think they are at deconstruction but one is at the refine panel. Something like this? I do not know your addon in detail. The only thing I know is that several crafting related addons rely on a proper handling of the smithing mode/enchanting enchantigMode and this addon somehow changes it either too early or in between or whatever, so that they will break the others, or run their code and get strange behaviours. Please check the lines where you set the mode. If at the time this happens you are currently NOT at a deconstruction panel of SMITHING or a extraction panel of ENCHANTING then you cannot simply change the modes and go on. Either the lines in your code to set the mode are simply not needed and you can remove them. Which should fix at least some possible addon error. Or you'd need to find another way like simulating a button press on the smithing/enchanting tabs to activate the deconstruction/extract tab properly. Example: Lua Code:
This makes no sense? Why should you start refining if you are not at the refining tab. You need to check this before using the function instead of setting the SMITHING.mode to refining youself. If ppl are at the refining tag they can refine, if not they need to switch. If you set the mode to refine you let other addons call their code for prehooks/Posthooks to SMITHING.SetMode or ZO_Smithing.SetMode ![]() Same for the other panels. I do not understand why these checks and changes are needed? If the users are not able to refine or deconstruct if they ARE NOT at the desired panel, you shouldn't need this at all. So better build checks in before like remove the keybinds properly so they cannot start the process if they are not at the panel etc. Another hint: You should maybe add an event for the craft_failed as well, if you started crafting. Not always the crafting will work normal and if it fails and you do leave the station or not, your internal variables are messed. You maybe should use your cleanup functions (CleanupAfterRefining(), ...) also at event_craft_failed and event_crafting_station_interact_end to be sure everthing is reset properly for next mass crafting.
Last edited by Baertram : 08/19/19 at 08:20 AM.
|
||
![]() |
![]() |
Baertram |
View Public Profile |
Send a private message to Baertram |
Find More Posts by Baertram |
Add Baertram to Your Buddy List |
![]() |
|||
Forum posts: 7
File comments: 74
Uploads: 0
|
Also the code you just quoted is what I have updated it to just in the last hour. See if anyone can replicate the error. ZO_MenuBar_SelectDescriptor will set the appropriate panel just as if the user had clicked the button.
Last edited by MaralieRindu : 08/19/19 at 08:21 AM.
|
||
![]() |
![]() |
MaralieRindu |
View Public Profile |
Send a private message to MaralieRindu |
Find More Posts by MaralieRindu |
Add MaralieRindu to Your Buddy List |
![]() |