IsJusta Decon Carried Items List : TradeSkill Mods : Elder Scrolls Online AddOns

(15 Kb)

Download

Updated: 07/16/24 01:54 PM

Add Favorite

Install Help

Portal

Bugs

Features

Pictures

File Info

Compatibility:	Update 43 (10.1.0) Gold Road (10.0.0) Scions of Ithelia (9.3.0) Endless Archive (9.2.5) base-game patch (9.1.5) Necrom (9.0.0) Scribes of Fate (8.3.5)
Updated:	07/16/24 01:54 PM
Created:	08/21/20 11:16 AM
Monthly downloads:	365
Total downloads:	12,957
Favorites:	17
MD5:
Categories:	TradeSkill Mods, Game Controller

10.1.0

IsJusta Decon Carried Items List Popular! (More than 5000 hits)

Version: 3.3.2

by: IsJustaGhost [More]

Requires LibAddonMenu-2.0

Works in Gamepad Mode and Keyboard/Mouse Mode

NEW:
3

added option to auto "Add All". This option will add all filtered items to be deconstructed on opening the station or assistant.

Features

Refine
1. Adds the option to only show raw materials there are enough to refine. (10 or more)
Deconstruct
1. Adds the "Carried" tab
  - this tab only show items carried in the "Backpack"
  - all items are sorted the same as originally.
2. Changes the original tabs ( Jewelry, Weapons, Apparel )
  - now only shows banked items
3. Disables and removes the option to show banked items in keyboard mode since it becomes redundant.
Deconstruction Assistant
1. Adds the "Carried" tab
  - this tab only show items carried in the "Backpack"
  - all items are sorted the same as originally.
2. Changes the original tabs ( All, Jewelry, Weapons, Apparel, Glyphs)
  - now only shows banked items
3. Disables and removes the option to show banked items in keyboard mode since it becomes redundant.
"Automation"
1. "Add All"
  - this button is used to quickly add all deconstructable items in the curent tab in one click, based on filters, up to 100 items.
  - if all posible items that can be added using "Add All" hae been selected, it will become "Clear Selections".
  - by cefualt it is only added to Refine and the "Carried" tab and can be added to the other tabs be setting "Use Add All for others".
2. "Open to Deconstruct" {requested by K1nor}
  - auto-opens to Deconstruct and slects the "Carried" tab if you are carying items that pass the filters.
  - use with caution. Some addons require opening the station to a specific tab. Using this option could cause issues with those addons.
3. "Auto open Assistant."
  - skips the dialogue and slects the "Carried" tab if you are carying items that pass the filters.
4. "Auto Add All"
  - this option will auto "Add All" filtered items to be deconstructed on opening the station or assistant.
5. "Filters"
  - include Default -- Items not covered by the other filters
  - include Set Items
  - include Ornate
  - include Intricate
  - include Retraited
  - include Reconstructed
  - include Researchable

Each filter has it's own:

include only items at or above the Min Quality set.
include only items at or below the Max Quality set.

If a filter is not enabled, anything that would be included in that filter will be excluded from "Add All"

"Clean refinement tab" option in settings.

! May conflict with other inventory filtering addons !
If using FCO CraftFilter, when not viewing "Carried" tab, selecting to hide banked items will result in an empty list. The original tabs* only show banked items.

* "original tabs" = the original deconstruct tabs for the current station ( Jewelry, Weapons, Apparel )

- - - 3.3.2
○ patched filtering to prioritize enabled filters

- - - 3.3.1
○ patched filtering to not always include the default filter

- - - 3.3
○ changed how initializing per system object is handled
○ added option to prevent stolen items from being added with "Add All".
○ added, allow stolen by quality, filter

- - - 3.2.6
○ removed debug output

- - - 3.2.5
○ compatibility update

- - - 3.2.4
○ stepping up the version to make sure it does not conflict with being updated by clients

- - - 3.2.3
○ removed debug messages
○ changed when the setting for include banked items is set
○ hid the include bank check box in keyboard mode
○ fixed resetting to show banked

- - - 3.2.2
○ fixed BOP and trad-able dialogue not showing when warn is active.
○ fixed keyboard mode not refreshing keybind when changing inventories
○ improved compatibility with Advanced Filters

