Om-maximenu version 6.x-1.43 issue

can you show me the paths of your links?

Hi, currently, I am not sure if all the third party effects support ie6, it will also be hard support ie6 if they do not.

Hi.

I also have this issue in a 7.x-dev version.

The message appears in all pages of the site, for authenticated users, or not, and is:

Notice: Undefined offset: 1 in om_path_query() (line 575 of /home/mysite/public_html/sites/all/modules/om_maximenu/inc/om_maximenu.utils.inc).

Curiously, the style in use also is Dropdown.

I could see that this problem started after I add more than three links in Maximenu (while there were only three links, was working perfectly).

Any ideas?

Tks.

Edit: This occurs in any browser

This is a excellent module but I can not use it on my site because of this bug, which was also reported by another user.

If not work as desired, the error is critical

In the line 575 of the file om_maximenu.utils.inc, we have the following code:

if (!empty($query)) $out[$query[0]] = $query[1];

Anything is wrong. Can anyone help us?

Tks.

Confirming this is also an issue for me.
I think it has something to do with invalid input in the 'path query' field. When I put 'path-to-node' it creates this error, but when I put 'node=path-to-node' it does not.

astutonet, can you check your fourth menu item, and see what you've put in the 'path query' field. Try emptying those fields, and see what you get. If the error disappears, we know where the problem is.

Just to clarify, that line of code seems to be exploding the 'path query' value, and storing the results as an array. When you don't have the '=' in there, there is nothing in the second array, and hence the error.

If line 575 was changed from if (!empty($query)) $out[$query[0]] = $query[1]; to if (!isset($query[1])) $out[$query[0]] = $query[1]; then this should fix the issue for other users.

The downfall of this approach is that it won't be obvious to the user that something has failed. It would be much better to have something there along these lines:

 if ((isset($query[0]) && (!isset($query[1] ) {
  // create validation error, with message saying path query is invalid
 } else {
  $out[$query[0]] = $query[1];
}

ok, can you try this:

<?php
/**
 * Path queries 
 *
 * On D7 $option['query'] becomes purely an array
 * cannot accept anymore a string
 *
 */
function om_path_query($path_query = array()) { 
  //dsm($path_query);       
  $out = array();
  $queries = explode('&', $path_query);

  if (!empty($queries)) {
    foreach ($queries as $kquery => $query) {
      if (!empty($query)) {
        $query = explode('=', $query);
        if (!empty($query) && isset($query[0]) && isset($query[1])) {
          $out[$query[0]] = $query[1];
        }
        else {
          drupal_set_message(t('Invalid input.'));
        }  
      }
    }
  }
  else {
    $query = explode('=', $path_query);
    if (!empty($query) && isset($query[0]) && isset($query[1])) {
      $out[$query[0]] = $query[1];
    }  
    else {
      drupal_set_message(t('Invalid input.'));
    }  
  }
  return $out;
} 

?>

Pat Redmond, the field in error had a friendly url, wich was entered accidentally. It was enough to remove the url and the error disappeared.

Have you tested the code above?

danielhonrade, the problem with that code is that it lets you save the invalid path entry, and then displays the error on every page...
I think you need to implement the code where the save occurs.

astutonet, I've tested my code, and it removes the error. Unfortunately it allows you to save the path, and then it just ignores it.

I am using Om-maximenu version 6.x-1.43, in this version only having style = dropdown option, in later version dont have, what's the problem is, if i enable version 6.x-1.43 om maximenu then the
backup & migrate and Payment module is crashed,else if i enable later version 6.x-1.24 no other module is crashed, in this version 6.x-1.24 of om maximenu dont have the option style = dropdown then only other module is working fine, if other version having style option dropdown so its affect other module, Please provide me the solution, i need to enable style= drop down option simultaneously backup & migrate and payment should work. thanks in advance

you just have to resave all you menus by clicking save in the Om Maximenu admin ui