Skip to content

Commit c7301f0

Browse files
authored
Merge pull request #67 from cloudinary/isNamedNodeMap-fix-for-firefox
Is named node map fix for firefox
2 parents a292306 + e81deee commit c7301f0

File tree

2 files changed

+13
-1
lines changed

2 files changed

+13
-1
lines changed

README.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,9 @@
11
Cloudinary
22
==========
3+
The official [Cloudinary](https://cloudinary.com) SDK for AngularJS (AKA Angular 1).
4+
5+
Cloudinary's Angular 2 SDK can be found [here](https://github.com/cloudinary/cloudinary_angular/tree/angular_next)
6+
and on [npm](https://www.npmjs.com/package/@cloudinary/angular)
37

48
Cloudinary is a cloud service that offers a solution to a web application's entire image management pipeline.
59

js/angular.cloudinary.js

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,15 @@
2828
*/
2929
var toCloudinaryAttributes = function( source, filter) {
3030
var attributes = {};
31-
var isNamedNodeMap = source && (source.constructor.name === "NamedNodeMap" || source instanceof NamedNodeMap);
31+
var isNamedNodeMap;
32+
if (window.NamedNodeMap) {
33+
isNamedNodeMap = source && (source.constructor.name === "NamedNodeMap" || source instanceof NamedNodeMap);
34+
} else if (window.MozNamedAttrMap) {
35+
// https://developer.mozilla.org/en-US/docs/Web/API/NamedNodeMap
36+
// https://www.fxsitecompat.com/en-CA/docs/2013/namednodemap-has-been-renamed-to-moznamedattrmap/
37+
// In Firefox versions 22 - 33 the interface "NamedNodeMap" was called "MozNamedAttrMap"
38+
var isNamedNodeMap = source && (source.constructor.name === "MozNamedAttrMap" || source instanceof MozNamedAttrMap);
39+
}
3240
angular.forEach(source, function(value, name){
3341
if( isNamedNodeMap) {
3442
name = value.name;

0 commit comments

Comments
 (0)