Use isset to address the deprecated message related to $check_key_exists parameter from AssetDataRegistry

[nefeline]

Reported by @roykho on #46139, opening this issue so we can circle back and ensure that:

For this since we're deprecating the parameter, what do you think if we check for isset instead. This way, the caller will get a deprecated feedback whether they passed 'true' or 'false' there.

[shameemreza]

Posibly Another report here: 8131880-zen
Thread link: https://wordpress.org/support/topic/log-messages-the-automatticwoocommerceblocksassetsassetdataregistry/

The Automattic\WooCommerce\Blocks\Assets\AssetDataRegistry::add() argument is deprecated since version 8.9. The $check_key_exists parameter is no longer used: all duplicate data will be ignored if the key exists by default

• WC Version: 8.8.3
• WP Version: 6.5.2
• HP Version: 8.0.30

[nefeline]

Hi @shameemreza 👋 !

Could you please refer to #46744 (comment) ? That issue has further details on this.

In summary, the decision to deprecate the $check_key_exists parameter was deliberate, and here on this issue we are just considering expanding the notice to all use-cases.