-
Notifications
You must be signed in to change notification settings - Fork 44
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
PIPRES-348: Move subscription options to subscription tab #832
PIPRES-348: Move subscription options to subscription tab #832
Conversation
mandan2
commented
Oct 17, 2023
•
edited
Loading
edited
|
||
/** @var Mollie */ | ||
protected $module; | ||
|
||
public function __construct() | ||
{ | ||
parent::__construct(); | ||
$this->leagueContainer = new LeagueServiceContainerProvider(); | ||
|
||
$this->serviceProvider = new LeagueServiceContainerProvider(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
dont really like this approach you already have $this->module, you can getService as intended now, by using new you are skipping extendable options of this provider.
* | ||
* @throws \Throwable | ||
*/ | ||
private function processForm(FormHandlerInterface $formHandler, Request $request) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems weird, you are calling submit, but check if it's submitted later on there should be a better solution inside Core of PS.
*/ | ||
public function validateConfiguration(array $configuration): bool | ||
{ | ||
return isset( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Useless function tbh.
* PIPRES-348: WIP move subscription options to subscription tab * added carrier repository to provide with carrier choices, moved template to another file * phpstan * adjusted PS 1.7.6 support * removed validations from formProcess * simplified formProcess