Skip to content

Conversation

neznaika0
Copy link
Contributor

@neznaika0 neznaika0 commented Oct 12, 2025

Description
Improving the description View.

  • We don't have a test to check using Cell getDataProperty(), getViewProperty() or $this->data, $this->view. I didn't quite understand the prohibition for data.
  • The $view property is not specified, as this is the fullpath to the file, and not, as for the View, a relative name (with a namespace Blog\Cells\alerts).
  • I came across a misunderstanding of how include() works, so I described my case.
  • The Views folder in namespace is optional - the locator always searches for this folder.
Снимок экрана от 2025-10-12 15-49-49

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 added the documentation Pull requests for documentation only label Oct 14, 2025
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.

2 participants