From 6729a2d0d2faeb07ca8e79b5162c3be269e01b66 Mon Sep 17 00:00:00 2001 From: brettp Date: Sat, 16 Apr 2011 22:10:52 +0000 Subject: Fixes #3310. Applied lie2815's patch to use activate.php and deactivate.php instead of manifest on_de/activate fields in plugins. git-svn-id: http://code.elgg.org/elgg/trunk@9002 36083f99-b078-4883-b0ff-0f9b5a30f544 --- mod/blog/manifest.xml | 1 - mod/blog/start.php | 15 ++++----------- mod/categories/manifest.xml | 2 -- mod/categories/start.php | 21 --------------------- 4 files changed, 4 insertions(+), 35 deletions(-) (limited to 'mod') diff --git a/mod/blog/manifest.xml b/mod/blog/manifest.xml index 4d853a9aa..f0ec96e8b 100644 --- a/mod/blog/manifest.xml +++ b/mod/blog/manifest.xml @@ -16,5 +16,4 @@ simple true - blog_on_activate diff --git a/mod/blog/start.php b/mod/blog/start.php index 7d42e80b9..33c6ee20d 100644 --- a/mod/blog/start.php +++ b/mod/blog/start.php @@ -18,7 +18,7 @@ elgg_register_event_handler('init', 'system', 'blog_init'); * Init blog plugin. */ function blog_init() { - + elgg_register_library('elgg:blog', elgg_get_plugins_path() . 'blog/lib/blog.php'); // add a site navigation item @@ -101,14 +101,14 @@ function blog_page_handler($page) { // @todo remove the forwarder in 1.9 // forward to correct URL for bookmarks pre-1.7.5 blog_url_forwarder($page); - + // push all blogs breadcrumb elgg_push_breadcrumb(elgg_echo('blog:blogs'), "blog/all"); if (!isset($page[0])) { $page[0] = 'all'; } - + $page_type = $page[0]; switch ($page_type) { case 'owner': @@ -143,7 +143,7 @@ function blog_page_handler($page) { $params = blog_get_page_content_list(); break; } - + $params['sidebar'] .= elgg_view('blog/sidebar', array('page' => $page_type)); $body = elgg_view_layout('content', $params); @@ -224,13 +224,6 @@ function blog_ecml_views_hook($hook, $entity_type, $return_value, $params) { return $return_value; } -/** - * Runs when blog plugin is activated. See manifest file. - */ -function blog_on_activate() { - add_subtype('object', 'blog', 'ElggBlog'); -} - /** * When upgrading, check if the ElggBlog class has been registered as this * was added in Elgg 1.8 diff --git a/mod/categories/manifest.xml b/mod/categories/manifest.xml index f02d7a8cb..43df13d02 100644 --- a/mod/categories/manifest.xml +++ b/mod/categories/manifest.xml @@ -14,6 +14,4 @@ 2010030101 advanced - categories_on_activate - categories_on_deactivate diff --git a/mod/categories/start.php b/mod/categories/start.php index 190996d9b..ff5b01efc 100644 --- a/mod/categories/start.php +++ b/mod/categories/start.php @@ -54,24 +54,3 @@ function categories_save($event, $object_type, $object) { } return TRUE; } - -/** - * Add a reminder to set default categories. - */ -function categories_on_activate() { - $site = elgg_get_site_entity(); - - if (!$site->categories) { - $url = elgg_normalize_url('admin/plugin_settings/categories'); - $message = elgg_echo('categories:on_enable_reminder', array($url)); - elgg_add_admin_notice('categories_admin_notice_no_categories', $message); - } - return TRUE; -} - -/** - * Clean up admin notices on disable. - */ -function categories_on_deactivate() { - elgg_delete_admin_notice('categories_admin_notice_no_categories'); -} -- cgit v1.2.3