116 - Merchant - AutoSell unusable BoP weapons and armor
What does the provided patch do?
Provided patch adds functionality of selling Soulbound items (weapons and armor) that can't be equipped by our character. Also provided is option to turn this functionality on and off (default off).
Please provide any additional information below.
I was inspired by similar functionality in Reagent Restocker addon by CobraA1. Item checking is done in very similar way. Maybe tooltip scanning for Armor and Weapon skill, but that may require some more parsing, as Armor Skills spells tooltips are not consistent, or maybe some other way, definitely will think about it.
| Name | Description | Size | MD5 |
|---|---|---|---|
| merchant.patch | Patch for r332 created with ... | 7.8 KiB | 635da1f24eba... |
| merchant.patch | Updated patch for r334 created ... | 7.8 KiB | 9e39dc43a19b... |
| merchant.patch | Updated patch for r380 created ... | 7.8 KiB | 166b3ec33efb... |
| merchant.patch | Updated patch for r386 created ... | 7.8 KiB | a150c7d5be61... |
| merchant.patch | Updated patch for r395 created ... | 7.8 KiB | deee25bdd362... |
| merchant.patch | Updated patch for r402 created ... | 7.8 KiB | c3585e18c16b... |
| User | When | Change |
|---|---|---|
| deadlyslaya | May 14, 2012 at 22:06 UTC | Changed status from New to Declined |
| tomkuzyno | Oct 11, 2011 at 09:06 UTC | Added attachment merchant.patch |
| tomkuzyno | Sep 26, 2011 at 15:09 UTC | Added attachment merchant.patch |
| tomkuzyno | Sep 17, 2011 at 06:46 UTC | Added attachment merchant.patch |
| tomkuzyno | Sep 16, 2011 at 10:22 UTC | Added attachment merchant.patch |
| tomkuzyno | Aug 30, 2011 at 06:46 UTC | Changed description:+ plain$What does the provided patch do? + Provided patch adds functionality of selling Soulbound items (weapons and armor) that can't be equipped by our character. Also provided is option to turn this functionality on and off (default off). + + Please provide any additional information below. + I was inspired by similar functionality in Reagent Restocker addon by CobraA1. Item checking is done in very similar way. Maybe tooltip scanning for Armor and Weapon skill, but that may require some more parsing, as Armor Skills spells tooltips are not consistent, or maybe some other way, definitely will think about it. |
| tomkuzyno | Aug 30, 2011 at 06:45 UTC | Added attachment merchant.patch |
| tomkuzyno | Aug 29, 2011 at 17:03 UTC | Deleted attachment merchant.patch: deprecated |
| tomkuzyno | Aug 29, 2011 at 17:03 UTC | Added attachment merchant.patch |
| tomkuzyno | Aug 29, 2011 at 17:02 UTC | Changed description:Provided patch adds functionality of selling Soulbound items (weapons and armor) that can't be equipped by our character. Also provided is option to turn this functionality on and off (default off). Please provide any additional information below. - I was inspired by similar functionality in Reagent Restocker addon by CobraA1. Item checking is done in the same way, as I was unable for now to find something different (i don't like this chain-if's, but better this than nothing). Maybe tooltip scanning for Armor and Weapon skill, but that may require some more parsing, as Armor skill tooltip is not consistent between cloth-wearing classes and others, or maybe some other way, definitely will think about it. + I was inspired by similar functionality in Reagent Restocker addon by CobraA1. Item checking is done in very similar way. Maybe tooltip scanning for Armor and Weapon skill, but that may require some more parsing, as Armor Skills spells tooltips are not consistent, or maybe some other way, definitely will think about it. |
| tomkuzyno | Aug 29, 2011 at 14:05 UTC | Added attachment merchant.patch |
| tomkuzyno | Aug 29, 2011 at 14:05 UTC | Create |
- 1 comment
- 1 comment
Facts
- Last updated
- May 14, 2012
- Reported
- Aug 29, 2011
- Status
- Declined - We decided not to take action on this ticket.
- Type
- Patch - Source code patch for review
- Priority
- Medium - Normal priority.
- Votes
- 1
- Reply
- #2
deadlyslaya May 14, 2012 at 22:05 UTC - 0 likesAfter going through this and implementing it on my end, even though it was a nice idea, I decided not to include it. It takes to much time to scan each tooltip and to much memory for the tables for such a small feature.
Zista