The mouseover URL is correct but when you click to email a story the URL becomes
URL q=emailpage%26nid%3D6

The URL should be q=emailpage&nid=6

CommentFileSizeAuthor
#6 emailpage.diff.txt1.75 KBdrumm

Comments

lanesharon’s picture

Google is still 'seeing' these nodes in this format, but I am using clean url's. So, mine show up as:
http://www.gates-chili-high.org/emailpage&nid%3D97

In my error logs, as I mentioned here already. Why Google can read them, I have no idea. When I am not signed in, I do not get the emailpage link, so how can Google?

This has me baffled!

millerrd’s picture

Here's some fixes for the url problem. in emailpage.module update the following lines:
line 50:
$links[] = l(t('email this %type', array('%type' => t($e_l_t))), "emailpage", array('title' => EMAIL_PAGE_TITLE), "nid=$node->nid");

line 54:
$links[] = l(t("email this ".$e_l_t), "emailpage", array("title" => EMAILPAGE_TITLE), "nid=$node->nid"); return $links;

seaneffel’s picture

This fixed my problem, has it been submitted for permanent fix yet?

dewolfe001’s picture

Closer, but I found a bug in the URL is gave up.

When I tried to use the emailpage it gave me a gimped up URL:

http://example.com/drupal//drupal/?q=en/node/19

when I really wanted:

http://example.com/drupal/?q=en/node/19

so, in emailpage,module on lines 76 and 78, I changed it from this:

$base_url .'/'.

to this:

$host['scheme']."://".$host['host'].

That seemed to patch it up so that it works now.

casperl’s picture

This is the same bug described in http://drupal.org/node/89028

drumm’s picture

Status: Active » Needs review
StatusFileSize
new1.75 KB

Here is a patch to fix this issue. The problem was improperly using the l() function.

darren oh’s picture

Title: Faulty URL q=emailpage%26nid%3D6 » Faulty URL results in page not found
davemybes’s picture

Patch works perfectly for me. Thanks for that.

darren oh’s picture

Status: Needs review » Closed (fixed)

This patch is not relevant to the latest release.