Skip to content

[FEATURE] Make it possible to assign variables to the view #884

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

Closed
wants to merge 2 commits into from

Conversation

georgringer
Copy link
Contributor

Using a slot is nice but currently there is no way to assign items to the view besides using xclass. E.g. https://github.com/thomasrawiel/powermailcaptcha/blob/develop/Classes/Controller/FormController.php as example

  • setSettings can't be used because the settings are assigned to the view way earlier
  • changing the slot is not an option because breaking

Using a slot is nice but currently there is no way to assign items to the view besides using xclass. E.g. https://github.com/thomasrawiel/powermailcaptcha/blob/develop/Classes/Controller/FormController.php as example

- `setSettings` can't be used because the settings are assigned to the view way earlier
- changing the slot is not an option because breaking
georgringer added a commit to georgringer/powermailcaptcha that referenced this pull request Jun 2, 2023
Instead of using XCLASS, a slot of powermail can be used

Be aware: this requires currently in2code-de/powermail#884

Resolves: thomasrawiel#8
@mschwemer mschwemer deleted the branch in2code-de:develop December 28, 2023 10:19
@mschwemer mschwemer closed this Dec 28, 2023
@mschwemer mschwemer reopened this Dec 28, 2023
@mschwemer
Copy link
Collaborator

Closing it, since there are no signals any more.

If you think, there is still the need for the event based code, I would be happy, if you provide a PR for that.

Best regards

@mschwemer mschwemer closed this Oct 2, 2024
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