Skip to content

feat: add ValidationInterface::getValidated() - #7429

Merged
kenjis merged 2 commits into
codeigniter4:4.4from
kenjis:add-ValidationInterface-getValidated
Apr 15, 2023
Merged

feat: add ValidationInterface::getValidated()#7429
kenjis merged 2 commits into
codeigniter4:4.4from
kenjis:add-ValidationInterface-getValidated

Conversation

@kenjis

@kenjis kenjis commented Apr 13, 2023 •
edited
Loading

Copy link
Copy Markdown
Member

Description
Follow-up #7420

PHPStan reports when using $this->validator->getValidated() in Controller:

Call to an undefined method CodeIgniter\Validation\ValidationInterface::getValidated().

Checklist:

  • Securely signed commits
  • Component(s) with PHPDoc blocks, only if necessary or adds value
  • Unit testing, with >80% coverage
  • User guide updated
  • Conforms to style guide

@kenjis kenjis added enhancement PRs that improve existing functionalities breaking change Pull requests that may break existing functionalities 4.4 labels Apr 13, 2023
@kenjis
kenjis merged commit 298a392 into codeigniter4:4.4 Apr 15, 2023
@kenjis
kenjis deleted the add-ValidationInterface-getValidated branch April 15, 2023 07:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@michalsn michalsn michalsn approved these changes
@paulbalandan paulbalandan paulbalandan approved these changes
+1 more reviewer
@ddevsr ddevsr ddevsr approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

breaking change Pull requests that may break existing functionalities enhancement PRs that improve existing functionalities

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@kenjis @michalsn @paulbalandan @ddevsr