- Patch #26139 by webchick / Kieran / documentation team: improved admin help of core modules! /

1 files changed, 12 insertions, 9 deletions

diff --git a/modules/ping.module b/modules/ping.module
index f8006346e..d43138378 100644
--- a/modules/ping.module
+++ b/modules/ping.module
@@ -12,17 +12,20 @@
 function ping_help($section) {
   switch ($section) {
     case 'admin/help#ping':
-      $output .= t("
-      <p>Drupal can automatically send notifications (called \"pings\") to the %pingomatic to tell them that your site has changed.  In turn pingomatic.com will ping other services like weblogs.com, Technorati, blo.gs, BlogRolling, Feedster.com, Moreover, etc.</p>
-      <p>The ping feature requires crontab.</p>", array('%pingomatic' => '<a href="http://pingomatic.com/">http://pingomatic.com/</a>'));
-      break;
-
+      $output = '<p>'. t('The ping module is useful for notifying interested sites that your site has changed.  It automatically sends notifications (called "pings") to the <a href="%external-http-pingomatic-com">pingomatic</a> service to tell it that your site has changed. In turn pingomatic will ping other services such as weblogs.com, Technorati, blo.gs, BlogRolling, Feedster.com, Moreover, etc.', array('%external-http-pingomatic-com' => 'http://pingomatic.com/')) .'</p>';
+      $output .= '<p>'. t('The ping module requires <code>cron</code> or a similar periodic job scheduler to be enabled.') .'</p>';
+      $output .= t('<p>You can:</p>
+<ul>
+<li> enable or disable the ping module at <a href="%admin-modules">administer &gt;&gt; modules</a>.</li>
+<li>run your cron job at your sites <a href="%file-cron">cron page</a>.</li>
+<li>read about <a href="%external-http-drupal-org-node-23714">configuring cron jobs</a>.</li>
+</ul></p>
+', array('%admin-modules' => url('admin/modules'), '%file-cron' => 'cron.php', '%external-http-drupal-org-node-23714' => 'http://drupal.org/node/23714'));
+      $output .= '<p>'. t('For more information please read the configuration and customization handbook <a href="%ping">Ping page</a>.', array('%ping' => 'http://www.drupal.org/handbook/modules/ping/')) .'</p>';
+      return $output;
     case 'admin/modules#description':
-      $output = t('Alerts other sites when your site has been updated.');
-      break;
+      return('Alerts other sites when your site has been updated.');
   }
-
-  return $output;
 }
 
 /**