Crafting Station Bug

[circonian]

Ug, I spent an hour trying to find a bug in an addon I'm working on & its not in my addon. Looks like a game bug, can anyone confirm?

1. Go to a clothier crafting station
2. Reload your UI to reset all of the games variables
3. Open the crafting station
4. Click on the creation tab (nothing else)
5. Go to blacksmithing crafting station and open it.
6. Click on the deconstruction tab (nothing else)
7. Go back to the clothier crafting station & open it.
8. Click on the Creation tab

It seems to happen no matter which stations you do it at as long as you only click those buttons in that order.

[Garkin]

In my opinion this is a bug on ZOS side which was introduced in patch 1.5.6 because of changes in crafting UI.

Unfortunately this error occurs in local function, so it will be a bit difficult to fix by addon.

I'd probably try to redefine GetMaterialQuantity function, something like:

function ZO_SharedSmithingCreation:GetMaterialQuantity(patternIndex, materialIndex)
    local quantity
    if self.selectedMaterialCountCache[patternIndex] then
        quantity = self.selectedMaterialCountCache[patternIndex][materialIndex]
    end
    if quantity == nil then
        local _, _, stack = GetSmithingPatternMaterialItemInfo(patternIndex, materialIndex)
        self:SetMaterialQuantity(patternIndex, materialIndex, stack)
        quantity = stack
    end
    return quantity or 0
end

Or if you want to redefine just instance of this class, you are looking for function:

SMITHING.creationPanel:GetMaterialQuantity(patternIndex, materialIndex)

[QuadroTony]

i cant reproduce it without any addons! you sure about it?

US client? i use liveeu

P.S. - the same with all my addons, i have 112
no errors so far

[Ayantir]

Hello,

Your order list isn't good

The bug is

- Go on a smithing station (wood/blacksmith/clothier)
- Open creation tab
- leave
- Go on another smithing station of same type
- Go on creation tab
- bug

It will be reproduced with any smithing station
and yes as garkin said, we can't do anything at all

let's zos fix their bugs ?

[ZOS_ChipHilseberg]

I'll take a look at it today to see if I can reproduce it.

[Garkin]

Originally Posted by ZOS_ChipHilseberg I'll take a look at it today to see if I can reproduce it.

Everywhere, where are you using method :GetMaterialQuantity is a check if return value exists. But in the SetupFunction in :InitializeMaterialList is this check missing. There should probably be something like this:

local function SetupFunction(control, data, selected, selectedDuringRebuild, enabled)
 
    --original code here
 
    local currentSelectedQuantity = self:GetMaterialQuantity(data.patternIndex, data.materialIndex)
    --added check if return value exists:
    if not currentSelectedQuantity then
        self:SetMaterialQuantity(data.patternIndex, data.materialIndex, data.min)
        currentSelectedQuantity = data.min
    end
 
   --rest of the original code
 
end

[circonian]

Originally Posted by Ayantir Hello, Your order list isn't good The bug is - Go on a smithing station (wood/blacksmith/clothier) - Open creation tab - leave - Go on another smithing station of same type - Go on creation tab - bug It will be reproduced with any smithing station and yes as garkin said, we can't do anything at all let's zos fix their bugs ?

Strange if the order I listed didn't work for you, it works for me every time. That was just the sequence of events that I managed to figure out would reproduce it every time. I don't doubt there is more than one way to reproduce it. I tried to find out where it was being caused in the code, but it 'might' be in something not available to use (or I may have missed it, I couldn't find it).

[circonian]

Originally Posted by QuadroTony i cant reproduce it without any addons! you sure about it? US client? i use liveeu P.S. - the same with all my addons, i have 112 no errors so far

Strange, I removed all addons, reloadedUI, logged completely out & back in, restarted my computer (all in case something was somehow remaining between reloadUI's). It happens every time.

[circonian]

Originally Posted by Garkin In my opinion this is a bug on ZOS side which was introduced in patch 1.5.6 because of changes in crafting UI. Unfortunately this error occurs in local function, so it will be a bit difficult to fix by addon. I'd probably try to redefine GetMaterialQuantity function, something like: function ZO_SharedSmithingCreation:GetMaterialQuantity(patternIndex, materialIndex)     local quantity     if self.selectedMaterialCountCache[patternIndex] then         quantity = self.selectedMaterialCountCache[patternIndex][materialIndex]     end     if quantity == nil then         local _, _, stack = GetSmithingPatternMaterialItemInfo(patternIndex, materialIndex)         self:SetMaterialQuantity(patternIndex, materialIndex, stack)         quantity = stack     end     return quantity or 0 end Or if you want to redefine just instance of this class, you are looking for function: SMITHING.creationPanel:GetMaterialQuantity(patternIndex, materialIndex)

Thanks for looking into it Garkin. Even though I removed all addons, logged out & back in, its nice to know its not just me.