ReactiveState: add namespaces

[steph643]

I would prefer to use global helpers like this:

<template name='VideoAuthor'>
  Author name is {{AppState.videoAuthor.name}}.
</template>

Otherwise I need to prefix my root path in order to avoid mix up with template helpers...

[luisherranz]

Interesting. We can add it to the initialisation options:

AppState = new MeteorFlux.AppState({ prefix: 'state' });

Would that be ok?

[steph643]

That looks great.

[steph643]

Global helpers make sense in the case of a singleton object. So, to me, global helpers should be part of AppState, not ReactiveState.

[luisherranz]

Totally true, they should be in a different package

[steph643]

Just spent 1 hour because of this. Suppose you use AppState.set('doc', object) in your code, then the following code has very unexpected results:

<template name=myTemplate1>
  {{>myTemplate2 doc=whatever}}
</template>

<!-- Args: doc -->
<template name=myTemplate2>
  <div>doc.name</div> <!-- Here doc is not the template parameter! -->
</template>

[luisherranz]

That's not because of AppState, but because of how Meteor global Template helpers work.

If you do:

Template.registerHelper('dod', function() {
  return object;
});

it will happen the same.

If you make a PR with a namespace option for ReactiveState I will accept it.

For example:

AppState = new ReactiveState({
  namespace: 'state'
});

<template name=myTemplate2>
  <div>doc.name</div> <!-- Here doc is the template parameter -->
  <div>state.doc</div> <!-- Here doc is the AppState parameter -->
</template>