First, thanks for your great efforts on the relation module.

I'm not sure whether to report this here or at the admin_views module. When enabling admin_views (http://drupal.org/project/admin_views) and trying to access

  • Find content > Relations

a fatal error occurs:

Fatal error: require_once() [function.require]: Failed opening required '/usr/www/public_html/sites/all/modules/relation/node.admin.inc' (include_path='.:/usr/local/lib/php/') in /usr/www/public_html/includes/menu.inc on line 514

After disabling the admin_views module everything works again.

CommentFileSizeAuthor
#7 relation-admin-views-breaks-relation-tab-1739332-7.patch489 bytessteveoliver

Comments

danepowell’s picture

danepowell commented
Priority: Minor » Normal

Same issue here. Know of any workarounds?

danepowell’s picture

danepowell commented
Status: Active » Postponed

This was supposed to be fixed in admin_views, but I'm still seeing the error with the admin_views dev release: #1673820: Fatal error when there's a views page with admin/content/xxx path

imclean’s picture

imclean commented

I also had this error at one point but it seemed to go away when I fixed another. Make sure you're using the latest dev of admin_views and try this Views patch: #1771140: view_menu_alter() adding to existing "access arguments" causes user_access() error

imclean’s picture

imclean commented

I spoke too soon, still getting the "Failed opening /sites/all/modules/relation/node.admin.inc" error.

imclean’s picture

imclean commented

Until admin_views is fixed, you can get around it by adding a 'file' value in relation_ui.module.

  // Relations listing.
  $items['admin/content/relation'] = array(
    'title' => 'Relations',
    'file' => 'relation_ui.module',    
    'page callback' => 'relation_ui_admin_content',
    'access arguments' => array('administer relations'),
    'description' => 'View, edit and delete all the available relations on your site.',
    'type' => MENU_LOCAL_TASK,
  );

Make sure you clear the cache afterwards.

sthzg’s picture

sthzg commented

#5 worked for me as a simple workaround.

steveoliver’s picture

steveoliver commented
Status: Postponed » Needs review
StatusFileSize
new relation-admin-views-breaks-relation-tab-1739332-7.patch489 bytes

I don't see any reason not to include the 'file' property. Nor do I imagine testbot could have anything to say about this, but let's see...

imclean’s picture

imclean commented

Sounds good steveolive, and judging by the admin_views issue there's nothing there to fix so this would be the way forward.

steveoliver’s picture

steveoliver commented
Status: Needs review » Closed (fixed)

Thanks, sthzg and imclean. Committed to 7.x-1.x (f227a48).