- - - 3.2.1
○ fixed filtering so now filters are check in priority from top to bottom in the settings.
-- If a filter is enabled and, that item falls under a filter below it, the last filter will be the one used even if it's disabled.
-- If a researchable item falls under any other filter, it will be ignored if the filter researchable is disabled.
○ again, hoping that the Add All Keybind updates properly. Changed a variable in the check.

- - - 3.2
○ updated default filters to work with libFilters
○ added libFilters universalDeconTabKeyToLibFiltersFilterType for "carried" tab
○ added limited compatibility with Advanced Filters

- - - 3.1
○ fixed gamepad mode not letting category change in clothier station
○ fixed keyboard crafting stations causing error
○ Add All keybind now checks if extraction slot has items if un-added item would exceed max
-- this will properly change the keybind from Add All to Clear Selections.

- - - 3
○ simplified functionality to allow easier management
○ added option to allow filtered Add All to happen on open

- - 2.7
○ change how filtering is done
- each "trait" has it's own filter settings

- - - 2.6
○ added minimum quality to filters
○ updated API version

- - - 2.5.5
○ HOTFIX update for gamepad mode. No longer relying on the initial GetCurrentFilter check since it will cause errors for gamepad mode.
Now IsCustomTab will handle discrepancies.

- - - 2.5.4
○ HOTFIX for gamepad mode. Implemented an overwrite to the newly added function to gamepad mode GetCurrentFilter
this function was requested in order to make functionality match keyboard mode. However the function is incomplete.

- - - 2.5.3
○ HOTFIX for gamepad mode. Implemented an overwrite to the newly added function to gamepad mode GetCurrentFilter
this function was requested in order to make functionality match keyboard mode. However the function is incomplete.

- - - 2.5.2
○ attempt to fix Advanced Filters conflict by removing some unused experimental lib functions.

- - - 2.5.1
○ attempt to fix conflict with Advanced Filters.

- - - 2.5
○ completely rewrote the entire addon.
○ added support for the deconstruction assistant
-- the deconstruction assistant will now have a "Carried" tab.
○ automation improvement. If "Open to decon" is enabled, it will only do so if there are deconstruct-able items on you.
○ If "Auto open Assistant." is enabled, interacting with the decon assistant will auto enter deconstruct.
-- if deconstruct-able carried items are present, it will auto select the "Carried" tab.
○ added max item quality to requirements for automation, select-able in the settings.

- - - 2.4.2
○ compatibility update.
removed requirement for the experimental library

- - - 2.4.1
○ compatibility update.

- - - 2.4
○ implemented support for LibHaF
○ added support for FCO_ItemSaver for deconstruction
○ added function to prevent items being added to crafting slot if it would exceed the limit.
○ updated for API 101034.

- - - 2.3
○ replaced filter function used from ZO_SharedSmithingExtraction_DoesItemPassFilter

- - - 2.2.7
○ added French translation courtesy of fzr6n7
○ added Intricate to Include in add all
○ changed where EVENT_ADD_ON_LOADED is

- - - 2.2.6
○ commented out "show name and version" on load
○ updated api version

- - - 2.2.5
○ fixed filtering for carried tab. will now be filtered with libFilters

- - - 2.2.4
○ fixed error in Gamepad mode from trying to open the options menu for Deconstruct

- - - 2.2.3
○ changed how keybinds are made
○ added option to enable add all for others
○ additional minor improvements

- - - 2.2.2
○ some minor improvements
○ in keyboard mode, will also auto select carried tab

- - - 2.2.1
○ fixed bug caused from switching game modes while using a smithing station

- - - 2.2
○ add option to enter station to Deconstruction tab.

- - - 2.1.4
○ fixed DoesItemPassFilter. Carried items will now show up

- - - 2.1.3
○ added LibFilters-3.0 as optional dependency
○ list will now be filtered, if filtering is available

- - - 2.1.2
○ added Add All to refine
○ fixed some issues

- - - 2.1.1
○ attempt to fix slot stack issue

- - - 2.1
○ added keybind for "Add All"
○ added options to include in "Add All"

- - - 2.0.1
○ api update

- - - 2
○ some code changes and added gamepad support
○ disabled the show in bank option from base game

