From 23c8d7b3fb316daa7c70552ed47e63e9bbcd300f Mon Sep 17 00:00:00 2001 From: Michael Gapczynski Date: Mon, 8 Aug 2011 15:49:16 -0400 Subject: [PATCH] Show which parent folder is shared with the user in the sharing drop down --- apps/files_sharing/ajax/getitem.php | 10 ++++++---- apps/files_sharing/js/share.js | 5 +---- 2 files changed, 7 insertions(+), 8 deletions(-) diff --git a/apps/files_sharing/ajax/getitem.php b/apps/files_sharing/ajax/getitem.php index 3a3be404a3..43105d334e 100644 --- a/apps/files_sharing/ajax/getitem.php +++ b/apps/files_sharing/ajax/getitem.php @@ -9,12 +9,14 @@ $source = $userDirectory.$_GET['source']; $users = OC_Share::getMySharedItem($source); $source = dirname($source); while ($source != "" && $source != "/" && $source != "." && $source != $userDirectory) { - $parentUsers = array(); $values = array_values(OC_Share::getMySharedItem($source)); - for ($i = 0; $i < count($values); $i++) { - $parentUsers[basename($source)."-".$i] = $values[$i]; + if (count($values) > 0) { + $parentUsers = array(); + for ($i = 0; $i < count($values); $i++) { + $parentUsers[basename($source)."-".$i] = $values[$i]; + } + $users = array_merge($users, $parentUsers); } - $users = array_merge($users, $parentUsers); $source = dirname($source); } echo json_encode($users); diff --git a/apps/files_sharing/js/share.js b/apps/files_sharing/js/share.js index 0b97010c4f..d6cf45bf44 100644 --- a/apps/files_sharing/js/share.js +++ b/apps/files_sharing/js/share.js @@ -128,10 +128,7 @@ function createShareDropdown(filenames, files) { $.each(users, function(index, row) { if (typeof(index) == 'string') { // TODO typeof not always working, group together users that have parent folders shared with them - list += "
  • disabled"; - list += index; - list += row.uid_shared_with; - list += "
  • "; + list += "
  • Parent folder "+index.substr(0, index.lastIndexOf('-'))+" shared with "+row.uid_shared_with+"
  • "; } else { list += "
  • "; list += row.uid_shared_with;