Deprecate Drupal\locale\Locale and its one method

Yes, we can remove the Drupal\locale\Locale::config() method because in files such as core/modules/locale/locale.bulk.inc and core/modules/locale/tests/src/Functional/LocaleConfigTranslationImportTest.php, the function of the injected service is directly called instead of going through the config() function. Therefore, it's more efficient to directly utilize the injected service in these files.

Replaced Drupal\locale\Locale::config() with \Drupal::service('locale.config_manager'), Verified the Checks Using core/scripts/dev/commit-code-check.sh --cached (All Checks Passed). Please Review

Don't see the deprecation.

Deprecated the config() method.

I reverted the removing Locale::config and updated the documentation.
The unittest is failing because I added the @deprecated tag to the documentation but the %deprecation-version% and %removal-version% doesn't exist.
However, I don't know them.
Please review it.

> phpcs --standard=core/phpcs.xml.dist --parallel="$( (nproc || sysctl -n hw.logicalcpu || echo 4) 2>/dev/null)" -- '--report-full' '--report-summary' '--report-\Micheh\PhpCodeSniffer\Report\Gitlab=phpcs-quality-report.json'
FILE: /builds/issue/drupal-3422915/core/modules/locale/src/Locale.php
--------------------------------------------------------------------------------
FOUND 2 ERRORS AND 1 WARNING AFFECTING 3 LINES
--------------------------------------------------------------------------------
 19 | ERROR   | The text '@deprecated Use
    |         | \Drupal::service('locale.config_manager') instead.' does not
    |         | match the standard format: @deprecated in %deprecation-version%
    |         | and is removed from %removal-version%. %extra-info%.
 21 | WARNING | The @see url
    |         | '\Drupal\Core\TypedData\TypedDataManager::create()' does not
    |         | match the standard: http(s)://www.drupal.org/node/n or
    |         | http(s)://www.drupal.org/project/aaa/issues/n
 24 | ERROR   | The trigger_error message '__METHOD__ () is deprecated. Use
    |         | \Drupal::service(\'locale.config_manager\') instead.' does not
    |         | match the strict standard format: %thing% is deprecated in
    |         | %deprecation-version% and is removed from %removal-version%.
    |         | %extra-info%. See %cr-link%

Deprecation isn’t correct. Recommend looking through the repo for examples of deprecation format

I modified the documentation according to Drupal deprecation policy and added its test.
Please review it.

The test fails, but it doesn't seem to be related to the changes.
https://git.drupalcode.org/issue/drupal-3422915/-/jobs/1193931 -> test summary

Seems small enough to deprecate in 10.3 for removal in 11.

Will need a change record and the link points to that. Can be simple and would search the change record list for other deprecation examples for to write one (usually just a sentence or two)

And for the random failure would try and pull the latest 11.x code into this branch. That’s resolved it on other issues.

Good work with the test!

On this ticket whaat happens if you click add change notice?

If that doesn’t work will try and help on Monday

I made a mistake in rebase once, so I force pushed the pre-rebase, updated the 11.x branch of the repository to the latest, and rebased the branch to on top of that.

Thank you for confirming my account.
I could create a new change records.
https://www.drupal.org/node/3437110

Deprecation CR looks good.

Thanks for updating link in code too.

Thanks for working on this everyone!

Some unrelated change snuck into the MR - probably due to an IDE - we need to remove that. Also I don;t think this issue should be introducing a camel cased variable everywhere. I
think we should replace Locale::config() we \Drupal::service('locale.config_manager') and be done.

Also we need to add a deprecation to the class body so that we can safely remove the class in 11.x. I. think this is fine here because this in not a plugin and therefore won't trigger unexpected deprecations only good ones.

• Added documentation about the deprecation of the Locale class.
• Removed unnecessary periods.
• About using $localConfigManager as a temporary variable of Locale::config()
• Removed $localConfigManager if it was only used once.
• Renamed to $locale_config_manager if it was used more than once.

Please review it

Deprecation seems good now. Marking this so we don't miss the 10.3 boat.

If we're doing this for removal in 11.0.0, we should also have an 11.x-specific MR here that does the removal directly without the bc layer / test coverage etc. The existing MR should be re-targeted/rebased for 10.3.x

Haven't been asking for this previously, but we're adding new deprecations nearly as quickly as we're removing them still, so think we need to start using that workflow.

Looks ready again

I've fixed up the 10.3.x MR so that the test is correct - this does not return a \Drupal\Core\Config\ConfigManager object it returns a \Drupal\locale\LocaleConfigManager object.

Unfortunately the 11.x MR does not do the removal so that needs work.

Also we use \Drupal::service() over Drupal::service()

Rebased the 10.3.x MR as the failure appeared to be a random Umami bug.

Believe all feedback has been addressed

Committed dc9a4c5 and pushed to 10.3.x. Thanks!
Committed 4c93088 and pushed to 11.x. Thanks!