From f27f1a23be067f53a1e3c192eef12eb5045eb1b9 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?David=20Barrag=C3=A1n=20Merino?=
 <david.barragan@kaleidos.net>
Date: Mon, 28 Mar 2016 20:44:28 +0200
Subject: [PATCH] Fix Issue #227 (tg-sub): Redirect to home if a leave a
 project from the admin panel

---
 app/coffee/modules/admin/memberships.coffee   | 27 ++++++++++---------
 app/coffee/modules/team/main.coffee           |  2 +-
 .../lightbox-leave-project-warning.jade       |  6 ++---
 3 files changed, 19 insertions(+), 16 deletions(-)

diff --git a/app/coffee/modules/admin/memberships.coffee b/app/coffee/modules/admin/memberships.coffee
index d50009b0..127d3959 100644
--- a/app/coffee/modules/admin/memberships.coffee
+++ b/app/coffee/modules/admin/memberships.coffee
@@ -385,7 +385,8 @@ module.directive("tgMembershipsRowRoleSelector", ["$log", "$tgRepo", "$tgConfirm
 ## Member Actions Directive
 #############################################################################
 
-MembershipsRowActionsDirective = ($log, $repo, $rs, $confirm, $compile, $translate, currentUserService, lightboxFactory) ->
+MembershipsRowActionsDirective = ($log, $repo, $rs, $confirm, $compile, $translate, $location,
+                                  $navUrls, lightboxFactory) ->
     activedTemplate = """
     <div class="active"
          translate="ADMIN.MEMBERSHIP.STATUS_ACTIVE">
@@ -448,20 +449,22 @@ MembershipsRowActionsDirective = ($log, $repo, $rs, $confirm, $compile, $transla
             $confirm.askOnDelete(title, message).then (askResponse) ->
                 onSuccess = =>
                     askResponse.finish()
+                    if member.user != $scope.user.id
+                        if $scope.page > 1 && ($scope.count - 1) <= $scope.paginatedBy
+                            $ctrl.selectFilter("page", $scope.page - 1)
 
-                    if $scope.page > 1 && ($scope.count - 1) <= $scope.paginatedBy
-                        $ctrl.selectFilter("page", $scope.page - 1)
+                        $ctrl.loadMembers()
+                    else
+                        $location.path($navUrls.resolve("home"))
 
-                    $ctrl.loadMembers()
-
-                    text = $translate.instant("ADMIN.MEMBERSHIP.SUCCESS_DELETE")
-                    $confirm.notify("success", null, text)
+                    text = $translate.instant("ADMIN.MEMBERSHIP.SUCCESS_DELETE", {message: message})
+                    $confirm.notify("success", text, null, 5000)
 
                 onError = =>
                     askResponse.finish(false)
 
                     text = $translate.instant("ADMIN.MEMBERSHIP.ERROR_DELETE", {message: message})
-                    $confirm.notify("error", null, text)
+                    $confirm.notify("error", text)
 
                 $repo.remove(member).then(onSuccess, onError)
 
@@ -469,13 +472,12 @@ MembershipsRowActionsDirective = ($log, $repo, $rs, $confirm, $compile, $transla
             event.preventDefault()
 
             if $scope.project.owner.id == member.user
-                currentUser = currentUserService.getUser()
-                isCurrentUser = currentUser.get('id') == member.user
+                isCurrentUser = $scope.user.id == member.user
 
                 lightboxFactory.create("tg-lightbox-leave-project-warning", {
                     class: "lightbox lightbox-leave-project-warning"
                 }, {
-                    currentUser: isCurrentUser,
+                    isCurrentUser: isCurrentUser,
                     project: $scope.project
                 })
             else
@@ -487,7 +489,8 @@ MembershipsRowActionsDirective = ($log, $repo, $rs, $confirm, $compile, $transla
     return {link: link}
 
 module.directive("tgMembershipsRowActions", ["$log", "$tgRepo", "$tgResources", "$tgConfirm", "$compile",
-                                             "$translate", "tgCurrentUserService", "tgLightboxFactory", MembershipsRowActionsDirective])
+                                             "$translate", "$tgLocation", "$tgNavUrls", "tgLightboxFactory",
+                                             MembershipsRowActionsDirective])
 
 
 #############################################################################
diff --git a/app/coffee/modules/team/main.coffee b/app/coffee/modules/team/main.coffee
index 40b08514..b82a61e9 100644
--- a/app/coffee/modules/team/main.coffee
+++ b/app/coffee/modules/team/main.coffee
@@ -248,7 +248,7 @@ LeaveProjectDirective = ($repo, $confirm, $location, $rs, $navurls, $translate,
                 lightboxFactory.create("tg-lightbox-leave-project-warning", {
                     class: "lightbox lightbox-leave-project-warning"
                 }, {
-                    currentUser: true,
+                    isCurrentUser: true,
                     project: $scope.project
                 })
             else
diff --git a/app/partials/common/lightbox/lightbox-leave-project-warning.jade b/app/partials/common/lightbox/lightbox-leave-project-warning.jade
index 24210682..51fd7caa 100644
--- a/app/partials/common/lightbox/lightbox-leave-project-warning.jade
+++ b/app/partials/common/lightbox/lightbox-leave-project-warning.jade
@@ -7,14 +7,14 @@ div.content
 
     h2.title {{'LIGHTBOX.LEAVE_PROJECT_WARNING.TITLE' | translate}}
 
-    div(ng-if="currentUser")
+    div(ng-if="isCurrentUser")
         p {{'LIGHTBOX.LEAVE_PROJECT_WARNING.CURRENT_USER_OWNER.DESC' | translate}}
 
         a.button-green(tg-nav="project-admin-home:project=project.slug", href="")
             span(translate="LIGHTBOX.LEAVE_PROJECT_WARNING.CURRENT_USER_OWNER.BUTTON")
 
-    div(ng-if="!currentUser")
+    div(ng-if="!isCurrentUser")
         p {{'LIGHTBOX.LEAVE_PROJECT_WARNING.OTHER_USER_OWNER.DESC' | translate}}
 
         a.button-green(tg-nav="project-admin-home:project=project.slug", href="")
-            span(translate="LIGHTBOX.LEAVE_PROJECT_WARNING.OTHER_USER_OWNER.BUTTON")
\ No newline at end of file
+            span(translate="LIGHTBOX.LEAVE_PROJECT_WARNING.OTHER_USER_OWNER.BUTTON")