Remove SystemHook observer

4 files changed, 8 insertions, 18 deletions

diff --git a/app/observers/base_observer.rb b/app/observers/base_observer.rb
index 260d1f05db3..5a821d269a2 100644
--- a/app/observers/base_observer.rb
+++ b/app/observers/base_observer.rb
@@ -10,4 +10,8 @@ class BaseObserver < ActiveRecord::Observer
   def log_info message
     Gitlab::AppLogger.info message
   end
+
+  def system_hook_service
+    SystemHooksService.new
+  end
 end
diff --git a/app/observers/system_hook_observer.rb b/app/observers/system_hook_observer.rb
deleted file mode 100644
index 80de177b9a2..00000000000
--- a/app/observers/system_hook_observer.rb
+++ /dev/null
@@ -1,17 +0,0 @@
-class SystemHookObserver < BaseObserver
-  observe :user, :project, :users_project
-
-  def after_create(model)
-    system_hook_service.execute_hooks_for(model, :create)
-  end
-
-  def after_destroy(model)
-    system_hook_service.execute_hooks_for(model, :destroy)
-  end
-
-  private
-
-  def system_hook_service
-    SystemHooksService.new
-  end
-end
diff --git a/app/observers/user_observer.rb b/app/observers/user_observer.rb
index fba0f1006d9..a7b1a857e29 100644
--- a/app/observers/user_observer.rb
+++ b/app/observers/user_observer.rb
@@ -1,12 +1,13 @@
 class UserObserver < BaseObserver
   def after_create(user)
     log_info("User \"#{user.name}\" (#{user.email}) was created")
-
     notification.new_user(user)
+    system_hook_service.execute_hooks_for(user, :create)
   end
 
   def after_destroy user
     log_info("User \"#{user.name}\" (#{user.email})  was removed")
+    system_hook_service.execute_hooks_for(user, :destroy)
   end
 
   def after_save user
diff --git a/app/observers/users_project_observer.rb b/app/observers/users_project_observer.rb
index 44c72b30187..b0c37d1d222 100644
--- a/app/observers/users_project_observer.rb
+++ b/app/observers/users_project_observer.rb
@@ -7,6 +7,7 @@ class UsersProjectObserver < BaseObserver
     )
 
     notification.new_team_member(users_project)
+    system_hook_service.execute_hooks_for(users_project, :create)
   end
 
   def after_update(users_project)
@@ -19,5 +20,6 @@ class UsersProjectObserver < BaseObserver
       action: Event::LEFT,
       author_id: users_project.user.id
     )
+    system_hook_service.execute_hooks_for(users_project, :destroy)
   end
 end