rezecib

In the middle of the component PlayerController:OnUpdate, there's a check: if self.handler ~= nil and self.inst:HasTag("usingmagiciantool") then self:CancelPlacement() self:CancelDeployPlacement() self:CancelAOETargeting() end However, below this, there's a check to see if the player has an item on their cursor, which then starts placement mode: local placer_item = controller_mode and self:GetCursorInventoryObject() or self.inst.replica.inventory:GetActiveItem() --show deploy placer if self.deploy_mode and self.placer == nil and placer_item ~= nil and placer_item.replica.inventoryitem ~= nil and placer_item.replica.inventoryitem:IsDeployable(self.inst) then local placer_name = placer_item.replica.inventoryitem:GetDeployPlacerName() local placer_skin = placer_item.AnimState:GetSkinBuild() --hack that relies on the build name to match the linked skinname if placer_skin == "" then placer_skin = nil end if self.deployplacer ~= nil and (self.deployplacer.prefab ~= placer_name or self.deployplacer.skinname ~= placer_skin) then self:CancelDeployPlacement() end if self.deployplacer == nil then self.deployplacer = SpawnPrefab(placer_name, placer_skin, nil, self.inst.userid ) if self.deployplacer ~= nil then self.deployplacer.components.placer:SetBuilder(self.inst, nil, placer_item) self.deployplacer.components.placer.testfn = function(pt) local mouseover = TheInput:GetWorldEntityUnderMouse() return placer_item:IsValid() and placer_item.replica.inventoryitem ~= nil and placer_item.replica.inventoryitem:CanDeploy(pt, mouseover, self.inst, self.deployplacer.Transform:GetRotation()), (mouseover ~= nil and not mouseover:HasTag("walkableplatform") and not mouseover:HasTag("walkableperipheral") and not mouseover:HasTag("ignoremouseover")) or TheInput:GetHUDEntityUnderMouse() ~= nil end self.deployplacer.components.placer:OnUpdate(0) --so that our position is accurate on the first frame end end else self:CancelDeployPlacement() end In the case of deploy-placement, this means that on every frame placement is cancelled and restarted immediately on the same frame. It seems that the intent here was to not allow placement, but placement is actually functional despite this, somehow. Rather than thrashing on creating and destroying the placer, maybe this should just allow placement? Or the placer creation block should check the same condition to avoid thrashing. This is particularly problematic with the Geometric Placement mod, because creating placers creates a grid, which is more expensive than just destroying and recreating the placer itself.

Expected behavior: When paused, mousing over the UI or invalid ground tiles hides the pitchfork tile indicator (matching behavior when unpaused) Observed behavior: When paused, mousing over the UI or invalid ground tiles keeps the pitchfork tile indicator and it won't disappear until unpaused I believe this is due to PlayerController:OnWallUpdate missing the section of PlayerController:OnUpdate that handles action selection and updating placers. (this came to my attention due to a crash with Geometric Placement, caused by inadventently sharing some global state that came into conflict when multiple grids are shown simultaneously)

Normally with a controller, numbers show on badges when you open the inventory (right trigger). However, in scripts/widgets/statusdisplays:ShowStatusNumbers(), it doesn't check if the moisture badge is visible when setting the number to show. This is fine when first loading in if you're dry, but the moisture badge never clears the number text after it has been set, so if you get wet and dry off, it leaves a "1" in the text. Opening the controller inventory again shows this floating "1" even though the rest of the badge is hidden. This should be an easy fix-- the moisture badge already tracks whether it's active or not in self.active, so you can just check that in statusdisplays:ShowStatusNumbers.

function Text:SetSize(sz) if LOC then sz = sz * LOC.GetTextScale() end self.inst.TextWidget:SetSize(sz) self.size = sz end function Text:SetSize(sz) return self.size end I assume the second entry was intended to get GetSize. It also seems a bit problematic that self.size stores the inflated size, so if one were to store sometextwidget.size and use it to set it later, it would cause compounding of the LOC.GetTextScale() multiplier.

In scripts/prefabs/veggies.lua, placers were added for each of the veggie seeds in the MakeVeggie function. However, they don't get added to the returned prefab table because only the first three return values are captured. This is also an obstacle for Geometric Placement compatibility, as the placer prefab is missing.