Skip to content
New issue

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

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

In the README.md examples, the #people block helper is not defined. #259

Closed
TaraRed opened this issue Jun 19, 2012 · 2 comments · May be fixed by hixio-mh/handlebars.js#2, XirdigH/handlebars.js#2, bigcommerce/handlebars.js#2 or Jircs1/handlebars.jsjircs#3
Closed

In the README.md examples, the #people block helper is not defined. #259

TaraRed opened this issue Jun 19, 2012 · 2 comments · May be fixed by hixio-mh/handlebars.js#2, XirdigH/handlebars.js#2, bigcommerce/handlebars.js#2 or Jircs1/handlebars.jsjircs#3
Labels
docs
Milestone
1.0

Comments

@TaraRed
Copy link

TaraRed commented Jun 19, 2012

In the Block Helpers section of the README.md there is no Handlebars.registerHelper call that defines what {{#people}}{{/people}} does. As a result, people are assuming it to be a shorthand of {{#each people}}{{/each}} which is however nowhere mentioned that such behavior happens.

Please do either of:

  1. Introduce a Handlebars.registerHelper call in the README.md to define what the people block helper does.
  2. Clarify everywhere that {{#people}}{{/people}} is a shorthand of {{#each people}}{{/each}}, however I don't think this behavior is present and shouldn't be present for ambiguity reasons and name clashes.
@wagenet
Copy link
Collaborator

wagenet commented Jul 31, 2012

My understanding is that we handle behavior 2 for compatibility with mustache. This should probably be better documented.

@kpdecker
Copy link
Collaborator

This behavior is outlined in the "Non-Empty Lists" docs in the mustache spec. I'm not sure the best way to document this in a clearer way short of rewriting most mustache spec in our own documentation.

kpdecker added a commit that referenced this issue May 19, 2013
@kpdecker
Update block helper docs in README
7fb9750 
Fixes #231, #259
alejandroiglesias added a commit to alejandroiglesias/handlebars.js that referenced this issue May 27, 2013
@alejandroiglesias
Update to 1.0.11
c2c3808 
commit 5f34991
Author: kpdecker <[email protected]>
Date:   Mon May 27 12:29:57 2013 -0500

    Update in the wild section

    Fixes handlebars-lang#246

commit c1020a0
Author: kpdecker <[email protected]>
Date:   Mon May 27 11:52:32 2013 -0500

    Add unicode support for ID tokens

    Fixes handlebars-lang#433
    Fixes handlebars-lang#469

commit bff71d7
Author: kpdecker <[email protected]>
Date:   Sat May 25 12:32:16 2013 -0500

    Update release notes

commit 4d95d58
Author: kpdecker <[email protected]>
Date:   Sat May 25 12:27:31 2013 -0500

    Rebuild

commit 47e0aa7
Merge: 6bf9c6f f5fe6d0
Author: Kevin Decker <[email protected]>
Date:   Sat May 25 10:07:46 2013 -0700

    Merge pull request handlebars-lang#526 from jamesgorrie/master

    Fixing regex on partial names

commit 6bf9c6f
Merge: 533be69 10669e0
Author: kpdecker <[email protected]>
Date:   Wed May 22 08:40:28 2013 -0400

    Merge branch 'master' of github.com:wycats/handlebars.js

commit 533be69
Author: kpdecker <[email protected]>
Date:   Tue May 21 13:12:11 2013 -0500

    Use better variable name

commit a97820f
Author: kpdecker <[email protected]>
Date:   Tue May 21 13:03:41 2013 -0500

    Update testing notes

    Fixes handlebars-lang#467

commit f6b6361
Author: kpdecker <[email protected]>
Date:   Tue May 21 13:03:01 2013 -0500

    Fix typo

commit 10669e0
Merge: 5ba17c9 69bcdf3
Author: Kevin Decker <[email protected]>
Date:   Tue May 21 08:46:27 2013 -0700

    Merge pull request handlebars-lang#533 from blessenm/amd-precompilation-fix

    Fix for the issue handlebars-lang#517

commit 69bcdf3
Author: Blessan Mathew <[email protected]>
Date:   Tue May 21 20:50:46 2013 +0530

    Fix for issue handlebars-lang#517.

    Now when compiiled with amd option and a single input argument, we are
    checking to see whether the input is a file or folder.

commit 5ba17c9
Merge: 7fb9750 94cc419
Author: kpdecker <[email protected]>
Date:   Sun May 19 11:47:01 2013 -0500

    Merge branch 'master' of github.com:wycats/handlebars.js

commit 7fb9750
Author: kpdecker <[email protected]>
Date:   Sun May 19 11:45:20 2013 -0500

    Update block helper docs in README

    Fixes handlebars-lang#231, handlebars-lang#259

commit c95d9d6
Author: kpdecker <[email protected]>
Date:   Sun May 19 11:30:26 2013 -0500

    Use context rather than arg in README examples

commit 3aef7f6
Author: kpdecker <[email protected]>
Date:   Sun May 19 11:30:04 2013 -0500

    Misc readme updates

commit f5fe6d0
Author: James Gorrie <[email protected]>
Date:   Fri May 17 16:58:34 2013 +0100

    Allowed for points to be in partial path name

commit 94cc419
Merge: 4cf0410 d649b35
Author: Kevin Decker <[email protected]>
Date:   Tue May 14 21:10:14 2013 -0700

    Merge pull request handlebars-lang#515 from jjclark1982/master

    Allow requiring ".handlebars" files within Node.js scripts

commit d649b35
Author: Jesse Clark <[email protected]>
Date:   Wed May 15 10:50:01 2013 +1000

    Only run "require" tests when require() is available

commit 229b82b
Author: Jesse Clark <[email protected]>
Date:   Wed May 15 10:38:45 2013 +1000

    Add unit tests for require() handler

commit 531be77
Author: Jesse Clark <[email protected]>
Date:   Tue May 14 22:26:20 2013 +1000

    Publish a Node.js require() handler for handlebars files

commit 4cf0410
Author: kpdecker <[email protected]>
Date:   Mon May 13 23:09:21 2013 -0500

    1.0.11

commit 2ec5a97
Author: kpdecker <[email protected]>
Date:   Mon May 13 23:08:43 2013 -0500

    Update release notes

commit ddc4d31
Author: kpdecker <[email protected]>
Date:   Mon May 13 23:07:29 2013 -0500

    Rev to rc4

commit 85dd394
Author: kpdecker <[email protected]>
Date:   Fri May 10 01:35:25 2013 -0500

    Add regex escapes

commit 8b65fa7
Author: kpdecker <[email protected]>
Date:   Fri May 10 01:24:11 2013 -0500

    Update release notes

commit fa6cec2
Author: kpdecker <[email protected]>
Date:   Fri May 10 01:02:13 2013 -0500

    Update client packager versions

commit 436973f
Author: kpdecker <[email protected]>
Date:   Fri May 10 01:00:39 2013 -0500

    Version update script

commit a279678
Merge: d355d1f 8a5705a
Author: Kevin Decker <[email protected]>
Date:   Thu May 9 22:37:23 2013 -0700

    Merge pull request handlebars-lang#505 from MikeMayer/master

    Include a .nuspec package file to include handlebars.js on NuGet

commit d355d1f
Author: Kevin Decker <[email protected]>
Date:   Fri May 10 00:35:05 2013 -0500

    Fix links in release notes

commit 71c5e36
Merge: 44ae572 fc52a65
Author: Kevin Decker <[email protected]>
Date:   Thu May 9 22:29:27 2013 -0700

    Merge pull request handlebars-lang#489 from DevinClark/bower-update

    Added a component.json file for Twitter's Bower package manager.

commit 44ae572
Author: kpdecker <[email protected]>
Date:   Fri May 10 00:14:55 2013 -0500

    Add release notes

    Fixes handlebars-lang#471

commit 7cd9cdb
Merge: ef32573 090ee7c
Author: Kevin Decker <[email protected]>
Date:   Thu May 9 21:03:03 2013 -0700

    Merge pull request handlebars-lang#513 from tommydudebreaux/master

    Using handlebars with browserify causes IE to throw - TypeError: Invalid calling object

commit 090ee7c
Author: Tommy Messbauer <[email protected]>
Date:   Thu May 9 17:17:34 2013 -0500

    added local pointer to handlerbars.utils to allow this to browserify properly for IE

commit 8a5705a
Author: MikeMayer <[email protected]>
Date:   Wed May 1 09:58:17 2013 -0700

    Explicitly only include the handlebars.js from the dist folder

commit 344bb90
Author: MikeMayer <[email protected]>
Date:   Wed May 1 09:52:55 2013 -0700

    Create handlebars.js.nuspec for inclusion in NuGet

    Added a .nuspec file for those people who use Visual Studio and NuGet and might want to use handlebars.js deployed from NuGet in their application.

commit fc52a65
Author: DevinClark <[email protected]>
Date:   Thu Apr 25 21:05:21 2013 -0500

    Changed component.json to bower.json per Bower 0.9.0 update.

commit 5c9aa9e
Author: DevinClark <[email protected]>
Date:   Mon Apr 8 12:28:13 2013 -0500

    Added a component.json file for Twitter's Bower package manager.

    See [https://github.com/twitter/bower#defining-a-package](Defining a
    package) for more information about how the file was generated.
This was referenced Dec 18, 2020
Open
Open
@snyk-bot snyk-bot mentioned this issue Jan 22, 2022
Open
@Jircs1 Jircs1 mentioned this issue Mar 19, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
docs
Projects
None yet
Milestone
1.0
3 participants
@wagenet @kpdecker @TaraRed