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

waspper’s picture

Also:
Using url(), we could use '<front>', and... I think so... Other "special strings"...

serjas’s picture

Status: Active » Needs work

@waspper .. Thanks for the suggestion.. Will check this

serjas’s picture

Status: Needs work » Needs review

committed to latest dev branch. Thanks waspper. Now module is not depending on cleanurl :)

serjas’s picture

Status: Needs review » Closed (fixed)