Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Hotfix/view quick start #373

Merged
merged 2 commits into from Oct 13, 2012

Conversation

Projects
None yet
3 participants
Owner

weierophinney commented Oct 8, 2012

Fixes issues noted in the comments on http://framework.zend.com/manual/2.0/en/modules/zend.view.quick-start.html

  • was referencing ListenerAggregate instead of ListenerAggregateInterface
  • confusing mix of BarController:doSomethingAction and BazBatController::doSomethingCrazyAction

Also, in reading it, it was using DI configuration, and not view_manager configuration.

This PR fixes all of the above.

weierophinney added some commits Oct 8, 2012

@weierophinney weierophinney Reference ListenerAggregateInterface, not ListenerAggregate 6a58a71
@weierophinney weierophinney Unified examples
- Use BazBatController and doSomethingCrazyAction everywhere (instead of
  BarController and doSomethingAction)
- Show view_manager config instead of DI config
c477c6a

@robertbasic robertbasic commented on the diff Oct 8, 2012

docs/languages/en/modules/zend.view.quick-start.rst
- 'parameters' => array(
- 'displayExceptions' => true,
- 'displayNotFoundReason' => true,
- 'notFoundTemplate' => 'site/404',
- ),
- ),
+ // The TemplatePathStack takes an array of directories. Directories
+ // are then searched in LIFO order (it's a stack) for the requested
+ // view script. This is a nice solution for rapid application
+ // development, but potentially introduces performance expense in
+ // production due to the number of stat calls necessary.
+ //
+ // The following adds an entry pointing to the view directory
+ // of the current module. Make sure your keys differ between modules
+ // to ensure that they are not overwritten -- or simply omit the key!
+ 'template_path_stac' => array(
@robertbasic

robertbasic Oct 8, 2012

Contributor

template_path_stack :)

@Freeaqingme Freeaqingme merged commit c477c6a into zendframework:master Oct 13, 2012

Member

Freeaqingme commented Oct 13, 2012

@robertbasic fixed with b3d5dfe

Forgot to merge with --no-ff tough :(

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