Closes #1015: Added a prompt before deleting an access collection. Also cleaned...

git-svn-id: https://code.elgg.org/elgg/trunk@3363 36083f99-b078-4883-b0ff-0f9b5a30f544

diff --git a/views/default/friends/collection.php b/views/default/friends/collection.php
index cb3e2674cc6ac18802a452a03f84d2472a8d9692..bcb127c4a9858dfc378b5e6184abcf6b9b2fc070 100644 (file)
--- a/views/default/friends/collection.php
+++ b/views/default/friends/collection.php
@@ -26,17 +26,19 @@
                        \r
                        echo "<li><h2>";\r
                \r
-               //as collections are private, check that the logged in user is the owner\r
-               if($coll->owner_guid == $_SESSION['user']->getGUID())\r
-                   echo "<div class=\"friends_collections_controls\"> <a href=\"" . $vars['url'] . "action/friends/deletecollection?collection={$coll->id}\" class=\"delete_collection\" onclick=\"$('div.friends_picker').empty();\"> </a>";\r
-                   \r
+                       //as collections are private, check that the logged in user is the owner\r
+                       if($coll->owner_guid == $_SESSION['user']->getGUID()) {\r
+                               $confirm = addslashes(elgg_echo('question:areyousure'));\r
+                               echo "<div class=\"friends_collections_controls\"> <a href=\"" . $vars['url'] . "action/friends/deletecollection?collection={$coll->id}\" class=\"delete_collection\" onclick=\"if (!confirm('{$confirm}')) { return false; }; $('div.friends_picker').empty();\"> </a>";\r
+                       }\r
+                       \r
                        echo "</div>";\r
                        echo $coll->name;\r
                        echo " (<span id=\"friends_membership_count{$vars['friendspicker']}\">{$count}</span>) </h2>";\r
-               \r
-               // individual collection panels\r
-               if($friends = $vars['collection']->entities){\r
-                       $content = elgg_view('friends/collectiontabs', array('owner' => $_SESSION['user'], 'collection' => $vars['collection'], 'friendspicker' => $vars['friendspicker']));\r
+                       \r
+                       // individual collection panels\r
+                       if($friends = $vars['collection']->entities){\r
+                               $content = elgg_view('friends/collectiontabs', array('owner' => $_SESSION['user'], 'collection' => $vars['collection'], 'friendspicker' => $vars['friendspicker']));\r
                                echo elgg_view('friends/picker',array('entities' => $friends, 'value' => $members, 'content' => $content, 'replacement' => '', 'friendspicker' => $vars['friendspicker']));\r
                                ?>\r
                                \r
@@ -45,12 +47,12 @@ $(document).ready(function () {
                \r
                $('#friends_picker_placeholder<?php echo $vars['friendspicker']; ?>').load('<?php echo $vars['url']; ?>friends/pickercallback.php?username=<?php echo $_SESSION['user']->username; ?>&type=list&collection=<?php echo $vars['collection']->id; ?>');\r
                                \r
-    });\r
+       });\r
 </script>\r
                                <?php\r
-           }\r
-           \r
-           // close friends_picker div and the accordian list item\r
-           echo "</li>";\r
+                       }\r
+                       \r
+                       // close friends_picker div and the accordian list item\r
+                       echo "</li>";\r
 \r
 ?>
\ No newline at end of file