…` calls (derrabus)

This PR was merged into the 7.2 branch.

Discussion
----------

[PsrHttpMessageBridge] Remove `uniqid()` from `tempnam()` calls

| Q             | A
| ------------- | ---
| Branch?       | 7.2
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Issues        | Part of #57588
| License       | MIT

PHP's `tempnam()` function should already make sure we get a collision-free random temporary file. We should not need additional randomness from `uniqid()`.

Commits
-------

1a05a66 Remove uniqid() from tempnam() calls

This PR was merged into the 7.2 branch.

Discussion
----------

Remove useless `uniqid` in tempnam calls in tests

| Q             | A
| ------------- | ---
| Branch?       | 7.2
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Issues        | Part of #57588
| License       | MIT

For creating a temporary directory, I use `tempnam` then remove the file and create the directory with the same name.

Commits
-------

d5edc93 Remove useless uniqid in tempnam calls

…put in case of error (GromNaN)

This PR was merged into the 7.2 branch.

Discussion
----------

[TwigBridge] Remove random file name for `lint:twig` output in case of error

| Q             | A
| ------------- | ---
| Branch?       | 7.2
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Issues        | Part of #57588
| License       | MIT

In the `lint:twig`, `uniqid` is used to generate a file name when linting a template from STDIN.
- I don't find any reason to use a unique file name. The file name is never stored when the template is tokenized and compiled. The cache is not used.
- This meaningless file name is displayed in the output in case of error, which is better replaced by the text "Standard Input".

Before:
```
$ bin/console lint:twig - < vendor/symfony/twig-bridge/Resources/views/Email/zurb_2/notification/body.html.twig
  ERROR  in sf_6681e2dd64a2e2.02015003 (line -1)
  >> The "inky_to_html" filter is part of the InkyExtension, which is not installed/enabled; try running "composer require twig/inky-extra".
 [WARNING] 0 Twig files have valid syntax and 1 contain errors.
```

After:
```
$ bin/console lint:twig - < vendor/symfony/twig-bridge/Resources/views/Email/zurb_2/notification/body.html.twig
  ERROR  in Standard Input (line -1)
  >> The "inky_to_html" filter is part of the InkyExtension, which is not installed/enabled; try running "composer require twig/inky-extra".
 [WARNING] 0 Twig files have valid syntax and 1 contain errors.
```

Note: the `line -1` is due to the lack of context provided to  `Twig\Extra\TwigExtraBundle\MissingExtensionSuggestor::suggestFilter()`

For reference, this was introduced by #10843

Commits
-------

a97acbd Remove random file name for lint:twig output in case of error

…ult_domain` expression result (GromNaN)

This PR was merged into the 7.2 branch.

Discussion
----------

[TwigBridge]  Use constant var name to cache `trans_default_domain` expression result

| Q             | A
| ------------- | ---
| Branch?       | 7.2
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Issues        | Part of #57588
| License       | MIT

When `trans_default_domain` is used with an expression, the result of the expression is cached into a variable and this variable is stored in the `Scope` to be used for each following usage of the `|trans` filter.

This var name doesn't need to be random:
- there is only 1 value in the scope, more than 1 variable at a time is never necessary.
- if `trans_default_domain` is called a second time, the same variable can be reassigned.

The only benefit of using a random var name would be to prevent usage in the template. The name `__internal_trans_default_domain` self-explains that it is not meant to be used.

Commits
-------

ab116bb Use constant var name to cache trans_default_domain expression result

…ests (xabbuh)

This PR was merged into the 7.2 branch.

Discussion
----------

[Config][Lock][SecurityBundle]  do not use uniqid() in tests

| Q             | A
| ------------- | ---
| Branch?       | 7.2
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Issues        | part of #57588
| License       | MIT

Commits
-------

1b3d6bf do not use uniqid() in tests

… uniqid() in tests (xabbuh)

This PR was merged into the 7.2 branch.

Discussion
----------

[Filesystem][Messenger][PsrHttpMessageBridge] do not use uniqid() in tests

| Q             | A
| ------------- | ---
| Branch?       | 7.2
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Issues        | part of #57588
| License       | MIT

updates the remaining tests that were not covered by #57665

Commits
-------

ee7ffbd do not use uniqid() in tests

…bbuh)

This PR was merged into the 7.2 branch.

Discussion
----------

do not use `uniqid()` for generating dev tool tokens

| Q             | A
| ------------- | ---
| Branch?       | 7.2
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Issues        | part of #57588
| License       | MIT

Commits
-------

5ad7ab9 do not use uniqid() for generating dev tool tokens

…temporary file (derrabus)

This PR was merged into the 7.2 branch.

Discussion
----------

[Cache] Use tempnam() instead of uniquid() to produce a temporary file

| Q             | A
| ------------- | ---
| Branch?       | 7.2
| Bug fix?      | no
| New feature?  |no
| Deprecations? | no
| Issues        | Part of #57588
| License       | MIT

Commits
-------

3465a2b [Cache] Use tempnam() instead of uniquid()

…porary files (xabbuh)

This PR was merged into the 7.2 branch.

Discussion
----------

[DomCrawler][Filesystem] stop using uniqid() to create temporary files

| Q             | A
| ------------- | ---
| Branch?       | 7.2
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Issues        | part of #57588
| License       | MIT

Commits
-------

ffb21c1 stop using uniqid() to create temporary files