- - - 1.5.3
○ version update

- - - 1.5.1
Created unique functions for sorting carried decon smithing list

- - - 1.5
Changed how the objects are registered.

- - - 1.4.1
Changed the way a function is hooked.

- - - 1.4
Updated game version

- - - 1.3
Added
○ texture to empty deconstruction slot if in Carried list

- - - 1.2
Added
○ option to enable auto-hiding of the carried list if list is empty

○ implemented a cleaner method of creating the carried tab. no longer uses PostHook

- - - 1.1.1
Added
○ no item text to Carried decon lists

1.1
ADDED:
○ optional setting to only show the raw materials you have at least 10 of for Refinement.

Optional Files (0)

Archived Files (50)

File Name	Version	Size	Uploader	Date
IsJusta Decon Carried Items List	3.3.1	15kB	IsJustaGhost	07/10/24 06:56 AM
IsJusta Decon Carried Items List	3.3	15kB	IsJustaGhost	07/09/24 07:45 AM
IsJusta Decon Carried Items List	3.2.6	16kB	IsJustaGhost	06/15/23 03:59 PM
IsJusta Decon Carried Items List	3.2.5	16kB	IsJustaGhost	06/15/23 03:53 PM
IsJusta Decon Carried Items List	3.2.4	16kB	IsJustaGhost	06/15/23 07:18 AM
IsJusta Decon Carried Items List	3.2.3	16kB	IsJustaGhost	06/14/23 05:36 PM
IsJusta Decon Carried Items List	3.2.1	16kB	IsJustaGhost	06/07/23 07:45 PM
IsJusta Decon Carried Items List	3.2	15kB	IsJustaGhost	06/06/23 06:17 PM
IsJusta Decon Carried Items List	3.1	15kB	IsJustaGhost	06/06/23 12:47 AM
IsJusta Decon Carried Items List	3	14kB	IsJustaGhost	06/05/23 06:47 PM
IsJusta Decon Carried Items List	2.7	16kB	IsJustaGhost	10/11/22 06:14 PM
IsJusta Decon Carried Items List	2.6	15kB	IsJustaGhost	10/09/22 06:18 PM
IsJusta Decon Carried Items List	2.5.5	15kB	IsJustaGhost	06/20/22 08:04 PM
IsJusta Decon Carried Items List	2.5.4	15kB	IsJustaGhost	06/20/22 04:44 PM
IsJusta Decon Carried Items List	2.5.3	15kB	IsJustaGhost	06/20/22 04:18 PM
IsJusta Decon Carried Items List	2.5.2	15kB	IsJustaGhost	05/07/22 10:50 AM
IsJusta Decon Carried Items List	2.5.1	17kB	IsJustaGhost	04/22/22 11:00 AM
IsJusta Decon Carried Items List	2.5	17kB	IsJustaGhost	04/21/22 08:48 PM
IsJusta Decon Carried Items List	2.4.2	13kB	IsJustaGhost	03/17/22 06:49 PM
IsJusta Decon Carried Items List	2.4.1	19kB	IsJustaGhost	03/17/22 05:26 AM
IsJusta Decon Carried Items List	2.4	19kB	IsJustaGhost	03/16/22 10:37 PM
IsJusta Decon Carried Items List	2.3	12kB	IsJustaGhost	03/13/22 03:03 PM
IsJusta Decon Carried Items List	2.2.7	12kB	IsJustaGhost	02/21/22 04:32 PM
IsJusta Decon Carried Items List	2.2.6	11kB	IsJustaGhost	12/08/21 03:37 PM
IsJusta Decon Carried Items List	2.2.6	11kB	IsJustaGhost	12/08/21 03:29 PM
IsJusta Decon Carried Items List	2.2.5	11kB	IsJustaGhost	11/16/21 08:50 PM
IsJusta Decon Carried Items List	2.2.4	12kB	IsJustaGhost	11/14/21 10:38 PM
IsJusta Decon Carried Items List	2.2.3	11kB	IsJustaGhost	11/10/21 06:07 PM
IsJusta Decon Carried Items List	2.2.2	10kB	IsJustaGhost	11/08/21 04:12 PM
IsJusta Decon Carried Items List	2.2.1	8kB	IsJustaGhost	10/26/21 02:23 PM
IsJusta Decon Carried Items List	2.2	8kB	IsJustaGhost	10/26/21 02:10 PM
IsJusta Decon Carried Items List	2.1.4	7kB	IsJustaGhost	10/25/21 03:21 PM
IsJusta Decon Carried Items List	2.1.4	7kB	IsJustaGhost	10/25/21 03:07 PM
IsJusta Decon Carried Items List	2.1.3	7kB	IsJustaGhost	10/25/21 02:19 PM
IsJusta Decon Carried Items List	2.1.2	7kB	IsJustaGhost	10/24/21 12:39 PM
IsJusta Decon Carried Items List	2.1.1	7kB	IsJustaGhost	10/04/21 02:03 AM
IsJusta Decon Carried Items List	2.1	7kB	IsJustaGhost	10/03/21 09:05 PM
IsJusta Decon Carried Items List	2.0.1	6kB	IsJustaGhost	08/25/21 12:44 PM
IsJusta Decon Carried Items List	2	5kB	IsJustaGhost	06/13/21 07:24 AM
IsJusta Decon Carried Items List	1.5.3	5kB	IsJustaGhost	06/03/21 12:21 PM
IsJusta Decon Carried Items List	1.5.1	5kB	IsJustaGhost	03/16/21 11:55 PM
IsJusta Decon Carried Items List	1.5	4kB	IsJustaGhost	03/03/21 09:25 PM
IsJusta Decon Carried Items List	1.4.1	4kB	IsJustaGhost	02/19/21 08:38 AM
IsJusta Decon Carried Items List	1.4	4kB	IsJustaGhost	11/17/20 06:34 PM
IsJusta Decon Carried Items List	1.3	6kB	IsJustaGhost	09/09/20 06:35 AM
IsJusta Decon Carried Items List	1.2	3kB	IsJustaGhost	09/07/20 08:23 AM
IsJusta Decon Carried Items List	1.1.1	3kB	IsJustaGhost	09/07/20 05:48 AM
IsJusta Decon Carried Items List	1.1.1	11kB	IsJustaGhost	08/28/20 01:06 AM
IsJusta Decon Carried Items List	1.1	3kB	IsJustaGhost	08/27/20 12:32 AM
IsJusta Decon Carried Items List	1	2kB	IsJustaGhost	08/21/20 11:16 AM

