Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[ObjectMapper] Provide the target object to the map condition #1

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 3 commits into
base: allow-different-targets
Choose a base branch
from

Conversation

GromNaN
Copy link

@GromNaN GromNaN commented Apr 1, 2025

Proposed implementation for symfony#60028

@soyuka soyuka force-pushed the allow-different-targets branch 8 times, most recently from 53b56d6 to 1910509 Compare April 2, 2025 14:05
@fabpot fabpot force-pushed the allow-different-targets branch from fd55ff0 to a5698aa Compare May 6, 2025 06:50
soyuka pushed a commit that referenced this pull request Jun 20, 2025
…he secret you are trying to reveal could not be decrypted (jack-worman)

This PR was submitted for the 7.4 branch but it was merged into the 7.2 branch instead.

Discussion
----------

[FrameworkBundle] Fixes getting a type error when the secret you are trying to reveal could not be decrypted

| Q             | A
| ------------- | ---
| Branch?       | 7.2
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Issues        |
| License       | MIT

Fixes getting a type error when the secret you are trying to reveal could not be decrypted.
```txt
In SymfonyStyle.php line 322:

  Symfony\Component\Console\Style\SymfonyStyle::writeln(): Argument #1 ($messages) must be of type Traversable|array|string, null given, called in /Users/jworman/PhpstormProjects/gtx/vendor/symfony/framework-bundle/Command/Sec
  retsRevealCommand.php on line 67

```

Not sure if this is considered a bug fix or a new feature

Commits
-------

480e7d1 Fix-type-error-when-revealing-broken-secret
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants