Scrap deletes all pets you've learned the maximum amount of with no option to turn it off

[Mill3rman]

Is there an existing issue for this?

• I have searched the existing open and closed issues.

Description

Starting in 11.1.8 Scrap was updated to automatically mark any pet you know the maximum amount of. Pair this with the "Destroy Worthless" option, and scrap will simply delete any pets in your bags. There is no option to change this behavior.

I have no idea who thought it was a good idea to permanently delete pets and include that as the default when there is no way to reverse it if you happened to click it because you didn't know the behavior changed, but enabling user data loss as the default is a very bad look.

At a minimum, there should be an option for this behavior, though that option should also be off by default, not on.

I have alts that have dozens of pets I sell on the AH, which means Scrap is effectively unusable at the moment.

Scrap Version

11.1.8

World of Warcraft Flavor

Retail

World of Warcraft Region

US/NA

Tested with only Scrap

I got this issue with only Scrap enabled

Lua Error

Reproduction Steps

1. Log on to a character with a caged pet that your account already knows the maximum number of.
2. Notice that the pet is marked as junk
3. Notice that there is no option to disable this "feature"
4. Notice Scrap tries to delete the pet if you sell junk
5. Notice Scrap will silently delete the pet if "Destroy Worthless" option is also checked.

Last Working Version

11.1.7

Screenshots

No response

[1Holy-Z]

ya this is a problem lol. Not updating past 11.1.7 is the only choice. That and grey items arent selling to vendor still in version 11.1.10

[PsyberWolf]

After buying it back, what happens when you drag the item to the sell button? Usually, it turns it into a nonjunk item. Does it not do this for the caged pet?

[Mill3rman]

You can't buy it back, it gets destroyed, that's a major part of the problem. Sure, you could mark all of your pets as useful, but the fact this was added as the default at all, much less with no option to turn it off is just ridiculous.

[lilflea12]

Yeah, not sure this was a great feature to add. Any pets you have in your bags that you can not learn, just get deleted. Glad I saw this before i got on my alt with all my pets that i have stored to sell.

Love the addon! But going back to version 11.1.7 before this "feature" was added until this is fixed or changed.

[Clonebrush]

This feature introduced with 11.1.8 completely ruined the addon for me, and made it very risky to use. Expensive items are being accidentally lost forever.

This new feature needs to be disabled by default and marked with a warning.

I'm reverting back to 11.1.7 until this is fixed.

[1Holy-Z]

HELLO??? PLEASE REVERT THIS OR MAKE IT OPTIONAL

[jnichel]

Just learned this the hard way. Lost an Anubisath Idol and a Viscidus Globule after running Temple of Ahn'Qiraj. Figures, two pets that sell for a good chunk of gold. Why couldn't it have been a Mini Mindslayer? I can't give those things away. Back to 11.1.7 I go.