Comment Options

Page 1 of 7

Last »

07/13/24, 07:13 AM

Tiath

Forum posts: 0

File comments: 9

Uploads: 0

Originally Posted by IsJustaGhost

I may have implemented this change incorrectly

No. I think you're right and I made a mistake somewhere.

Today I reinstalled 3.3.1 from scratch and only changed shouldAddItem to:

Lua Code:

local function shouldAddItem(bagId, slotIndex)
    local filters = {}
    
    -- only need to run this onse per item
    if IsItemBoPAndTradeable(bagId, slotIndex) then
        return savedVars.ignoreTradeable
    end
    
    -- Add relevent filters in order
    if IsItemStolen(bagId, slotIndex) then
        if savedVars.ignoreStolen then
            return false
        else
            table.insert(filters, qualityFilters['stolen'])
        end
    end
 
    -- Quality filters.
    local hasSet = GetItemLinkSetInfo(GetItemLink(bagId, slotIndex))
    if hasSet then
        table.insert(filters, qualityFilters['setItems'])
    end
    
    local traitInformation = GetItemTraitInformation(bagId, slotIndex)
    if qualityFilters[traitInformation] then
        table.insert(filters, qualityFilters[traitInformation])
    end
    
    if ZO_IsTableEmpty(filters) then
        table.insert(filters, qualityFilters['default'])
    end
    
    local displayQuality = GetItemDisplayQuality(bagId, slotIndex)
    
    for _, filter in ipairs(filters) do
        -- first filter, in order of added, returns true, stop and add item.
        if not filterItem(filter, displayQuality) then
            return false
        end
    end
    return true
end

And it appears to be working as expected...

After collecting more set pieces to test with, it's all fine using 3.3.1 and the modified shouldAddItem. It looks like any weirdness with the filterTable must have been a mistake on my part.

