Just tried out the new 6.x-2.0 version and would like to thank you for this great module.

However I found that uc_aac.js makes its POST requests to the wrong URL, because it tries to determine the site's $base_path from an undefined variable.

In uc_aac.js at line 6 it has: 

    url: Drupal.settings.base_path + "?q=uc_aac",

According to the Drupal.settings array it should be:

    url: Drupal.settings.basePath + "?q=uc_aac",

Interestingly it works with the original code on Apache with clean urls enabled, but it doesn't work at all on Lighttpd using mod_magnet and the well-known lua script.

Comments

jantoine’s picture

jantoine commented
Version: 6.x-2.0 » 6.x-2.x-dev
Status: Active » Fixed

Melon,

Thanks for reporting this issue. The fix has been committed.

Cheers,

Antoine

Status: Fixed » Closed (fixed)

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

hanoii’s picture

hanoii
Primary language Spanish
Location 🇦🇷UTC-3
commented
Status: Closed (fixed) » Active

Sorry to bring this one back to life, but I looked at the -dev version and this is not fixed. I checked on the cvs logs and it was committed to the HEAD, not to the DRUPAL-6--2 branch. Unless you are using head for something else (like for drupal 7 or a future 3.x branch), I'd recommend you to develop only on the 2.x branch, you'll always will have the chance to merge all those changes easily via CVS into HEAD whenever you need to, and in the meantime, you are free from committing the fixes for both HEAD and DRUPAL-6--2 as it seems you are currently doing.

jantoine’s picture

jantoine commented
Status: Active » Fixed

Hanoii,

Thanks for catching this. I have applied the fix to the 6.x-2.x branch.

Cheers,

Status: Fixed » Closed (fixed)

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