From fcda3a338c1a2cc08337dcc594a3f67bc4e8d879 Mon Sep 17 00:00:00 2001
From: Michael Gapczynski <GapczynskiM@gmail.com>
Date: Wed, 20 Jul 2011 16:41:39 -0400
Subject: [PATCH] Fix bug in constructor for appending numbers to already
 existing targets

---
 apps/files_sharing/lib_share.php | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/apps/files_sharing/lib_share.php b/apps/files_sharing/lib_share.php
index 848c44788a..ae730a21a5 100644
--- a/apps/files_sharing/lib_share.php
+++ b/apps/files_sharing/lib_share.php
@@ -47,16 +47,18 @@ class OC_SHARE {
 					$target = "/".$uid."/files/Share/".basename($source);
 					$check = OC_DB::prepare("SELECT target FROM *PREFIX*sharing WHERE target = ? AND uid_shared_with = ?");
 					$result = $check->execute(array($target, $uid))->fetchAll();
+					// Check if target already exists for the user, if it does append a number to the name
 					if (count($result) > 0) {
 						if ($pos = strrpos($target, ".")) {
 							$name = substr($target, 0, $pos);
 							$ext = substr($target, $pos);
 						} else {
 							$name = $target;
+							$ext = "";
 						}
 						$counter = 1;
 						while (count($result) > 0) {
-							$newTarget = $name."_".$counter.isset($ext);
+							$newTarget = $name."_".$counter.$ext;
 							$result = $check->execute(array($newTarget, $uid))->fetchAll();
 							$counter++;
 						}
-- 
GitLab