chore(): move away from extend/clone

[ShaMan123]

A semantic PR that replaces fabric.util.object.extend or the shallow version of fabric.util.object.clone with Object.assign to prepare for the js/ts migration.
I replaced relevant code after using the transformer (It uses extend as a search value).
The deep version of clone is used sparsely and in the future could be replaced by lodash.cloneDeep (~44KB vs. ~4KB of lang_object 😬). BTW I'm not sure it is really necessary in most cases.

[github-actions]

Code Coverage Summary

> fabric@5.1.0 coverage:report
> nyc report --reporter=lcov --reporter=text

-----------|---------|----------|---------|---------|-----------------------------------------------
File       | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s                             
-----------|---------|----------|---------|---------|-----------------------------------------------
All files  |   83.06 |    76.01 |    85.7 |   82.78 |                                               
 fabric.js |   83.06 |    76.01 |    85.7 |   82.78 | ...,30876,30950,30961-31026,31149,31248,31484 
-----------|---------|----------|---------|---------|-----------------------------------------------

[github-actions]

Code Coverage Summary

> fabric@5.1.0 coverage:report
> nyc report --reporter=lcov --reporter=text

-----------|---------|----------|---------|---------|-----------------------------------------------
File       | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s                             
-----------|---------|----------|---------|---------|-----------------------------------------------
All files  |   83.07 |       76 |    85.7 |   82.79 |                                               
 fabric.js |   83.07 |       76 |    85.7 |   82.79 | ...,30864,30938,30949-31014,31137,31236,31472 
-----------|---------|----------|---------|---------|-----------------------------------------------

[github-actions]

Code Coverage Summary

> fabric@5.1.0 coverage:report
> nyc report --reporter=lcov --reporter=text

-----------|---------|----------|---------|---------|-----------------------------------------------
File       | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s                             
-----------|---------|----------|---------|---------|-----------------------------------------------
All files  |   83.06 |       76 |    85.7 |   82.79 |                                               
 fabric.js |   83.06 |       76 |    85.7 |   82.79 | ...,30862,30936,30947-31012,31135,31234,31470 
-----------|---------|----------|---------|---------|-----------------------------------------------

[github-actions]

Code Coverage Summary

> fabric@5.1.0 coverage:report
> nyc report --reporter=lcov --reporter=text

-----------|---------|----------|---------|---------|-----------------------------------------------
File       | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s                             
-----------|---------|----------|---------|---------|-----------------------------------------------
All files  |   83.05 |    75.97 |    85.7 |   82.77 |                                               
 fabric.js |   83.05 |    75.97 |    85.7 |   82.77 | ...,30861,30935,30946-31011,31134,31233,31469 
-----------|---------|----------|---------|---------|-----------------------------------------------

[ShaMan123]

Take a close look at src/gradient.class.js tom make sure the cloning is really redundant

[asturur]

this is fine. I'll merge it now.
Let's try to not open more PRs before we can migrate.

[ShaMan123]

Agreed. I opened this to make the migration easier. Less complication בתאריך יום א׳, 12 ביוני 2022, 15:00, מאת Andrea Bogazzi ‏< ***@***.***>:

…

Merged #8001 <#8001> into master. — Reply to this email directly, view it on GitHub <#8001 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AIGAW4IDDYTTVZ3ZS6GV3CTVOXGMBANCNFSM5YRC4HOA> . You are receiving this because you authored the thread.Message ID: ***@***.***>