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

Following this
example, you could load the **blog_view.php** file from **example/blog/Views** by prepending the namespace to the view name:
example, you could load the **blog_view.php** file from **example/blog/Views** by prepending the namespace to the view name
(specifying the end directory ``Views`` as an ``Example\Blog\Views\`` not required):
Copy link
Member

Choose a reason for hiding this comment

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

I don't understand this part.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The Views directory is hard-coded when searching for a template. You can skip this part Blog\Views\ == Blog\

Copy link
Member

Choose a reason for hiding this comment

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

Thanks for explaining

Suggested change
(specifying the end directory ``Views`` as an ``Example\Blog\Views\`` not required):
(specifying the end directory ``Views`` as in ``Example\Blog\Views\`` is not required):

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.

3 participants