deprecated two site functions and removed a use of previously deprecated function...

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

diff --git a/engine/classes/ElggSite.php b/engine/classes/ElggSite.php
index 6c70d176b965322abd9291cb30df841abe179f52..93d442e0afe665b00de1646042eb059be9d0f4ea 100644 (file)
--- a/engine/classes/ElggSite.php
+++ b/engine/classes/ElggSite.php
@@ -204,7 +204,14 @@ class ElggSite extends ElggEntity {
         * @return array of ElggUsers
         */
        public function getMembers($limit = 10, $offset = 0) {
-               get_site_members($this->getGUID(), $limit, $offset);
+               return elgg_get_entities_from_relationship(array(
+                       'relationship' => 'member_of_site',
+                       'relationship_guid' => $this->getGUID(),
+                       'inverse_relationship' => TRUE,
+                       'types' => 'user',
+                       'limit' => $limit,
+                       'offset' => $offset
+               ));
        }
 
        /**

diff --git a/engine/lib/sites.php b/engine/lib/sites.php
index 79b71d901d2c76966cecca4378271d473045ad65..cf838e720a571ffc8fc4da7f1ba04f7f2eeb12df 100644 (file)
--- a/engine/lib/sites.php
+++ b/engine/lib/sites.php
@@ -139,8 +139,12 @@ function remove_site_user($site_guid, $user_guid) {
  * @param int $offset    Offset
  *
  * @return mixed
+ * @deprecated 1.8 Use ElggSite::getMembers()
  */
 function get_site_members($site_guid, $limit = 10, $offset = 0) {
+       elgg_deprecated_notice("get_site_members() deprecated.
+               Use ElggSite::getMembers()", 1.8);
+
        $site_guid = (int)$site_guid;
        $limit = (int)$limit;
        $offset = (int)$offset;
@@ -162,8 +166,13 @@ function get_site_members($site_guid, $limit = 10, $offset = 0) {
  * @param bool $fullview  Whether or not to display the full view (default: true)
  *
  * @return string A displayable list of members
+ * @deprecated 1.8 Use elgg_list_entities_from_relationships() with relationship
+ *                 'member_of_site'
  */
 function list_site_members($site_guid, $limit = 10, $fullview = true) {
+       elgg_deprecated_notice("list_site_members() deprecated.
+               Use elgg_list_entities_from_relationships()", 1.8);
+
        $offset = (int) get_input('offset');
        $limit = (int) $limit;
        $options = array(
@@ -179,7 +188,6 @@ function list_site_members($site_guid, $limit = 10, $fullview = true) {
        $entities = get_site_members($site_guid, $limit, $offset);
 
        return elgg_view_entity_list($entities, $count, $offset, $limit, $fullview);
-
 }
 
 /**

diff --git a/engine/lib/views.php b/engine/lib/views.php
index f20eabfbfffbc74b8a9655d26b66bd328982253a..0c9e83bab38eb2bda6010743259a2e81e61348c6 100644 (file)
--- a/engine/lib/views.php
+++ b/engine/lib/views.php
@@ -786,13 +786,11 @@ function elgg_view_annotation(ElggAnnotation $annotation, $bypass = true, $debug
  * Returns a rendered list of entities with pagination. This function should be
  * called by wrapper functions.
  *
- * @see list_entities()
- * @see list_user_objects()
+ * @see elgg_list_entities()
  * @see list_user_friends_objects()
- * @see list_entities_from_metadata()
- * @see list_entities_from_metadata_multi()
- * @see list_entities_from_relationships()
- * @see list_site_members()
+ * @see elgg_list_entities_from_metadata()
+ * @see elgg_list_entities_from_relationships()
+ * @see elgg_list_entities_from_annotations()
  *
  * @param array $entities       List of entities
  * @param int   $count          The total number of entities across all pages

diff --git a/mod/thewire/index.php b/mod/thewire/index.php
index 36dc5ddab0c20aa73e077088326fd089672abab0..6cd49e472fa8524eb3c11e230676a3aebbf5d81d 100644 (file)
--- a/mod/thewire/index.php
+++ b/mod/thewire/index.php
@@ -27,7 +27,12 @@
                $area2 .= elgg_view("thewire/forms/add");
 
        // Display the user's wire
-               $area2 .= list_user_objects($page_owner->getGUID(),'thewire');
+               $options = array(
+                       'type' => 'object',
+                       'subtype' => 'thewire',
+                       'owner_guid' => $page_owner->getGUID()
+               );
+               $area2 .= elgg_list_entities($options);
 
        //select the correct canvas area
                $body = elgg_view_layout("one_column_with_sidebar", array('content' => $area2));