Last edited by Tiath : 07/13/24 at 09:20 AM.

07/11/24, 08:42 AM

IsJustaGhost

Forum posts: 39

File comments: 333

Uploads: 23

Originally Posted by Tiath

Originally Posted by IsJustaGhost

This is the true order of the qualityFilters. But, the order does not matter for this table.

Yes. Sorry, I agree with you: I can see that it shouldn't depend on how filterTable is ordered, because shouldAddItem is constructing its own local filters table.

But it does work if you change the filterTable order - and doesn't work if you don't

I can't see why that is at the moment (it's weird), and I need to go away for work so can't tinker with it any more until the weekend.

Everything in that statemen should be true, except for "filterTable order"
There are often things that occur with addons that make no sense.

How I changed this part

Lua Code:

for _, filter in ipairs(filters) do
        if not qualityFilter(filter, displayQuality) then 
-- if this returns false then break and do not add the item, else check next filter. If the filter is added but disabled, it returns false and dose not add item.
-- in other words if qualityFilter(filter, displayQuality) == false then
            return false
        end
    end

If no filter returns false, then return true and add item.
return true

The idea behind this change is , if a filter passes but another is disabled or does not, then the item is not added. So, only false flitters will prevent an item from being added. Instead of checking for a single match as it did before.

I may have implemented this change incorrectly. The above is how it should work in theory.

Any disabled filter should return false and not add the item. if quality is not in range then it should return false and not add the item. I mean, a return is absolute. everything in shouldAddItem is stopped and the value is returned. At that point. do what it must for the reason shouldAddItem was called, and move on to next item.

Oh, I do ipairs the filterTable but, since I do not use the index I just hash it out.
for _, info in ipairs(filterTable) do

07/11/24, 08:02 AM

Tiath

Forum posts: 0

File comments: 9

Uploads: 0

Originally Posted by IsJustaGhost

This is the true order of the qualityFilters. But, the order does not matter for this table.

I can't see why that is at the moment (it's weird), and I need to go away for work so can't tinker with it any more until the weekend.

Last edited by Tiath : 07/11/24 at 08:19 AM.

07/11/24, 07:45 AM

IsJustaGhost

Forum posts: 39

File comments: 333

Uploads: 23

only one of these can be used per item
ornate, intricate, retraited and reconstructed
as it depends on what trait the item has.

If the trait exists on the item then, it adds that filter to the table.

Lua Code:

local traitInformation = GetItemTraitInformation(bagId, slotIndex)
    if qualityFilters[traitInformation] then
        table.insert(filters, qualityFilters[traitInformation])
    end

With that said..
As you can see, filterTable is not what is used to get the filter. That table is used to build the filters by "id" in
self.savedVars.traitOptions[info.id] = traitOption
and
qualityFilters[info.id]

Example qualityFilters["stolen"]
The only thing effected by table order...

This "ipairs"

Lua Code:

for _, filter in ipairs(filters) do
        if not qualityFilter(filter, displayQuality) then
            return false
        end
    end

and that order is based no this order

Lua Code:

if IsItemStolen(bagId, slotIndex) then
        if savedVars.ignoreStolen then
            return false
        else
            table.insert(filters, qualityFilters['stolen']) -- 1
        end
    end
 
    -- Quality filters.
    local hasSet = GetItemLinkSetInfo(GetItemLink(bagId, slotIndex))
    if hasSet then
        table.insert(filters, qualityFilters['setItems']) -- 2
    end
    
    local traitInformation = GetItemTraitInformation(bagId, slotIndex)
    if qualityFilters[traitInformation] then
        table.insert(filters, qualityFilters[traitInformation]) -- 3
    end
    
    if ZO_IsTableEmpty(filters) then
        table.insert(filters, qualityFilters['default']) -- 5
    end
    -- Depending on what ones are actually added. 
    -- if it's not stolen then 1 is setItems

This is the true order of the qualityFilters. But, the order does not matter for this table.

Lua Code:

