hi,
Thanks for the great module. it is fantastic, i have been testing this module before deploying on the live site. i created the jobs, then i let a another user apply for the job. when i receive the email. which has links to them. but when i click on links, i get page not found because the Url being provided in the email is wrong.

instead of this url

http://www.samtesting.hucknalllocal.com/samwiri/local-jobs/security-jobs/jobs/huck
i get the following url, with repeated extension

http://www.samtesting.hucknalllocal.com/samwiri/samwiri/local-jobs/security-jobs/jobs/huck

i hope to here from you or any drupler with a solution, have a look at the snapshot bellow

CommentFileSizeAuthor
#3 email_path_fix.patch2.72 KBmandclu
snapshot_of_problem.png28.9 KBsamkat12

Comments

Peng.Pif’s picture

Peng.Pif commented

I have the same problem. I think it may be something in the job_mail fucntion causing it to happen? I will look into it and see if I can spot anything. I will post back if I can find a fix.

samkat12’s picture

samkat12 commented

Thanks, I await your feedback.

mandclu’s picture

mandclu commented
StatusFileSize
new email_path_fix.patch2.72 KB

Did either of you have a manual entry for $base_url in your sites? In any case, the attached patch revises the module's url processing to rely more fully Drupal core's url() function. Let me know if it solves your issues.

kbahey’s picture

kbahey commented

Martin

An alternate way is to use:

$base_url . base_path() . blah

Can't check now if url() does that too.

mandclu’s picture

mandclu commented

True, but ultimately that approach feels like it's working against core, or at least outside of it. This should give you the same result:

url($path, array("absolute" => true))

But I believe it also checks if the incoming path is absolute to begin with, etc. That was part of the issue before, and I believe there may have been an issue if Drupal wasn't installed in the root directory that the extra directory was being added twice: once as part of $base_url and once by url().

Peng.Pif’s picture

Peng.Pif commented

Hi Surge, thanks for the patch. I will test it and let you know how I get on.

The only thing I did change was the 'drupal' folders name to 'sitename' when I uploaded it to the web server.

rafinskipg’s picture

rafinskipg commented

Anyone checked the solution??