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

Literal args comma fix #1577

Merged
merged 14 commits into from
May 22, 2015
Merged

Literal args comma fix #1577

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
be559b1
rename `effect.arg` to `effect.trigger` to avoid confusion with `effe…
May 18, 2015
db27411
support literal args in `_marshalArgs`
May 18, 2015
69c472f
require method expressions to use `,` as argument delimeter, trim whi…
May 18, 2015
3fd8fbb
basic test for computed expression containing literal args (must also…
May 18, 2015
89650a1
rename `standard/effects.html` to `standard/effectBuilder.html`
May 18, 2015
ab6ce1e
* do not treat literal arguments as effect triggers
May 19, 2015
6fd5ef8
`_executeStaticEffects` at configure-time
May 19, 2015
0fe9fe7
rudimentary test for a static effect (annotated computation with only…
May 19, 2015
42eaedd
* remove bogus support for arbitrary JSON in literal arguments, liter…
May 20, 2015
c001344
Accept no-arg computed fns as static. Fixes #1516.
kevinpschaaf May 22, 2015
cf5cba7
Update comment.
kevinpschaaf May 22, 2015
ffc361c
Add comment about escaping.
kevinpschaaf May 22, 2015
e6859b6
Move _executeStaticEffects to _applyConfig timing.
kevinpschaaf May 22, 2015
40a9578
Add corrected computedFromTrickyLiterals2 test.
kevinpschaaf May 22, 2015
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
_executeStaticEffects at configure-time
  • Loading branch information
@kevinpschaaf
Scott J Miles authored and kevinpschaaf committed May 20, 2015
commit 6fd5ef8cddce7a294cdecc738b627973f5c21efc
3 changes: 3 additions & 0 deletions src/standard/configure.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,10 @@
// configure: returns user supplied default property values
// combines with _config to create final property values
_configure: function() {
// TODO(sjmiles): does what?
this._configureAnnotationReferences();
// process static effects, e.g. computations that have only literal arguments
this._executeStaticEffects();
// get individual default values from property configs
var config = {};
// mixed-in behaviors
Expand Down