Wrap useDebounce call inside useCallback

[amustaque97]

Fixes: #33839

Description

This is done in order to avoid function call getting cancelled on multiple refresh

How has this been tested?

Screenshots

Types of changes

Checklist:

• My code is tested.
• My code follows the WordPress code style.
• My code follows the accessibility standards.
• I've tested my changes with keyboard and screen readers.
• My code has proper inline documentation.
• I've included developer documentation if appropriate.
• I've updated all React Native files affected by any refactorings/renamings in this PR (please manually search all *.native.js files for terms that need renaming or removal).

[ZebulanStanphill]

You shouldn't put refs inside dependency callbacks. It doesn't work the way you might think it works:

https://epicreact.dev/why-you-shouldnt-put-refs-in-a-dependency-array/

Also, becuase prevProps is an object, it will fail the identity equality check on every render due to props being a new object on every render. And prevProps is not even used inside the callback, so it shouldn't be in the dependency array anyway.

attributes, httpMethod, and urlQueryArgs are the actual dependencies of the callback. httpMethod can easily be added to the dependency array, but because attributes and urlQueryArgs are objects, you can't just put them in the dependency array directly, so you'd have to store their versions from the prior render using usePrevious, perform deep equality checks using Lodash's equals, and put the boolean result of those comparisons in the dependency array.

[tyxla]

Just a side note that since we no longer use Lodash in the entire repository (see #52571), we're using a couple of methods for comparison:

• shallow comparison: see the @wordpress/is-shallow-equal package.
• deep comparison: the fast-deep-equal package.

[Mamaduka]

Hi, @amustaque97

This PR came up during the weekly editor bug scrub, and I wanted to check if you plan to continue working on it.

[amustaque97]

Hi @Mamaduka, by this weekend. I should be able to work on the review comments 🙂

[Mamaduka]

Thanks for the reply, @amustaque97!

Zero pressure, I just wanted to check on the status 👍

[Mamaduka]

Thanks for working on this, @amustaque97! I believe this issue was resolved as part of different bug report - #69237.