The use of `.attr('data-*')` instead of `.data('*')` seems to be a pretty common practice throughout ownCloud, and is perhaps something that should be refactored globally, akin to the commit that changed everything to `===`.  Just a thought.