-
Notifications
You must be signed in to change notification settings - Fork 32
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.
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
Bug with ctx.js() #160
Labels
Comments
|
mishanga
added a commit
that referenced
this issue
Aug 21, 2015
Merged
mishanga
added a commit
that referenced
this issue
Aug 21, 2015
mishanga
added a commit
that referenced
this issue
Aug 21, 2015
@mishanga Thanks! |
@mishanga А в |
mishanga
added a commit
that referenced
this issue
Aug 24, 2015
@arikon да, выпустил 3.3.1 |
@mishanga Спасибо! |
@arikon блин, не так сделал. Выпущу пакет bh-3x с этим фиксом. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
After the second call to
ctx.js(true)
the value ofjs
is true. Should befalse
.js()
implementation in BH: https://github.com/bem/bh/blob/master/lib/bh.js#L437-L445It seems that after the first call
json.js
staysundefined
.On the second call this happens:
this.extend(undefined, true)
And then returns this: https://github.com/bem/bh/blob/master/lib/bh.js#L92
The text was updated successfully, but these errors were encountered: