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

updated navigation view helpers to \Zend\Navigation namespace#2200

Closed
zucchi wants to merge 3 commits into
zendframework:masterfrom
zucchi:zucchi/nav-refactor
Closed

updated navigation view helpers to \Zend\Navigation namespace#2200
zucchi wants to merge 3 commits into
zendframework:masterfrom
zucchi:zucchi/nav-refactor

Conversation

@zucchi

@zucchi zucchi commented Aug 19, 2012

Copy link
Copy Markdown
Contributor

I have now moved the Navigation view helpers into the \Zend\Navigation namespaces and folders.

This now logically groups the helpers in the same way that we have done with the form view helpers.

I have also modified the helpers so that it is no longer necessary to proxy through the navigation helper and you may directly call

$this->menu($container);
$this->breadcrumbs($container);
$this->links($container);
$this->sitemap($container);

@travisbot

Copy link
Copy Markdown

This pull request fails (merged 80d4049 into 5dd58b8).

@travisbot

Copy link
Copy Markdown

This pull request fails (merged e96fc3f into 5dd58b8).

@weierophinney

Copy link
Copy Markdown
Member

Same comment as #2199. We need to focus on BC during the RC phase, and only
make breaking changes when there are actual issues that cannot be addressed
otherwise.

On Sunday, August 19, 2012, The Travis Bot wrote:

This pull request failshttp://travis-ci.org/zendframework/zf2/builds/2169499(merged
e96fc3f e96fc3f8 into
5dd58b8 5dd58b80).


Reply to this email directly or view it on GitHubhttps://github.com//pull/2200#issuecomment-7854294.

Matthew Weier O'Phinney
matthew@weierophinney.net
http://mwop.net/

@travisbot

Copy link
Copy Markdown

This pull request fails (merged 8d35da4 into 5dd58b8).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants