Skip to content
This repository was archived by the owner on Jan 29, 2020. It is now read-only.

Fix 'Call to a member function hasChildren() on boolean' #27

Merged
merged 1 commit into from
Sep 14, 2015

Conversation

skl
Copy link
Contributor

@skl skl commented Sep 8, 2015

  • This seems to occur when the DefaultRenderingStrategy tries to render
    a result that is not an instance of Zend\View\Model\ModelInterface, in
    my case the result was a boolean true.
  • I'm checking for ModelInterface rather than ConsoleModel as the render
    method seems to work with any ModelInterface, not just ConsoleModel

Supersedes #26 due to rebase incorporating feedback.

- This seems to occur when the DefaultRenderingStrategy tries to render
  a result that is not an instance of Zend\View\Model\ModelInterface, in
  my case the result was a boolean true.
- I'm checking for ModelInterface rather than ConsoleModel as the render
  method seems to work with any ModelInterface, not just ConsoleModel
@skl
Copy link
Contributor Author

skl commented Sep 8, 2015

Sorry looks like I should've forced-push to update #26 rather than delete and push which creates a new PR... next time.

@weierophinney
Copy link
Member

@skl In the future, you can push as many times as you want to the branch from which you created a PR; it will update the PR with new commits and the cumulative changes. When you force push, it simply replaces the commits and changes.

@weierophinney weierophinney added this to the 2.5.2 milestone Sep 14, 2015
@weierophinney weierophinney self-assigned this Sep 14, 2015
@weierophinney weierophinney merged commit a96d349 into zendframework:master Sep 14, 2015
weierophinney added a commit that referenced this pull request Sep 14, 2015
Fix 'Call to a member function hasChildren() on boolean'
weierophinney added a commit that referenced this pull request Sep 14, 2015
weierophinney added a commit that referenced this pull request Sep 14, 2015
weierophinney added a commit that referenced this pull request Sep 14, 2015
@weierophinney
Copy link
Member

Tagged in both release-2.5.2 and release-2.4.8

@skl
Copy link
Contributor Author

skl commented Sep 14, 2015

Thanks 😄

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants