Hi,

There is an issue with the JS of supernav_page_left. The Ajax URL's are not processed through url() therefore when clean URLs are off, the Ajax fails. I did a quick patch, but did not check if there were any more cases such as. Here is the patch:

I don't know if the patch format is right or not, anyhow, here it is.

 /**
  * @file
@@ -618,7 +618,7 @@
         ". $js_content_vars ."
         slide_search('up');
         $.ajax({
-            url: 'supernav_search',
+            url: '". check_url(url('supernav_search')) ."',
             type: 'POST', 
             data: 'search=' + search + '&search_nav=' + search_nav + '&search_pages=' + search_pages". $js_content_query_string .",
             error: function(){
@@ -648,7 +648,7 @@
         ". $loading_div_js ."
         var bm_title =  $('.supernav-bookmark-input').val();
         $.ajax({
-            url: 'supernav_bookmarks',
+            url: '". check_url(url('supernav_bookmarks')) ."',
             type: 'POST', 
             data: 'url=' + window.parent.supernav_right.location.href + '&title=' + bm_title + '&action=add',
             error: function(){
@@ -669,7 +669,7 @@
         function supernav_bookmark_move(type,bmid) {
           ". $loading_div_js ."
           $.ajax({
-              url: 'supernav_bookmarks',
+              url: '". check_url(url('supernav_bookmarks')) ."',
               type: 'POST', 
               data: 'action=' + type + '&bmid=' + bmid,
               error: function(){

Comments

chrisshattuck’s picture

Status: Needs review » Fixed

Hello joseguevara,

Thanks for the patch, I have applied it to the Drupal 5 dev version. For some reason, I don't get the errors with the clean URLs enabled on my testing site, so I appreciate you pitching in. :)

Thanks!
Chris

jose.guevara’s picture

Chris,

You're welcome. The error was happening when the clean URLs were off, plus my dev sites are in sub-directories.

JG

Anonymous’s picture

Status: Fixed » Closed (fixed)

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