Closed (fixed)
Project:
Ajaxify Drupal with JQuery Ajax
Version:
7.x-1.6
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
12 Dec 2012 at 02:47 UTC
Updated:
24 Dec 2012 at 06:18 UTC
Hi guys... This module is great... Just "install and use" :)
I have an advice/observation:
In function l_ajax that actually looks like this:
function l_ajax($title, $path, $target) {
global $base_url;
if ($target == NULL) {
$target = variable_get('ajax_links_api_selector', '.region-content');
}
return theme('ajax_links_api_link', array(
'title' => $title,
'path' => $base_url . '/' . $path,
'target' => $target,
));
}
I tried to replace this line:
//...Rest of code
'path' => $base_url . '/' . $path,
//...Rest of code
For this one:
//...Rest of code
'path' => url($path, array('absolute' => TRUE)),
//...Rest of code
And that works fine if clean URL's are enabled or not...This would be great for some servers where this feature is not available, and would be "good" to have it "out-of-the-box".
If I'm wrong, please correct me :)
Comments
Comment #1
waspper commentedAlso:
Using url(), we could use '<front>', and... I think so... Other "special strings"...
Comment #2
serjas commented@waspper .. Thanks for the suggestion.. Will check this
Comment #3
serjas commentedcommitted to latest dev branch. Thanks waspper. Now module is not depending on cleanurl :)
Comment #4
serjas commented