I know, that the Pingback 1.0 specification do not claim to call the pingback server in the http-header and as link element. But there are other blog-softwares which only look for the link element. For those software it would be nice, if this modul would call the pingback server in the http-header and as a link element.

All it needs is this small patch for pingback.modul:

147d146
<           drupal_set_html_head('<link rel="pingback" href="' . $GLOBALS['base_url'] . '/xmlrpc.php">');
484c483
< }
---
> }
CommentFileSizeAuthor
#4 844974-pinback-link.patch774 bytesdave reid
#3 addHtmlHeadPingbackServer.patch535 bytesAnonymous (not verified)

Comments

andreashaugstrup’s picture

Please create patches in the unified format. See more: http://drupal.org/patch/create

Anonymous’s picture

np

--- pingback.module	2008-03-15 01:30:32.000000000 +0100
+++ pingbackNew.module	2010-07-08 08:23:48.027899568 +0200
@@ -144,6 +144,7 @@ function pingback_nodeapi(&$node, $op, $
         // Insert pingback header when a node is being viewed.
         if (arg(0) == 'node' && is_numeric(arg(1)) && arg(3) == NULL) {
           drupal_set_header('X-Pingback: '. $GLOBALS['base_url'] .'/xmlrpc.php');
+          drupal_set_html_head('<link rel="pingback" href="' . $GLOBALS['base_url'] . '/xmlrpc.php">');
         }
         break;
     }
Anonymous’s picture

StatusFileSize
new535 bytes
dave reid’s picture

Status: Active » Needs review
StatusFileSize
new774 bytes

Here's the proper way to do it...

dave reid’s picture

Status: Needs review » Fixed

Committed #4 to 6.x-1.x: http://drupal.org/cvs?commit=413314

dave reid’s picture

Also committed to HEAD: http://drupal.org/cvs?commit=413320

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.