Add elgg_deprecated_notice() so we don't have to re-write all the elgg_log() deprecat...

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

diff --git a/CHANGES.txt b/CHANGES.txt
index 2c47d2ff34a35d4be99fa579cd8f464f935b4f80..2e8e658a24649a32146938d59df56163194499f7 100644 (file)
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -38,6 +38,7 @@ http://code.elgg.org/elgg/.....
   * can_write_to_container() can be overridden with the container_permissions_check hook. #1164 (part 2).
   * Deprecated search_for_*().
   * Deprecated search_list*().
+  * Added elgg_deprecated_notice().
 
  Services API:
   * Separated user and api authenticate processing

diff --git a/engine/lib/access.php b/engine/lib/access.php
index b508f1fc0cb8d29607a73644fab6f097a97bc881..4c448721f3896f6b84e07044802d9e78635ec4bb 100644 (file)
--- a/engine/lib/access.php
+++ b/engine/lib/access.php
@@ -671,7 +671,7 @@ function elgg_view_access_collections($owner_guid) {
  */
 function get_entities_from_access_id($collection_id, $entity_type = "", $entity_subtype = "", $owner_guid = 0, $limit = 10, $offset = 0, $order_by = "", $site_guid = 0, $count = false) {
        // log deprecated warning
-       elgg_log('get_entities_from_access_id() was deprecated in 1.7 by elgg_get_entities()!', 'WARNING');
+       elgg_deprecated_notice('get_entities_from_access_id() was deprecated by elgg_get_entities()!', 1.7);
 
        if (!$collection_id) {
                return FALSE;
@@ -875,4 +875,4 @@ register_elgg_event_handler('init', 'system', 'access_init', 9999);
 
 // For overrided permissions
 register_plugin_hook('permissions_check', 'all', 'elgg_override_permissions_hook');
-register_plugin_hook('container_permissions_check', 'all', 'elgg_override_permissions_hook');
\ No newline at end of file
+register_plugin_hook('container_permissions_check', 'all', 'elgg_override_permissions_hook');

diff --git a/engine/lib/annotations.php b/engine/lib/annotations.php
index 6d5c0b46a73dca67543bfedfc2aa9d392e5ebb0a..105282c8cac3513405c22fbe517e22e43a911bba 100644 (file)
--- a/engine/lib/annotations.php
+++ b/engine/lib/annotations.php
@@ -695,8 +695,7 @@ function elgg_get_entity_annotation_where_sql($table, $names = NULL, $values = N
 function get_entities_from_annotations($entity_type = "", $entity_subtype = "", $name = "", $value = "",
 $owner_guid = 0, $group_guid = 0, $limit = 10, $offset = 0, $order_by = "asc", $count = false,
 $timelower = 0, $timeupper = 0) {
-
-       elgg_log('get_entities_from_annotations() was deprecated in 1.7 by elgg_get_entities_from_annotations()!', 'WARNING');
+       elgg_deprecated_notice('get_entities_from_annotations() was deprecated by elgg_get_entities_from_annotations().', 1.3);
 
        $options = array();
 

diff --git a/engine/lib/elgglib.php b/engine/lib/elgglib.php
index 698a34b931e6d557893b71ac805d9c1444faf4f0..cd426309d729c0525b10b45196746737ce39045d 100644 (file)
--- a/engine/lib/elgglib.php
+++ b/engine/lib/elgglib.php
@@ -561,7 +561,7 @@ function elgg_get_views($dir, $base) {
  * @param $base
  */
 function get_views($dir, $base) {
-       elgg_log('get_views() was deprecated in 1.7 by elgg_get_views()!', 'WARNING');
+       elgg_deprecated_notice('get_views() was deprecated by elgg_get_views()!', 1.7);
        elgg_get_views($dir, $base);
 }
 
@@ -1112,7 +1112,7 @@ function elgg_extend_view($view, $view_name, $priority = 501, $viewtype = '') {
  * @param $viewtype
  */
 function extend_view($view, $view_name, $priority = 501, $viewtype = '') {
-       elgg_log('extend_view() was deprecated in 1.7 by elgg_extend_view()!', 'WARNING');
+       elgg_deprecated_notice('extend_view() was deprecated by elgg_extend_view()!', 1.7);
        elgg_extend_view($view, $view_name, $priority, $viewtype);
 }
 
@@ -2076,6 +2076,55 @@ function run_function_once($functionname, $timelastupdatedcheck = 0) {
        }
 }
 
+/**
+ * Sends a notice about deprecated use of a function, view, etc.
+ * Note: This will ALWAYS at least log a warning.  Don't use to pre-deprecate things.
+ * This assume we are releasing in order and deprecating according to policy.
+ *
+ * @param str $msg Message to log / display.
+ * @param str $version human-readable *release* version the function was deprecated. No bloody A, B, (R)C, or D.
+ *
+ * @return bool
+ */
+function elgg_deprecated_notice($msg, $dep_version) {
+       // if it's a major release behind, visual and logged
+       // if it's a 2 minor releases behind, visual and logged
+       // if it's 1 minor release behind, logged.
+       // bugfixes don't matter because you're not deprecating between the, RIGHT?
+
+       if (!$dep_version) {
+               return FALSE;
+       }
+
+       $elgg_version = get_version(TRUE);
+       $elgg_version_arr = explode('.', $elgg_version);
+       $elgg_major_version = $elgg_version_arr[0];
+       $elgg_minor_version = $elgg_version_arr[1];
+
+       $dep_version_arr = explode('.', $dep_version);
+       $dep_major_version = $dep_version_arr[0];
+       $dep_minor_version = $dep_version_arr[1];
+
+       $last_working_version = $dep_minor_version - 1;
+
+       $visual = FALSE;
+
+       // use version_compare to account for 1.7a < 1.7
+       if (($dep_major_version < $elgg_major_version)
+       || (($elgg_minor_version - $last_working_version) > 1)) {
+               $visual = TRUE;
+       }
+
+       $msg = "Deprecated in $dep_version: $msg";
+
+       elgg_log($msg, 'WARNING');
+
+       if ($visual) {
+               register_error($msg);
+       }
+
+       return TRUE;
+}
 
 
 /**

diff --git a/engine/lib/entities.php b/engine/lib/entities.php
index 2519400d6ad9504d932ca31fcf88c0422b912783..485a45cf1f8ed82c097ca0bac5ee822414ac5013 100644 (file)
--- a/engine/lib/entities.php
+++ b/engine/lib/entities.php
@@ -1841,8 +1841,7 @@ function elgg_get_entities(array $options = array()) {
  */
 function get_entities($type = "", $subtype = "", $owner_guid = 0, $order_by = "", $limit = 10, $offset = 0,
 $count = false, $site_guid = 0, $container_guid = null, $timelower = 0, $timeupper = 0) {
-
-       elgg_log('get_entities() was deprecated in 1.7 by elgg_get_entities()!', 'WARNING');
+       elgg_deprecated_notice('get_entities() was deprecated by elgg_get_entities().', 1.7);
        // rewrite owner_guid to container_guid to emulate old functionality
        $container_guid = $owner_guid;
        $owner_guid = NULL;
@@ -2215,7 +2214,7 @@ function elgg_list_entities($options) {
  * @return unknown_type
  */
 function list_entities($type= "", $subtype = "", $owner_guid = 0, $limit = 10, $fullview = true, $viewtypetoggle = false, $pagination = true) {
-       elgg_log('list_entities() was deprecated in 1.7.  Use elgg_list_entities()!', 'WARNING');
+       elgg_deprecated_notice('list_entities() was deprecated by elgg_list_entities()!', 1.7);
 
        $options = array();
 
@@ -2529,7 +2528,7 @@ function delete_entity($guid, $recursive = true) {
  * @param int $owner_guid The GUID of the owning user
  */
 function delete_entities($type = "", $subtype = "", $owner_guid = 0) {
-       elgg_log('delete_entities() was deprecated in 1.7 because no one should use it.');
+       elgg_deprecated_notice('delete_entities() was deprecated because no one should use it.', 1.7);
        return false;
 }
 
@@ -3051,7 +3050,7 @@ function entities_page_handler($page) {
  * @return unknown_type
  */
 function list_registered_entities($owner_guid = 0, $limit = 10, $fullview = true, $viewtypetoggle = false, $allowedtypes = true) {
-       elgg_log('list_registered_entities() was deprecated in 1.7 by elgg_list_registered_entities().', 'WARNING');
+       elgg_deprecated_notice('list_registered_entities() was deprecated by elgg_list_registered_entities().', 1.7);
 
        $options = array();
 

diff --git a/engine/lib/group.php b/engine/lib/group.php
index 3a6f3adf1f6c416fb7e613f6070792bbbbca6856..ab6a7d3beb160672dee7da418bbb0492592e04a5 100644 (file)
--- a/engine/lib/group.php
+++ b/engine/lib/group.php
@@ -48,7 +48,7 @@ class ElggGroup extends ElggEntity
                        }
                        // Is $guid is an ElggGroup? Use a copy constructor
                        else if ($guid instanceof ElggGroup) {
-                               elgg_log('This type of usage of the ElggGroup constructor was deprecated in 1.7. Please use the clone method.', 'WARNING');
+                               elgg_deprecated_notice('This type of usage of the ElggGroup constructor was deprecated. Please use the clone method.', 1.7);
                                
                                foreach ($guid->attributes as $key => $value) {
                                        $this->attributes[$key] = $value;
@@ -870,7 +870,7 @@ function add_group_tool_option($name,$label,$default_on=true) {
  * @deprecated 1.7
  */
 function search_for_group($criteria, $limit = 10, $offset = 0, $order_by = "", $count = false) {
-       elgg_log('search_for_group() was deprecated in 1.7.', 'WARNING');
+       elgg_deprecated_notice('search_for_group() was deprecated by new search plugin.', 1.7);
        global $CONFIG;
 
        $criteria = sanitise_string($criteria);
@@ -910,7 +910,7 @@ function search_for_group($criteria, $limit = 10, $offset = 0, $order_by = "", $
  * @deprecated 1.7
  */
 function search_list_groups_by_name($hook, $user, $returnvalue, $tag) {
-       elgg_log('search_list_groups_by_name() was deprecated in 1.7.', 'WARNING');
+       elgg_deprecated_notice('search_list_groups_by_name() was deprecated by new search plugin', 1.7);
        // Change this to set the number of groups that display on the search page
        $threshold = 4;
 
@@ -941,7 +941,7 @@ function search_list_groups_by_name($hook, $user, $returnvalue, $tag) {
  * @deprecated 1.7
  */
 function list_group_search($tag, $limit = 10) {
-       elgg_log('list_group_search() was deprecated in 1.7.', 'WARNING');
+       elgg_deprecated_notice('list_group_search() was deprecated by new search plugin.', 1.7);
        $offset = (int) get_input('offset');
        $limit = (int) $limit;
        $count = (int) search_for_group($tag, 10, 0, '', true);

diff --git a/engine/lib/metadata.php b/engine/lib/metadata.php
index ea86616a8b523103318bb5d7577ac53c1c484109..17835e07eb1fd8d095324a09635d5b7dbdcb582e 100644 (file)
--- a/engine/lib/metadata.php
+++ b/engine/lib/metadata.php
@@ -785,7 +785,7 @@ function get_entities_from_metadata($meta_name, $meta_value = "", $entity_type =
 $owner_guid = 0, $limit = 10, $offset = 0, $order_by = "", $site_guid = 0,
 $count = FALSE, $case_sensitive = TRUE) {
 
-       elgg_log('get_entities_from_metadata() was deprecated in 1.7 by elgg_get_entities_from_metadata()!', 'WARNING');
+       elgg_deprecated_notice('get_entities_from_metadata() was deprecated by elgg_get_entities_from_metadata()!', 1.7);
 
        $options = array();
 
@@ -875,7 +875,7 @@ function list_entities_from_metadata($meta_name, $meta_value = "", $entity_type
 function get_entities_from_metadata_multi($meta_array, $entity_type = "", $entity_subtype = "",
 $owner_guid = 0, $limit = 10, $offset = 0, $order_by = "", $site_guid = 0,
 $count = false, $meta_array_operator = 'and') {
-       elgg_log('get_entities_from_metadata_multi() was deprecated in 1.7 by elgg_get_entities_from_metadata()!', 'WARNING');
+       elgg_deprecated_notice('get_entities_from_metadata_multi() was deprecated by elgg_get_entities_from_metadata()!', 1.7);
 
        if (!is_array($meta_array) || sizeof($meta_array) == 0) {
                return false;

diff --git a/engine/lib/objects.php b/engine/lib/objects.php
index 8f49ea1faae56b935da2e9d14be3e3c07092f39f..331578ed674e88fad544ef6a916f0c1365a2bcca 100644 (file)
--- a/engine/lib/objects.php
+++ b/engine/lib/objects.php
@@ -54,7 +54,7 @@ class ElggObject extends ElggEntity {
 
                        // Is $guid is an ElggObject? Use a copy constructor
                        else if ($guid instanceof ElggObject) {
-                               elgg_log('This type of usage of the ElggObject constructor was deprecated in 1.7. Please use the clone method.', 'WARNING');
+                               elgg_deprecated_notice('This type of usage of the ElggObject constructor was deprecated. Please use the clone method.', 1.7);
                                
                                foreach ($guid->attributes as $key => $value) {
                                        $this->attributes[$key] = $value;
@@ -296,7 +296,7 @@ function delete_object_entity($guid) {
  * @deprecated 1.7
  */
 function search_for_object($criteria, $limit = 10, $offset = 0, $order_by = "", $count = false) {
-       elgg_log('search_for_object was deprecated in 1.7.', 'WARNING');
+       elgg_deprecated_notice('search_for_object() was deprecated by new search plugin.', 1.7);
        global $CONFIG;
 
        $criteria = sanitise_string($criteria);
@@ -361,7 +361,7 @@ function objects_test($hook, $type, $value, $params) {
  *
  */
 function search_list_objects_by_name($hook, $user, $returnvalue, $tag) {
-       elgg_log('search_list_objects_by_name was deprecated in 1.7.', 'WARNING');
+       elgg_deprecated_notice('search_list_objects_by_name was deprecated by new search plugin.', 1.7);
 
        // Change this to set the number of users that display on the search page
        $threshold = 4;

diff --git a/engine/lib/relationships.php b/engine/lib/relationships.php
index c1f54472ade3ce7ed1fc54e7bd9d4d7b6a842b05..979d3ea671182f92603b906a592fe90252034e22 100644 (file)
--- a/engine/lib/relationships.php
+++ b/engine/lib/relationships.php
@@ -598,7 +598,7 @@ function get_entities_from_relationship($relationship, $relationship_guid, $inve
 $type = "", $subtype = "", $owner_guid = 0, $order_by = "", $limit = 10, $offset = 0,
 $count = false, $site_guid = 0) {
 
-       elgg_log('get_entities_from_relationship() was deprecated in 1.7 by elgg_get_entities_from_relationship()!', 'WARNING');
+       elgg_deprecated_notice('get_entities_from_relationship() was deprecated by elgg_get_entities_from_relationship()!', 1.7);
 
        $options = array();
 
@@ -801,7 +801,7 @@ function list_entities_by_relationship_count($relationship, $inverse_relationshi
  * @return array|int|false An array of entities, or the number of entities, or false on failure
  */
 function get_entities_from_relationships_and_meta($relationship, $relationship_guid, $inverse_relationship = false, $meta_name = "", $meta_value = "", $type = "", $subtype = "", $owner_guid = 0, $limit = 10, $offset = 0, $count = false, $site_guid = 0) {
-       elgg_log('get_entities_from_relationship_and_meta() was deprecated in 1.7 by elgg_get_entities_from_relationship()!', 'WARNING');
+       elgg_deprecated_notice('get_entities_from_relationship_and_meta() was deprecated by elgg_get_entities_from_relationship()!', 1.7);
 
        $options = array();
 
@@ -1052,4 +1052,4 @@ register_plugin_hook("import", "all", "import_relationship_plugin_hook", 3);
 register_plugin_hook("export", "all", "export_relationship_plugin_hook", 3);
 
 /** Register event to listen to some events **/
-register_elgg_event_handler('create','friend','relationship_notification_hook');
\ No newline at end of file
+register_elgg_event_handler('create','friend','relationship_notification_hook');

diff --git a/engine/lib/sites.php b/engine/lib/sites.php
index c69032bd25888535495d041d6d4627f8a2b7537b..e9da4bdabf4dc7842e6503a2376744cc175c50d2 100644 (file)
--- a/engine/lib/sites.php
+++ b/engine/lib/sites.php
@@ -54,7 +54,7 @@ class ElggSite extends ElggEntity {
 
                        // Is $guid is an ElggSite? Use a copy constructor
                        else if ($guid instanceof ElggSite) {
-                               elgg_log('This type of usage of the ElggSite constructor was deprecated in 1.7. Please use the clone method.', 'WARNING');
+                               elgg_deprecated_notice('This type of usage of the ElggSite constructor was deprecated. Please use the clone method.', 1.7);
                                
                                foreach ($guid->attributes as $key => $value) {
                                        $this->attributes[$key] = $value;
@@ -504,7 +504,7 @@ function get_site_by_url($url) {
  * @deprecated 1.7
  */
 function search_for_site($criteria, $limit = 10, $offset = 0, $order_by = "", $count = false) {
-       elgg_log('search_for_site() was deprecated in 1.7.', 'WARNING');
+       elgg_deprecated_notice('search_for_site() was deprecated by new search plugin.', 1.7);
        global $CONFIG;
 
        $criteria = sanitise_string($criteria);

diff --git a/engine/lib/users.php b/engine/lib/users.php
index 5e5486133213b24c8d3bbd55a6b5d1d581dcd152..3f866754f4bd1637f465c7e7aeb7cf22cde049f1 100644 (file)
--- a/engine/lib/users.php
+++ b/engine/lib/users.php
@@ -75,7 +75,7 @@ class ElggUser extends ElggEntity
 
                        // Is $guid is an ElggUser? Use a copy constructor
                        else if ($guid instanceof ElggUser) {
-                               elgg_log('This type of usage of the ElggUser constructor was deprecated in 1.7. Please use the clone method.', 'WARNING');
+                               elgg_deprecated_notice('This type of usage of the ElggUser constructor was deprecated. Please use the clone method.', 1.7);
 
                                foreach ($guid->attributes as $key => $value) {
                                        $this->attributes[$key] = $value;
@@ -872,7 +872,7 @@ function get_user_by_email($email) {
  * @deprecated 1.7
  */
 function search_for_user($criteria, $limit = 10, $offset = 0, $order_by = "", $count = false) {
-       elgg_log('search_for_user() was deprecated in 1.7.', 'WARNING');
+       elgg_deprecated_notice('search_for_user() was deprecated by new search.', 1.7);
        global $CONFIG;
 
        $criteria = sanitise_string($criteria);
@@ -918,7 +918,7 @@ function search_for_user($criteria, $limit = 10, $offset = 0, $order_by = "", $c
  * @deprecated 1.7
  */
 function list_user_search($tag, $limit = 10) {
-       elgg_log('list_user_search() deprecated in 1.7', 'WARNING');
+       elgg_deprecated_notice('list_user_search() deprecated by new search', 1.7);
        $offset = (int) get_input('offset');
        $limit = (int) $limit;
        $count = (int) search_for_user($tag, 10, 0, '', true);
@@ -1547,7 +1547,7 @@ function users_init() {
  * @deprecated 1.7
  */
 function search_list_users_by_name($hook, $user, $returnvalue, $tag) {
-       elgg_log('search_list_users_by_name() was deprecated in 1.7', 'WARNING');
+       elgg_deprecated_notice('search_list_users_by_name() was deprecated by new search', 1.7);
        // Change this to set the number of users that display on the search page
        $threshold = 4;
 

diff --git a/views/default/group/search/finishblurb.php b/views/default/group/search/finishblurb.php
index a030fbac5d12c132a8599e8304bc60032002fd90..9d3759cb31a42c586ce541ae5fb7f782fada3ced 100644 (file)
--- a/views/default/group/search/finishblurb.php
+++ b/views/default/group/search/finishblurb.php
@@ -6,7 +6,7 @@
  * @link http://elgg.org/\r
  * @deprecated 1.7\r
  */\r
-elgg_log('view groups/search/finishblurb was deprecated in 1.7', 'WARNING');\r
+elgg_deprecated_notice('view groups/search/finishblurb was deprecated.', 1.7);\r
 \r
 if ($vars['count'] > $vars['threshold']) {\r
 \r

diff --git a/views/default/group/search/startblurb.php b/views/default/group/search/startblurb.php
index 64425af69cb80cac9a44b3132436f54f6da08a23..f3acfaf5a847061f76d33467b3d8f45b5310ff2f 100644 (file)
--- a/views/default/group/search/startblurb.php
+++ b/views/default/group/search/startblurb.php
@@ -6,7 +6,7 @@
  * @link http://elgg.org/\r
  * @deprecated 1.7\r
  */\r
-elgg_log('view groups/search/startblurb was deprecated in 1.7', 'WARNING');\r
+elgg_deprecated_notice('view groups/search/startblurb was deprecated.', 1.7);\r
 ?>\r
 \r
 <div class="contentWrapper">\r

diff --git a/views/default/user/search/finishblurb.php b/views/default/user/search/finishblurb.php
index 87ab0b60db914314141c566e06af17504a5e316c..a8c14f3579b61a25327036da01bac5bc6976a2fa 100644 (file)
--- a/views/default/user/search/finishblurb.php
+++ b/views/default/user/search/finishblurb.php
@@ -7,7 +7,7 @@
  * @deprecated 1.7\r
  */\r
 \r
-elgg_log('view user/search/finishblurb was deprecated in 1.7', 'WARNING');\r
+elgg_deprecated_notice('view user/search/finishblurb was deprecated.', 1.7);\r
 \r
 if ($vars['count'] > $vars['threshold']) {\r
 \r

diff --git a/views/default/user/search/startblurb.php b/views/default/user/search/startblurb.php
index 2866ba9ad5a2532c45492c5bb1220b1326f4fb46..e069ec460c80fc05d5daca731a69886bc34df6e5 100644 (file)
--- a/views/default/user/search/startblurb.php
+++ b/views/default/user/search/startblurb.php
@@ -7,7 +7,7 @@
  * @link http://elgg.org/\r
  */\r
 \r
-elgg_log('view user/search/startblurb was deprecated in 1.7', 'WARNING');\r
+elgg_deprecated_notice('view user/search/startblurb was deprecated.', 1.7);\r
 \r
 ?>\r
 <div class="contentWrapper">\r