qualityFilters[1] = { -- ITEM_TRAIT_INFORMATION_INTRICATE == 1
    name = GetString(SI_ITEMTRAITTYPE9),
    id = ITEM_TRAIT_INFORMATION_INTRICATE,
}
qualityFilters[2] = { -- ITEM_TRAIT_INFORMATION_ORNATE == 2
    name = GetString(SI_ITEMTRAITTYPE10),
    id = ITEM_TRAIT_INFORMATION_ORNATE,
}
qualityFilters[3] = { -- ITEM_TRAIT_INFORMATION_CAN_BE_RESEARCHED == 3
    name = GetString('SI_ITEMTRAITINFORMATION', ITEM_TRAIT_INFORMATION_CAN_BE_RESEARCHED),
    id = ITEM_TRAIT_INFORMATION_CAN_BE_RESEARCHED,
}
qualityFilters[4] = {-- ITEM_TRAIT_INFORMATION_RETRAITED == 4
    name = GetString('SI_ITEMTRAITINFORMATION', ITEM_TRAIT_INFORMATION_RETRAITED),
    id = ITEM_TRAIT_INFORMATION_RETRAITED,
}
qualityFilters[5] = { -- ITEM_TRAIT_INFORMATION_RECONSTRUCTED == 5
    name = GetString('SI_ITEMTRAITINFORMATION', ITEM_TRAIT_INFORMATION_RECONSTRUCTED),
    id = ITEM_TRAIT_INFORMATION_RECONSTRUCTED,
}
qualityFilters['default'] = { -- default
    name = VAR_DEFAULT_STRING,
    id = 'default',
}
qualityFilters['setItems'] = { -- setItems
    name = GetString(SI_ITEM_SETS_BOOK_TITLE),
    id = 'setItems',
}
qualityFilters['stolen'] = { -- stolen
    name = GetString(SI_GAMEPAD_ITEM_STOLEN_LABEL),
    id = 'stolen',
    disabledCondition = function() return savedVars.ignoreStolen end,
}

Those numbers, 1-5, can't be changed. As they mean something to the game.

Last edited by IsJustaGhost : 07/11/24 at 08:13 AM.

07/11/24, 06:01 AM

Tiath

Forum posts: 0

File comments: 9

Uploads: 0

Originally Posted by IsJustaGhost

If you would like to test that.

This version of shouldAddItem works as expected (so far), if you also change filterTable to be in order of specificity:

i.e. ornate, intricate, retraited and reconstructed are mutually exclusive so their order isn't important, but the "bigger" filters that can have members that are also in other filters (e.g. a stolen set item that can be researched) should be ordered by increasing size at the bottom of the table: This allows shouldAddItem to make an exclusion decision based on the smaller, more specific, filter sets first.

Lua Code:

local filterTable = {
    { -- ITEM_TRAIT_INFORMATION_ORNATE
        name = GetString(SI_ITEMTRAITTYPE10),
        id = ITEM_TRAIT_INFORMATION_ORNATE,
    },
    { -- ITEM_TRAIT_INFORMATION_INTRICATE
        name = GetString(SI_ITEMTRAITTYPE9),
        id = ITEM_TRAIT_INFORMATION_INTRICATE,
    },
    { -- ITEM_TRAIT_INFORMATION_RETRAITED
        name = GetString('SI_ITEMTRAITINFORMATION', ITEM_TRAIT_INFORMATION_RETRAITED),
        id = ITEM_TRAIT_INFORMATION_RETRAITED,
    },
    { -- ITEM_TRAIT_INFORMATION_RECONSTRUCTED
        name = GetString('SI_ITEMTRAITINFORMATION', ITEM_TRAIT_INFORMATION_RECONSTRUCTED),
        id = ITEM_TRAIT_INFORMATION_RECONSTRUCTED,
    },
    { -- ITEM_TRAIT_INFORMATION_CAN_BE_RESEARCHED
        name = GetString('SI_ITEMTRAITINFORMATION', ITEM_TRAIT_INFORMATION_CAN_BE_RESEARCHED),
        id = ITEM_TRAIT_INFORMATION_CAN_BE_RESEARCHED,
    },
    { -- setItems
        name = GetString(SI_ITEM_SETS_BOOK_TITLE),
        id = 'setItems',
    },
    { -- stolen
        name = GetString(SI_GAMEPAD_ITEM_STOLEN_LABEL),
        id = 'stolen',
        disabledCondition = function() return savedVars.ignoreStolen end,
    },
    { -- default
        name = VAR_DEFAULT_STRING,
        id = 'default',
    },
}

