Skip to content

docs: add note to $allowedFields#6021

Merged
kenjis merged 2 commits intocodeigniter4:developfrom
kenjis:fix-docs-model.rst
Jun 1, 2022
Merged

docs: add note to $allowedFields#6021
kenjis merged 2 commits intocodeigniter4:developfrom
kenjis:fix-docs-model.rst

Conversation

@kenjis
Copy link
Copy Markdown
Member

@kenjis kenjis commented May 24, 2022

Description
Fixes #5904

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

Comment thread user_guide_src/source/models/model.rst Outdated
Co-authored-by: Abdul Malik Ikhsan <samsonasik@gmail.com>
@kenjis kenjis merged commit 167b26c into codeigniter4:develop Jun 1, 2022
@kenjis kenjis deleted the fix-docs-model.rst branch June 1, 2022 01:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Pull requests for documentation only

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: DataException "There is no data to update." doesn't fire if $primaryKey is present in $allowedFields

3 participants