hook_views_default_view inconsistency - if $views[name] differs from $views[name]->name, pages return 404

I installed some default views with hook_views_default_view, and put the views files in

.../views/filename.view<code>. I then used this hook to bring them in:

<?php
function test_views_default_views() {
  $files = file_scan_directory(drupal_get_path('module', 'test'). '/views', '.view');
  foreach ($files as $absolute => $file) {
    require $absolute;
    if (isset($view)) {
      $views[$file->name] = $view;
    }
  }
  return $views;
}
?>

This seemed to work, but I had one view which I couldn't edit, export or clone: all the pages returned a 404. I could disable it, though, which confused me greatly.

Anyway, I think I've tracked this down to the %views_ui_cache loader: views_ui_cache_load > views_get_view > views_get_default_view > views_discover_default_views . The essential problem is that thew views cache returns a merged version of all the key/value arrays from, but each value i.e. <code>$view

preserves its name attribute. Some aspects of Views use the key; others use $view->name.

That means that the following causes some pages to fail and some to succeed:

function test_views_default_views() {
  $views[foo] = new $view;
  $views[foo]->name = "bar";
  /* ... */
  return $views;
}

That means that hook_views_default_view requires an array of views, where each array key/value pair must have key == $value->name, or 404s start appearing. This isn't documented as far as I can see and there's no warning when you do this: the views happily appear at admin/build/view ; they just fail to appear at /clone , /edit or /export

Can anyone reproduce this? If so, can we have some sort of warning when this array is inconsistent according to what views.module needs?