12 - Auto-sell does not work if repair from guild funds fails
Submitted Via
Curse Client 4.0.1.2, Microsoft Windows NT 6.1.7600.0
Installed Version
v1.2.1
Description
If the addon tries to auto-repair with guild funds but fails because the guild currently does not allow it, then no gray items are sold.
Items should be sold even if auto-repair falls back from guild to own funds.
Steps to Reproduce
Have gray items in the bags.
Enable "Selling > Sell gray items".
Enable "Repair > Use guild bank funds".
Make sure that the guild bank has repairs not enabled.
Talk to a vendor that also repairs.
The addon tries to repair with guild funds but fails and repairs with own funds.
No gray items are sold.
I have not tested with items from the selling list but this should be checked as well.
Installed Addons
!BugGrabber - r143
A3Fu_Infoclock - A3Fu_Infoclock 1.3
Acheron - 1.1.10
AddonLoader - r106
Advanced Trade Skill Window - v0.7.7
AHsearch - 30200-2
ArkInventory - 3.02.24
Auctioneer - Auctioneer Suite 5.6.4424
AutoCombatLog - v1.2.3
BadBoy: Anti Spam Automatic Reporter - v3.6.0.0
BadBoy_Levels: Filter Whispers By Level - v2.1.2
Bartender4 - 4.4.1
Broker Recommended Zone - r77
Broker Recount Statistics - v1.72
Broker_DungeonDifficulty - 1.0
Broker_repair - v2.1_LovelyEdition
BuffEnough - 1.1.15
Buffet - 3.1.0.29-Release
BugSack - r175
cAutoConfirmLoot - cAutoConfirmLoot-1.3
Ceaphyrel's Auto Accept Invite - v.1.2.4 beta
Chatter - r20090919214124
ChocolateBar Broker Display - r82
Clique - r125-release
ConfigModeBlizzard - v1.0
ConfigModeDemos - v1.1-beta
Confirm Release - Confirm Release 1.5
Deadly Boss Mods - Deadly Boss Mods 4.29b-r2049
Deadly Boss Mods - Burning Crusade mods - DBM - Burning Crusade Mods v4-r25
Deadly Boss Mods - LibDataBroker - Deadly Boss Mods - LibDataBroker 4.10-r9
DrDamage - r1115
Elkano's BuffBars - 2.2.5 - r150
EnsidiaFails - r182
EPGPLootmaster - 0.4.4
ErrorToggle - ErrorToggle
FastQuest - 1.3
FastZoom - 3.1
FuBar_AddonSpamFu - r94
FuBar2Broker - r86-release
Gem Quota - r20090805
Grid - r1226
Grid_QuickHealth - Grid_QuickHealth-r81597.2
GridAlert - r51
GridIndicatorIconBar - v1.2.1
GridIndicatorText3 - 30200-1beta
GridSideIndicators - r40
GridStatusAFK - r39
GridStatusBanzai - 1.01
GridStatusHostileUnit - v0.71
GridStatusMending - v3.01.01.03 release
GridStatusMissingBuffs - r76
GridStatusMTs - 30200-1
GridStatusParty - v0.92
GridStatusRaidDebuff - r244
GridStatusRaidIcons - r38
GridStatusRes - v3.00.03.03 release
GridUnitMenu - 30100.2
HandyNotes - v1.1.4
IHML - I Have Macro Lol! - 2.7Beta-10-gab5d925
LibSink-2.0 - r67
Mapster - 1.2.3
Omen Threat Meter - v3.0.8-2-g32a9369
OmniCC - 2.5.7
OneButtonConfig - v1.2
oRA2 - r666-release
Outfitter - 4.8
Parrot - v.1.10.4-1-g38443da
PassLoot - r84
Pawn - 1.2.5
pError - 30200.3-Beta
PitBull Unit Frames 4.0 - v4.0.0-beta5-23-ge4c6b5b
pMinimap - 30200.5-Release
Postal - v3.2.2
Prayer of Mending Tracker - PoM Tracker 2.63
Quartz - r23-release
QuestHelper - 1.2.3
Reagent Restocker - v1.2.1
Recount - v3.2h beta3
SexyCooldown - v0.6.8-1-g0097e2a
SmartMount - v3.2-96
StickyLFG - StickyLFG 1.8
SurgeonGeneral - 1.8.0
Talented - r628
teksLoot - 3.2.0.7-Beta
TomTom - r226-release
TradeskillInfo - r346
Volumizer - 3.2.2-1.2.0
WinterTime - WinterTime
XLoot - r365
| User | When | Change |
|---|---|---|
| CobraA1 | Sun, 10 Jan 2010 19:08:11 | Changed status from Accepted to Declined |
| CobraA1 | Wed, 02 Dec 2009 07:09:59 | Changed milestone from None to Minor feature wrapup |
| CobraA1 | Sat, 07 Nov 2009 05:50:02 | Changed status from New to Accepted |
| Laren | Sun, 01 Nov 2009 12:12:09 | Create |
- 4 comments
- 4 comments
- #4
CobraA1 Sun, 10 Jan 2010 19:05:22Sorry, tried it, and unable to reproduce. I also don't see the issue in the code itself. For now, there's not much I can do except to say please record any error messages that appear in chat, and the workaround it simply to visit the vendor again, or to just turn off using guild funds for repairs if they don't allow it.
- #3
CobraA1 Sun, 13 Dec 2009 18:59:08Not fixed yet. I'm doing a major library update right now (moving from Waterfall to Ace 3), which is taking priority. It will be looked at after the library update.
- #2
Laren Sat, 12 Dec 2009 17:42:07I think this is fixed now, or? :)
- #1
CobraA1 Sat, 07 Nov 2009 05:53:04Heh, one of the previous authors even marked a TODO entry in the file. This is an old bug. But I'll look into it when I have the time.