Skip to content

docs: Note of hidden properties - #9754

Merged
michalsn merged 2 commits into
codeigniter4:developfrom
neznaika0:docs/entity-desc
Oct 15, 2025
Merged

docs: Note of hidden properties#9754
michalsn merged 2 commits into
codeigniter4:developfrom
neznaika0:docs/entity-desc

Conversation

@neznaika0

@neznaika0 neznaika0 commented Oct 11, 2025

Copy link
Copy Markdown
Contributor

Description
See #9748

  1. Why doesn't the documentation describe the case of underlining properties? They are skipped when calling Entity::toArray() but allowed in Entity::toRawArray()
  2. The behavior of superfluous properties is not described. How they will be shown for the entity and when working with the DB.
Снимок экрана от 2025-10-11 11-35-15

Checklist:

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

michalsn
michalsn requested changes Oct 13, 2025
View reviewed changes
Comment thread user_guide_src/source/models/entities/028.php
Comment thread user_guide_src/source/models/entities.rst Outdated
Comment thread user_guide_src/source/models/entities.rst Outdated
michalsn
michalsn reviewed Oct 13, 2025
View reviewed changes
Comment thread user_guide_src/source/models/entities.rst Outdated
Comment thread user_guide_src/source/models/entities.rst Outdated
Comment thread user_guide_src/source/models/entities/028.php
@neznaika0

neznaika0 commented Oct 13, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

@michalsn, updated. Can we add something related to the database? How can this be useful or bad?
Снимок экрана от 2025-10-13 17-13-31

michalsn
michalsn approved these changes Oct 14, 2025
View reviewed changes

@michalsn michalsn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks!

@michalsn michalsn added the documentation Pull requests for documentation only label Oct 14, 2025
@michalsn
michalsn merged commit 8e8bfbc into codeigniter4:develop Oct 15, 2025
48 checks passed
@neznaika0
neznaika0 deleted the docs/entity-desc branch October 15, 2025 06:36
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

Assignees

No one assigned

Labels

documentation Pull requests for documentation only

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@neznaika0 @michalsn @paulbalandan