You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Apr 12, 2024. It is now read-only.
During the work of 1.0.0.rc2, the directive handling was probably rewritten. Support seems to have been added for merging class attributes when replacing an element: f49eaf8
But if the original element does not have a class attribute, attr[name] will be undefined and the $interpolate function will crash on length = text.length at line 4644 in angular-1.0.0rc10.js (I don't have the src-files in front of me)
The simplest solution is probably to change the if-clause in addAttrInterpolateDirective function from if (name === 'class') {
to if (name === 'class' && attr[name]) {
I'll probably request a pull for that in a few hours if I find the time :-)
The text was updated successfully, but these errors were encountered:
maxmart
added a commit
to maxmart/angular.js
that referenced
this issue
Jun 6, 2012
During the work of 1.0.0.rc2, the directive handling was probably rewritten. Support seems to have been added for merging class attributes when replacing an element: f49eaf8
But if the original element does not have a class attribute,
attr[name]
will be undefined and the$interpolate
function will crash onlength = text.length
at line 4644 in angular-1.0.0rc10.js (I don't have the src-files in front of me)The simplest solution is probably to change the if-clause in
addAttrInterpolateDirective
function fromif (name === 'class') {
to
if (name === 'class' && attr[name]) {
I'll probably request a pull for that in a few hours if I find the time :-)
The text was updated successfully, but these errors were encountered: