From a479db1c964ea3442fa6017f37c746903f5c9e1d Mon Sep 17 00:00:00 2001
From: cmrd Senya <senya@riseup.net>
Date: Fri, 9 Dec 2016 21:08:38 +0200
Subject: [PATCH 1/3] a missing spy

---
 spec/javascripts/app/app_spec.js | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/spec/javascripts/app/app_spec.js b/spec/javascripts/app/app_spec.js
index a1c987596..8a045ed78 100644
--- a/spec/javascripts/app/app_spec.js
+++ b/spec/javascripts/app/app_spec.js
@@ -4,6 +4,7 @@ describe("app", function() {
       spyOn(app.Router.prototype, "initialize");
       spyOn(app, "setupDummyPreloads");
       spyOn(app, "setupUser");
+      spyOn(app, "setupAspects");
       spyOn(app, "setupHeader");
       spyOn(app, "setupBackboneLinks");
       spyOn(app, "setupGlobalViews");
@@ -16,6 +17,7 @@ describe("app", function() {
       expect(app.Router.prototype.initialize).toHaveBeenCalled();
       expect(app.setupDummyPreloads).toHaveBeenCalled();
       expect(app.setupUser).toHaveBeenCalled();
+      expect(app.setupAspects).toHaveBeenCalled();
       expect(app.setupHeader).toHaveBeenCalled();
       expect(app.setupBackboneLinks).toHaveBeenCalled();
       expect(app.setupGlobalViews).toHaveBeenCalled();

From f36a4cd1f551b19e07bd1cae324a663dd3d62c9e Mon Sep 17 00:00:00 2001
From: cmrd Senya <senya@riseup.net>
Date: Sat, 10 Dec 2016 00:19:47 +0200
Subject: [PATCH 2/3] don't use the deprecated ".selector" method

---
 spec/javascripts/app/app_spec.js              |  9 ++++--
 .../javascripts/app/views/search_view_spec.js |  2 +-
 spec/javascripts/app/views_spec.js            | 29 ++++++++++++-------
 .../mobile/mobile_application_spec.js         |  2 +-
 4 files changed, 27 insertions(+), 15 deletions(-)

diff --git a/spec/javascripts/app/app_spec.js b/spec/javascripts/app/app_spec.js
index 8a045ed78..1ade3b556 100644
--- a/spec/javascripts/app/app_spec.js
+++ b/spec/javascripts/app/app_spec.js
@@ -41,18 +41,23 @@ describe("app", function() {
   });
 
   describe("setupForms", function() {
+    beforeEach(function() {
+      spec.content().append("<textarea/> <input/>");
+    });
+
     it("calls jQuery.placeholder() for inputs", function() {
       spyOn($.fn, "placeholder");
       app.setupForms();
       expect($.fn.placeholder).toHaveBeenCalled();
-      expect($.fn.placeholder.calls.mostRecent().object.selector).toBe("input, textarea");
+      expect($.fn.placeholder.calls.mostRecent().object.is($("input"))).toBe(true);
+      expect($.fn.placeholder.calls.mostRecent().object.is($("textarea"))).toBe(true);
     });
 
     it("initializes autosize for textareas", function(){
       spyOn(window, "autosize");
       app.setupForms();
       expect(window.autosize).toHaveBeenCalled();
-      expect(window.autosize.calls.mostRecent().args[0].selector).toBe("textarea");
+      expect(window.autosize.calls.mostRecent().args[0].is($("textarea"))).toBe(true);
     });
   });
 
diff --git a/spec/javascripts/app/views/search_view_spec.js b/spec/javascripts/app/views/search_view_spec.js
index ffb652304..fe93aa651 100644
--- a/spec/javascripts/app/views/search_view_spec.js
+++ b/spec/javascripts/app/views/search_view_spec.js
@@ -10,7 +10,7 @@ describe("app.views.Search", function() {
       spyOn(app.views.SearchBase.prototype, "initialize");
       this.view = new app.views.Search({el: "#search_people_form"});
       var call = app.views.SearchBase.prototype.initialize.calls.mostRecent();
-      expect(call.args[0].typeaheadInput.selector).toBe("#search_people_form #q");
+      expect(call.args[0].typeaheadInput.is($("#search_people_form #q"))).toBe(true);
       expect(call.args[0].remoteRoute).toEqual({url: "/search"});
     });
 
diff --git a/spec/javascripts/app/views_spec.js b/spec/javascripts/app/views_spec.js
index 6992f0372..84c912d15 100644
--- a/spec/javascripts/app/views_spec.js
+++ b/spec/javascripts/app/views_spec.js
@@ -119,42 +119,49 @@ describe("app.views.Base", function(){
 
     context("calling out to third party plugins", function() {
       it("replaces .time with relative time ago in words", function() {
+        this.view.templateName = false;
         spyOn($.fn, "timeago");
+        this.view.$el.append("<time/>");
         this.view.render();
         expect($.fn.timeago).toHaveBeenCalled();
-        expect($.fn.timeago.calls.mostRecent().object.selector).toBe("time");
+        expect($.fn.timeago.calls.mostRecent().object.first().is("time")).toBe(true);
       });
 
       it("initializes tooltips declared with the view's tooltipSelector property", function(){
+        this.view.templateName = false;
         this.view.tooltipSelector = ".christopher_columbus, .barrack_obama, .block_user";
+        this.view.$el.append("<div class='christopher_columbus barrack_obama block_user'/>");
 
         spyOn($.fn, "tooltip");
         this.view.render();
-        expect($.fn.tooltip.calls.mostRecent().object.selector).toBe(".christopher_columbus, .barrack_obama, .block_user");
-      });
-
-      it("applies infield labels", function(){
-        spyOn($.fn, "placeholder");
-        this.view.render();
-        expect($.fn.placeholder).toHaveBeenCalled();
-        expect($.fn.placeholder.calls.mostRecent().object.selector).toBe("input, textarea");
+        expect(
+          $.fn.tooltip.calls.mostRecent().object.is(".christopher_columbus, .barrack_obama, .block_user")
+        ).toBe(true);
       });
     });
   });
 
   describe("#renderTemplate", function(){
+    beforeEach(function() {
+      this.view.$el.htmlOriginal = this.view.$el.html;
+      spyOn(this.view.$el, "html").and.callFake(function() {
+        this.htmlOriginal("<input><textarea/></input>");
+        return this;
+      });
+    });
+
     it("calls jQuery.placeholder() for inputs", function() {
       spyOn($.fn, "placeholder");
       this.view.renderTemplate();
       expect($.fn.placeholder).toHaveBeenCalled();
-      expect($.fn.placeholder.calls.mostRecent().object.selector).toBe("input, textarea");
+      expect($.fn.placeholder.calls.mostRecent().object.is("input, textarea")).toBe(true);
     });
 
     it("initializes autosize for textareas", function(){
       spyOn(window, "autosize");
       this.view.renderTemplate();
       expect(window.autosize).toHaveBeenCalled();
-      expect(window.autosize.calls.mostRecent().args[0].selector).toBe("textarea");
+      expect(window.autosize.calls.mostRecent().args[0].is("textarea")).toBe(true);
     });
   });
 });
diff --git a/spec/javascripts/mobile/mobile_application_spec.js b/spec/javascripts/mobile/mobile_application_spec.js
index 3b6ecc4c9..c5a301551 100644
--- a/spec/javascripts/mobile/mobile_application_spec.js
+++ b/spec/javascripts/mobile/mobile_application_spec.js
@@ -8,7 +8,7 @@ describe("Diaspora.Mobile", function(){
     it("calls autosize for textareas", function(){
       Diaspora.Mobile.initialize();
       expect(window.autosize).toHaveBeenCalled();
-      expect(window.autosize.calls.mostRecent().args[0].selector).toBe("textarea");
+      expect(window.autosize.calls.mostRecent().args[0].is($("textarea"))).toBe(true);
     });
 
     it("deactivates shield", function(){

From 2739259fb63509e1ce29aec63c2e574167d1026d Mon Sep 17 00:00:00 2001
From: cmrd Senya <senya@riseup.net>
Date: Sun, 29 Jan 2017 04:40:39 +0200
Subject: [PATCH 3/3] some more replacements for deprecated jquery methods

closes #7356
---
 Changelog.md                                              | 1 +
 app/assets/javascripts/api/authorization_page.js          | 2 +-
 app/assets/javascripts/app/views.js                       | 2 +-
 app/assets/javascripts/app/views/contact_stream_view.js   | 2 +-
 .../javascripts/app/views/notification_dropdown_view.js   | 2 +-
 .../javascripts/app/views/publisher/uploader_view.js      | 2 +-
 app/assets/javascripts/app/views/publisher_view.js        | 2 +-
 app/assets/javascripts/app/views/tags_view.js             | 2 +-
 spec/javascripts/app/router_spec.js                       | 8 ++++++--
 spec/javascripts/app/views/publisher_mention_view_spec.js | 2 +-
 spec/javascripts/app/views/publisher_view_spec.js         | 3 ++-
 11 files changed, 17 insertions(+), 11 deletions(-)

diff --git a/Changelog.md b/Changelog.md
index 7bea354f9..1d11ac178 100644
--- a/Changelog.md
+++ b/Changelog.md
@@ -2,6 +2,7 @@
 
 ## Refactor
 * Remove unused setPreload function [#7354](https://github.com/diaspora/diaspora/pull/7354)
+* Remove jQuery deprecations [#7356](https://github.com/diaspora/diaspora/pull/7356)
 
 ## Bug fixes
 
diff --git a/app/assets/javascripts/api/authorization_page.js b/app/assets/javascripts/api/authorization_page.js
index d61b941ec..84b0f31b8 100644
--- a/app/assets/javascripts/api/authorization_page.js
+++ b/app/assets/javascripts/api/authorization_page.js
@@ -1,5 +1,5 @@
 $(document).ready(function() {
-  $("#js-app-logo").error(function () {
+  $("#js-app-logo").on("error", function() {
     $(this).attr("src", ImagePaths.get("user/default.png"));
   });
 });
diff --git a/app/assets/javascripts/app/views.js b/app/assets/javascripts/app/views.js
index ebd8901a9..d0bede4ea 100644
--- a/app/assets/javascripts/app/views.js
+++ b/app/assets/javascripts/app/views.js
@@ -56,7 +56,7 @@ app.views.Base = Backbone.View.extend({
       .attr("data-template", _.last(this.templateName.split("/")));
 
     // add avatar fallback if it can't be loaded
-    this.$el.find(this.avatars.selector).error(this.avatars.fallback);
+    this.$el.find(this.avatars.selector).on("error", this.avatars.fallback);
 
     // add placeholder support for old browsers
     this.$("input, textarea").placeholder();
diff --git a/app/assets/javascripts/app/views/contact_stream_view.js b/app/assets/javascripts/app/views/contact_stream_view.js
index cbc254c37..0fe60155e 100644
--- a/app/assets/javascripts/app/views/contact_stream_view.js
+++ b/app/assets/javascripts/app/views/contact_stream_view.js
@@ -18,7 +18,7 @@ app.views.ContactStream = Backbone.View.extend({
     $("#paginate .loader").removeClass("hidden");
     $.ajax(this._fetchUrl(), {
       context: this
-    }).success(function(response) {
+    }).done(function(response) {
       if (response.length === 0) {
         this.onEmptyResponse();
       } else {
diff --git a/app/assets/javascripts/app/views/notification_dropdown_view.js b/app/assets/javascripts/app/views/notification_dropdown_view.js
index c3f8f1911..02c3a7137 100644
--- a/app/assets/javascripts/app/views/notification_dropdown_view.js
+++ b/app/assets/javascripts/app/views/notification_dropdown_view.js
@@ -96,7 +96,7 @@ app.views.NotificationDropdown = app.views.Base.extend({
 
   afterNotificationChanges: function(node) {
     node.find(".unread-toggle .entypo-eye").tooltip("destroy").tooltip();
-    node.find(this.avatars.selector).error(this.avatars.fallback);
+    node.find(this.avatars.selector).on("error", this.avatars.fallback);
   },
 
   finishLoading: function() {
diff --git a/app/assets/javascripts/app/views/publisher/uploader_view.js b/app/assets/javascripts/app/views/publisher/uploader_view.js
index 775cf69bb..841952112 100644
--- a/app/assets/javascripts/app/views/publisher/uploader_view.js
+++ b/app/assets/javascripts/app/views/publisher/uploader_view.js
@@ -149,7 +149,7 @@ app.views.PublisherUploader = Backbone.View.extend({
       dataType: "json",
       type: "DELETE",
       success: function() {
-        $.when(photo.fadeOut(400)).then(function(){
+        photo.fadeOut(400, function() {
           photo.remove();
 
           if( self.publisher.$(".publisher_photo").length === 0 ) {
diff --git a/app/assets/javascripts/app/views/publisher_view.js b/app/assets/javascripts/app/views/publisher_view.js
index de9245027..a80ef3555 100644
--- a/app/assets/javascripts/app/views/publisher_view.js
+++ b/app/assets/javascripts/app/views/publisher_view.js
@@ -44,7 +44,7 @@ app.views.Publisher = Backbone.View.extend({
     // if there is data in the publisher we ask for a confirmation
     // before the user is able to leave the page
     $(window).on("beforeunload", _.bind(this._beforeUnload, this));
-    $(window).unload(this.clear.bind(this));
+    $(window).on("unload", this.clear.bind(this));
 
     // sync textarea content
     if( this.hiddenInputEl.val() === "" ) {
diff --git a/app/assets/javascripts/app/views/tags_view.js b/app/assets/javascripts/app/views/tags_view.js
index 9b70df1e6..3dda23c05 100644
--- a/app/assets/javascripts/app/views/tags_view.js
+++ b/app/assets/javascripts/app/views/tags_view.js
@@ -6,7 +6,7 @@ app.views.Tags = Backbone.View.extend({
       app.publisher.setText("#"+ opts.hashtagName + " ");
     }
     // add avatar fallback if it can't be loaded
-    $(app.views.Base.prototype.avatars.selector).error(app.views.Base.prototype.avatars.fallback);
+    $(app.views.Base.prototype.avatars.selector).on("error", app.views.Base.prototype.avatars.fallback);
   }
 });
 // @license-end
diff --git a/spec/javascripts/app/router_spec.js b/spec/javascripts/app/router_spec.js
index bafbe5568..c6ac8653d 100644
--- a/spec/javascripts/app/router_spec.js
+++ b/spec/javascripts/app/router_spec.js
@@ -123,14 +123,18 @@ describe('app.Router', function () {
   });
 
   describe("gettingStarted", function() {
+    beforeEach(function() {
+      spec.content().append($("<div id='hello-there'>"));
+    });
+
     it("renders app.pages.GettingStarted", function() {
       app.router.navigate("/getting_started", {trigger: true});
-      expect(app.page.$el.selector).toEqual("#hello-there");
+      expect(app.page.$el.is($("#hello-there"))).toBe(true);
     });
 
     it("renders app.pages.GettingStarted when the URL has a trailing slash", function() {
       app.router.navigate("/getting_started/", {trigger: true});
-      expect(app.page.$el.selector).toEqual("#hello-there");
+      expect(app.page.$el.is($("#hello-there"))).toBe(true);
     });
   });
 
diff --git a/spec/javascripts/app/views/publisher_mention_view_spec.js b/spec/javascripts/app/views/publisher_mention_view_spec.js
index d4e9417e3..7f5ced47c 100644
--- a/spec/javascripts/app/views/publisher_mention_view_spec.js
+++ b/spec/javascripts/app/views/publisher_mention_view_spec.js
@@ -16,7 +16,7 @@ describe("app.views.PublisherMention", function() {
       this.view = new app.views.PublisherMention({ el: "#publisher" });
       expect(app.views.SearchBase.prototype.initialize).toHaveBeenCalled();
       var call = app.views.SearchBase.prototype.initialize.calls.mostRecent();
-      expect(call.args[0].typeaheadInput.selector).toBe("#publisher .typeahead-mention-box");
+      expect(call.args[0].typeaheadInput.is($("#publisher .typeahead-mention-box"))).toBe(true);
       expect(call.args[0].customSearch).toBeTruthy();
       expect(call.args[0].autoselect).toBeTruthy();
       expect(call.args[0].remoteRoute).toEqual({url: "/contacts"});
diff --git a/spec/javascripts/app/views/publisher_view_spec.js b/spec/javascripts/app/views/publisher_view_spec.js
index c672b33fa..2e7c3b4dd 100644
--- a/spec/javascripts/app/views/publisher_view_spec.js
+++ b/spec/javascripts/app/views/publisher_view_spec.js
@@ -614,7 +614,7 @@ describe("app.views.Publisher", function() {
     });
 
     context('photo removal', function() {
-      beforeEach(function() {
+      beforeEach(function(done) {
         this.view = new app.views.Publisher();
         this.view.wrapperEl.addClass("with_attachments");
         this.view.photozoneEl.html(
@@ -626,6 +626,7 @@ describe("app.views.Publisher", function() {
         );
 
         spyOn(jQuery, 'ajax').and.callFake(function(opts) { opts.success(); });
+        this.view.viewUploader.on("change", done);
         this.view.photozoneEl.find(".x").click();
       });