07/10/24, 11:18 PM

IsJustaGhost

Forum posts: 39

File comments: 333

Uploads: 23

Perhaps if I invert it and return false for first failed filter and return true by default

A disabled fitter would instantly return false. true false is add to list or not.

That may be just what's needed. I made the changes and tested it against my meager items and it's functionality had not changed. This means that inverting it did not seam to break it.

If you would like to test that.

Lua Code:

local function shouldAddItem(bagId, slotIndex)
    local filters = {}
    
    -- only need to run this onse per item
    if IsItemBoPAndTradeable(bagId, slotIndex) then
        return savedVars.ignoreTradeable
    end
    
    -- Add relevent filters in order
    if IsItemStolen(bagId, slotIndex) then
        if savedVars.ignoreStolen then
            return false
        else
            table.insert(filters, qualityFilters['stolen'])
        end
    end
 
    -- Quality filters.
    local hasSet = GetItemLinkSetInfo(GetItemLink(bagId, slotIndex))
    if hasSet then
        table.insert(filters, qualityFilters['setItems'])
    end
    
    local traitInformation = GetItemTraitInformation(bagId, slotIndex)
    if qualityFilters[traitInformation] then
        table.insert(filters, qualityFilters[traitInformation])
    end
    
    if ZO_IsTableEmpty(filters) then
        table.insert(filters, qualityFilters['default'])
    end
    
    local displayQuality = GetItemDisplayQuality(bagId, slotIndex)
    
    for _, filter in ipairs(filters) do
        -- first filter, in order of added, returns true, stop and add item.
        if not filterItem(filter, displayQuality) then
            return false
        end
    end
    return true
end

Last edited by IsJustaGhost : 07/10/24 at 11:36 PM.

07/10/24, 04:30 PM
Tiath Forum posts: 0 File comments: 9 Uploads: 0	Thanks for the patch I've just spotted something else that seems counter-intuitive with 3.3.1 (and is probably a filter precedence issue). If you have "Set items" ON but "Can research" OFF, set items that have traits which can be researched are included in Add All. My expectation with those filter settings is that "Can research" takes precedence and set items which can also be researched would be excluded.

07/09/24, 07:56 PM

IsJustaGhost

Forum posts: 39

File comments: 333

Uploads: 23

Originally Posted by Tiath

Yes, that looks like it.

I replaced shouldAddItem with your new version and it's working as expected, both with the "Ignore stolen" toggle and the stolen quality filter.

If that's confirmed, then I'll go ahead and patch this.

Thank you for the assistance in improving this add-on.

07/09/24, 04:22 PM

Tiath

Forum posts: 0

File comments: 9

Uploads: 0

Originally Posted by IsJustaGhost

The issue is likely because of this.

Lua Code:

if ZO_IsTableEmpty(filters) then
    table.insert(filters, qualityFilters['default'])
end

Yes, that looks like it.

I replaced shouldAddItem with your new version and it's working as expected, both with the "Ignore stolen" toggle and the stolen quality filter.

07/09/24, 01:37 PM

IsJustaGhost

Forum posts: 39

File comments: 333

Uploads: 23

The issue is likely because of this.

Lua Code:

local function shouldAddItem(bagId, slotIndex)
    local filters = {}
    
    -- Add relevent filters in order
    if IsItemStolen(bagId, slotIndex) then
-- Changing the flat stolen check like this then.  If it's enabled, then stop here and deny the item.
        if savedVars.ignoreStolen then
            return false
        end
        table.insert(filters, qualityFilters['stolen'])
    end
 
    local hasSet = GetItemLinkSetInfo(GetItemLink(bagId, slotIndex))
    if hasSet then
        table.insert(filters, qualityFilters['setItems'])
    end
    
    local traitInformation = GetItemTraitInformation(bagId, slotIndex)
    if qualityFilters[traitInformation] then
        table.insert(filters, qualityFilters[traitInformation])
    end
    
    table.insert(filters, qualityFilters['default'])

It's adding default with no conditions.
I'll try with

Lua Code:

