I've started to cleaning up the code in some places.
Noticed: https://git.drupalcode.org/issue/ai_search_block-3504975/-/pipelines/417335
And decided to give a hand (as a part of using this on live env.)
Will make an MR tomorrow so we can work on this and make sure pipeline is clean.
Note: one thing I'll need your hand in - is this piece as I don't know what exact values you meant:
It's under: modules/ai_search_block_log/src/Form/AiSearchBlockLogForm.php
...
switch ($result) {
case SAVED_NEW:
$this->messenger()->addStatus($this->t('New ai search block log %label has been created.', $message_args));
$this->logger('ai_search_block_log')->notice('New ai search block log %label has been created.', $logger_args);
break;
case SAVED_UPDATED:
$this->messenger()->addStatus($this->t('The ai search block log %label has been updated.', $message_args));
$this->logger('ai_search_block_log')->notice('The ai search block log %label has been updated.', $logger_args);
break;
default:
throw new \LogicException('Could not save the entity.');
}
These 2 constants need to be defined somewhere. I'll send an MR in the morning and you can add the edits there as well ^_^
Active
1.0
Code
These two are boilerplate I dont even remember from where. They can be removed.
Hey hey - made an MR - feel free to edit those bits out (or adjust).
Also, please test :)
Issue #3504998 by nikro, wouters_f: Code phpstan/phpcs
Automatically closed - issue fixed for 2 weeks with no activity.