From 609193a5da4750978031806a8504204e35730887 Mon Sep 17 00:00:00 2001 From: Izaak Alpert Date: Fri, 12 Apr 2013 12:55:02 -0400 Subject: [PATCH] Nil guard owner on project deletion It's possible to delete a project after the owner has left, this project will have a nil owner, this nil guards against that Change-Id: Ice5bca6aa91e8543a41d212d92e7753e9b4aa287 --- app/services/system_hooks_service.rb | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/app/services/system_hooks_service.rb b/app/services/system_hooks_service.rb index 132bb14a675..462128d33d2 100644 --- a/app/services/system_hooks_service.rb +++ b/app/services/system_hooks_service.rb @@ -27,9 +27,11 @@ class SystemHooksService name: model.name, path: model.path, project_id: model.id, + }) + data.merge!({ owner_name: model.owner.name, owner_email: model.owner.email - }) + }) unless model.owner.nil? when User data.merge!({ name: model.name, -- GitLab