Redmine 4 compatibility

app/controllers/app_notifications_controller.rb

@@ -34,7 +34,7 @@ def index
   def view
     @notification = AppNotification.find(params[:id])
     if @notification.recipient == User.current 
-      AppNotification.update(@notification, :viewed => true)
+      AppNotification.update(@notification.id, :viewed => true)
       if request.xhr?
         if @notification.is_edited?
           render :partial => 'issues/issue_edit', :formats => [:html], :locals => { :notification => @notification, :journal => @notification.journal }

db/migrate/001_create_app_notifications.rb

@@ -1,4 +1,4 @@
-class CreateAppNotifications < ActiveRecord::Migration
+class CreateAppNotifications < Rails.version < '5.1' ? ActiveRecord::Migration : ActiveRecord::Migration[4.2]
   def change
     create_table :app_notifications do |t|
       t.datetime :created_on

db/migrate/002_update_users.rb

@@ -1,4 +1,4 @@
-class UpdateUsers < ActiveRecord::Migration
+class UpdateUsers < Rails.version < '5.1' ? ActiveRecord::Migration : ActiveRecord::Migration[4.2]
     def self.up
         change_table :users do |t|
             t.column :app_notification, :boolean, :default => false

db/migrate/003_update_users2.rb

@@ -1,4 +1,4 @@
-class UpdateUsers2 < ActiveRecord::Migration
+class UpdateUsers2 < Rails.version < '5.1' ? ActiveRecord::Migration : ActiveRecord::Migration[4.2]
     def self.up
         change_table :users do |t|
             t.column :app_notification_desktop, :boolean, :default => false

lib/app_notifications_account_patch.rb

@@ -7,7 +7,8 @@ def self.included(base) # :nodoc:
     base.class_eval do
       unloadable # Send unloadable so it will not be unloaded in development
 
-      alias_method_chain :account, :in_app_option
+      alias_method :account_without_in_app_option, :account
+      alias_method :account, :account_with_in_app_option
     end
   end