[GTIN] Hide / disable GTIN also when YOAST is active.

[puntope]

Changes proposed in this Pull Request:

Closes part of #2617

This PR fixes a Bug in which GTIN is not hidden or disabled when YOAST is activated.

Screenshots:

Detailed test instructions:

1. Install YOAST for WooCommerce
2. Set the wp_option gla_install_versionas 2.8.6
3. Go to a Product - Google for WooCommerce
4. See the selector as disabled (if you selected "Enter custom value" it should be also disabled)
5. Set the wp_option gla_install_version as 2.8.8
6. See the selector hiden

Additional details:

Changelog entry

Fix - Hide/disable GTIN also when YOAST is active

[codecov]

Codecov Report

Attention: Patch coverage is 15.38462% with 11 lines in your changes missing coverage. Please review.

Project coverage is 65.2%. Comparing base (25d85de) to head (e5dfc63).
Report is 3 commits behind head on add/support-core-gtin-field.

Files with missing lines	Patch %	Lines
src/Admin/Input/SelectWithTextInput.php	14.3%	6 Missing ⚠️
src/Admin/Product/Attributes/AttributesForm.php	25.0%	3 Missing ⚠️
src/Admin/Input/Input.php	0.0%	2 Missing ⚠️

Additional details and impacted files

@@                       Coverage Diff                       @@
##             add/support-core-gtin-field   #2685     +/-   ##
===============================================================
+ Coverage                           64.7%   65.2%   +0.5%     
- Complexity                          4662    4674     +12     
===============================================================
  Files                                799     478    -321     
  Lines                              24627   19545   -5082     
  Branches                            1242       0   -1242     
===============================================================
- Hits                               15936   12737   -3199     
+ Misses                              8518    6808   -1710     
+ Partials                             173       0    -173     

Flag	Coverage Δ
js-unit-tests	?
php-unit-tests	65.2% <15.4%> (-0.2%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
src/Admin/Input/Input.php	97.5% <0.0%> (-2.5%)	⬇️
src/Admin/Product/Attributes/AttributesForm.php	93.5% <25.0%> (-3.8%)	⬇️
src/Admin/Input/SelectWithTextInput.php	91.4% <14.3%> (-8.6%)	⬇️

... and 322 files with indirect coverage changes

---- 🚨 Try these New Features:

• Flaky Tests Detection - Detect and resolve failed and flaky tests
• JS Bundle Analysis - Avoid shipping oversized bundles

[mikkamp]

Thanks for catching this extra scenario. It's working as expected. It took me a bit to understand how the field switches input type and why we are copying attributes from a different class. But I think that's more down to the decisions that were made in how to structure the field.