Make $-prefixed properties of $firebaseObject non-enumerable

Presently, the $$conf, $id and $priority properties of a $firebaseObject are normal, enumerable properties. This is inconvenient if I want to loop over the object’s properties with a for .. in loop or something like LoDash’s _.each() method.

Currently I work around this with code like

_.each(someFirebaseObject, function (value, key) {
    if (key.lastIndexOf('$', 0) === 0) {
        // Filter out AngularFire methods on object
        return;
    }
    else {
        // do something
    }
});

but this is ugly, and the ugliness is greater still in circumstances where what I’d really like to do is something like an _.map call.

It would be more convenient if these properties were created with Object.defineProperty to be non-enumerable.

Considerations:

• This will break IE 8 support (if the library isn’t broken in IE 8 already)
• … but you’re officially not supporting Angular 1.2.x any more, which was the last version of Angular to support IE 8, so you officially don’t support IE 8 anyway
• Concerns about IE 8 aside, this is technically a breaking change, albeit a minor one that probably won’t affect many people
• Angular are likewise considering making their $foo properties non-enumerable (see what is currently the final post at https://github.com/angular/angular.js/issues/6266) but may or may not follow through on this

I advocate making the change, but you may see things differently.

[1dolinski]

@jamestalmage Thank you, this helped me as well.

[jwngr]

Does @jamestalmage’s suggestion above of using $firebaseUtils not solve your problem? If so, please provide some more details.