if ZO_IsTableEmpty(filters) then
    table.insert(filters, qualityFilters['default'])
end

That way it only adds it if no other filter is added.
Id default is not added with other filters, it should stop disabled filters from returning false positives.

The complete function

Lua Code:

local function shouldAddItem(bagId, slotIndex)
    local filters = {}
    
    -- Add relevent filters in order
    if IsItemStolen(bagId, slotIndex) then
        if savedVars.ignoreStolen then return false end
        table.insert(filters, qualityFilters['stolen'])
    end
 
    local hasSet = GetItemLinkSetInfo(GetItemLink(bagId, slotIndex))
    if hasSet then
        table.insert(filters, qualityFilters['setItems'])
    end
    
    local traitInformation = GetItemTraitInformation(bagId, slotIndex)
    if qualityFilters[traitInformation] then
        table.insert(filters, qualityFilters[traitInformation])
    end
    
    if ZO_IsTableEmpty(filters) then
        table.insert(filters, qualityFilters['default'])
    end
    
    local displayQuality = GetItemDisplayQuality(bagId, slotIndex)
    
    for _, filter in ipairs(filters) do
        if filter:IsEnabled() and not filter:IgnoreTradeable(bagId, slotIndex) and defaultQualityFilter(filter, displayQuality) then
            return true
        end
    end
    return false
end

Can just be coppied and pasted in under the current one. Everything is read from top down and, since the function already exists, it is replaced by the lower one.

"default" should only work on things that are crafted for decon. No traits, no set item. Just plain axe or whatever..

07/09/24, 11:16 AM

Tiath

Forum posts: 0

File comments: 9

Uploads: 0

Originally Posted by IsJustaGhost

I added those options.
I was not able to fully test it, however. The ability to have every deconstruct able item in every quality and in every trait and, now, stolen and not, is way beyond me.
But, I tested it for general errors and basic functionality. If anything else comes up, just let me know and I'll do what I can.

I understand it's a massive undertaking to extensively test all combinations.

Something weird is happening with the filters, though... maybe a boolean double-negative somewhere (I'd offer to help poke around under the hood but don't know the API and haven't written any Lua in ~20 years

)

I've submitted a bug report for you.

It's not the end of the world if you need to revert, though: The downside is only having to be more selective when out in the field or going to visit a fence first.

07/09/24, 07:48 AM

IsJustaGhost

Forum posts: 39

File comments: 333

Uploads: 23

Originally Posted by Tiath

Generally, if you're optimising, you want to launder stolen items before deconstructing: You get less materials and inspiration back from deconstructing items while they're still flagged as stolen.

My preference would be to add a general "Ignore stolen items" like you already have for group tradeable items.

Last edited by IsJustaGhost : 07/09/24 at 07:49 AM.

07/09/24, 04:27 AM

IsJustaGhost

Forum posts: 39

File comments: 333

Uploads: 23

Originally Posted by Tiath

I'm back to it.
For now, what I'm going to do is include the 2 options.
The 1 will cancel out the other.
If ignoreStolen is enabled. the stolen quality filter will be disabled. If the stolen quality filter is enabled...
At least, that is my intent for now.

07/07/24, 07:29 AM
Tiath Forum posts: 0 File comments: 9 Uploads: 0	Generally, if you're optimising, you want to launder stolen items before deconstructing: You get less materials and inspiration back from deconstructing items while they're still flagged as stolen. My preference would be to add a general "Ignore stolen items" like you already have for group tradeable items.

07/07/24, 04:49 AM

IsJustaGhost

Forum posts: 39

File comments: 333

Uploads: 23

Re: Can we have a filter for Stolen Items?

Originally Posted by Tiath

Can we have a filter that ignores stolen items from "Add All"?

I'd prefer to launder or sell them to a fence, rather than have them swept up in deconstruction.

Right. I'll look into adding that.
On that note, any other suggestions?

One way I can do this is, add a filter for stolen to all filters. With a single check box option to ignore stolen.

Or, I can work on adding an ignore stolen per quality filter.

Or, it can be add as a quality filter for stolen. If the item is between this level and that level then allow it to be added.

Last edited by IsJustaGhost : 07/07/24 at 05:43 AM.

Page 1 of 7

Last »

Report this File