From 0fa30cfadd541f4b87741d0999012601988a51cd Mon Sep 17 00:00:00 2001
From: Karsten Heiken <karsten@disposed.de>
Date: Wed, 28 Sep 2011 16:28:07 +0200
Subject: [PATCH 1/2] Don't list the user himself when sharing a project

Sharing with yourself is not a good idea.
---
 application/views/projects/shares.php | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/application/views/projects/shares.php b/application/views/projects/shares.php
index 868b406..173ddb8 100644
--- a/application/views/projects/shares.php
+++ b/application/views/projects/shares.php
@@ -50,6 +50,10 @@
 						<select name="user_id" id="user_id" class="drop">
 <?php
 	foreach ($this->user->getAll() as $user):
+
+		// don't list the user himself
+		if($user['id'] === $this->access->profile()->id)
+			continue;
 ?>
 							<option value="<?=$user['id'];?>"><?=$user['firstname'];?> <?=$user['lastname'];?></option>
 <?php

From 493eb036c7254c37b1bb78db285046b9089eec82 Mon Sep 17 00:00:00 2001
From: Karsten Heiken <karsten@disposed.de>
Date: Wed, 28 Sep 2011 16:30:09 +0200
Subject: [PATCH 2/2] Better: Don't share with the user himself or the actual
 owner

---
 application/views/projects/shares.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/application/views/projects/shares.php b/application/views/projects/shares.php
index 173ddb8..662c046 100644
--- a/application/views/projects/shares.php
+++ b/application/views/projects/shares.php
@@ -52,7 +52,7 @@
 	foreach ($this->user->getAll() as $user):
 
 		// don't list the user himself
-		if($user['id'] === $this->access->profile()->id)
+		if($user['id'] === $this->access->profile()->id || $user['id'] === $project['owner'])
 			continue;
 ?>
 							<option value="<?=$user['id'];?>"><?=$user['firstname'];?> <?=$user['lastname'];?></option>