support for multiple views with similar names

Hi,

Nice module, however i believe i have found a bug/oversight in the code.

In the hook_menu on line 69 there is the following code:

$urls = views_get_all_urls();
      foreach ($urls as $key => $url) {
        if ($url && strpos($_GET['q'], $url) !== FALSE) {
          $view_name = $key;
          break;
        }
      }

as far as i can tell this is just searching for the existence of the current url ($_GET['q']) within any of your views.

I ran into a problem, however, because i had some views with similar urls.
eg

I had a pre-existing view at /tracker.

I went to add another view is at /growth-tracker.

However the above code found the view at /tracker before it found my new one and as a result my new view just didnt work: I couldn't go to /operations to set an operation for vbo and the view just returned nothing.

I propose that you should replace the previous code with the following

else {
      $current_url = $_GET['q'];
      
      	// See if the current url ends in /operations; if so, strip it off so we can see if that matches the path
      if (substr($current_url, -11) == '/operations') {
      	$current_url = substr($current_url, 0, (strlen($current_url) - 11));
      }
      
      $views_urls = views_menu(false); // get the non-cached views menu handlers
      foreach ($views_urls as $url) {
      	$view_path = $url['path'];
      	if (strpos($current_url . '/view', $url['path']) === 0) {
      		$view_name = $url['callback arguments'][0];
      		$path = $current_url . '/operations';
      		break;
      	}
      }

rather than searching all the views urls on your intsall, this looks at the current views menu handlers to retrieve the view's url.

Patch is attached

What do you think?

Josh