[Bug]: discardActiveObject called from object:modified handler causes loop [nuxtjs]

[matiszac]

CheckList

• I agree to follow this project's Code of Conduct
• I have read and followed the Contributing Guide
• I have read and followed the Issue Tracker Guide
• I have searched and referenced existing issues and discussions
• I am filing a BUG report.
• I have managed to reproduce the bug after upgrading to the latest version
• I have created an accurate and minimal reproduction

Version

6.0.2

In What environments are you experiencing the problem?

Firefox

Node Version (if applicable)

None

Link To Reproduction

https://stackblitz.com/edit/nuxt-starter-638zzj?file=app.vue

Steps To Reproduce

1. create canvas
2. create event handler that calls canvas.discardActiveObject()
3. attach handler to canvas.on('object:modified', handler)
4. create any object, like a rect and add it to the canvas.
5. select the object and move it, when releasing the mouse it will trigger object:modified and call the handler

Expected Behavior

I expect that it wont trigger object modifed because when calling discardActiveObject from a button lets say, it does not trigger the object modifed event.

Actual Behavior

it will discard and modify in a loop

Error Message & Stack Trace

too many recursions

[asturur]

Discarding an object that is currently trasforming will terminate its transform action.
If what you want is to deselect on object:modified i would suggest you get target.canvas._currentTransform and you set it to null before deselecting.

I think the code should be fixed to avoid this infinite loop anyway.