There is link to Edit Flash Video under admin/settings/flashvideo/edit
But when I click on that link I simply get a blank screen and Drupal renders nothing.

CommentFileSizeAuthor
#11 flashvideo-6.x-1.4-rc2-menu.patch1.18 KBattheshow

Comments

vibrantsky’s picture

I have the same problem, i dont even know if there is supposed to be anything there?

running on Windows (was tough to get up).

ionuts71’s picture

Nobody knows?

MaffooClock’s picture

This is still an issue.

aznboy’s picture

i have it working with windows and that problem does not happen.

You also did not provide enough information.

What does you logs say. Is it perhaps you haven't given php enough memory to run.

chrisfromredfin’s picture

I am seeing this same behavior, and nothing is written to my error.log. As it turns out, it's not a "bug" per se -- it's just that the menu path is not meant to be called without another parameter. That is, admin/settings/flashvideo/edit/NODE-TYPE (or admin/settings/flashvideo/edit/global). Travist, I think an easy way to deal with it is to add an else condition for if ($node_type), that does looks like:

else {
  drupal_goto('admin/settings/flashvideo/edit/global');
}

But that might not be best practice. I'm sure there's a way to remove just the /edit from the menu should you need to, or "require" an argument.

Let me guess (other people who've reported this) -- you can see it in your black bar at the top (admin_menu module) but you don't see it if you use the regular administration menu in your sidebar (or whatever region you have it in)? It's just the way admin_menu is building out the menus. The good news is there's no error and there's nothing you're missing in your configuration.

travist’s picture

What about doing this?....

else {
   drupal_goto('admin/settings/flashvideo');
}

really, people shouldn't be going to the edit without a node type, so I really don't see any other way around this...

chrisfromredfin’s picture

Either way is fine - either redirect them to the menu to choose one, or redirect them to global. I don't have a compelling reason one way or the other, and you're the developer, so you get to choose. :)

MGN’s picture

This is still present in 6.x-1.4. The fix in #6 is correct, but I would also recommend setting the type of menu item to MENU_CALLBACK as

    $items['admin/settings/flashvideo/edit'] = array(
      'title' => 'Edit FlashVideo',
      'page callback' => 'flashvideo_settings_main',
      'page arguments' => array(4),
      'type' => MENU_CALLBACK,  // ADD THIS
      'access callback' => 'user_access',
      'access arguments' => array('administer flashvideo'),
      'parent' => 'admin/settings/flashvideo',
      'file' => 'flashvideo.admin.inc'
   );

This will eliminate the useless submenu that is currently appearing in the admin menu.

Hope this helps.

travist’s picture

MGN,

That is the best solution for this.... I just committed that and it will be added to the next release. Thanks for your efforts.

travist’s picture

Status: Active » Needs review
attheshow’s picture

Version: 6.x-1.3 » 6.x-1.4-rc2
StatusFileSize
new1.18 KB

I tried removing the 'parent' parameter from the menu call and that fixed this issue for me, without hurting access to the normal edit pages. It's a super small change, but I've attached a patch to this issue. The patch is for the latest release as of today (6.x-1.4-rc2).

travist’s picture

attheshow....

Thank you so much for this. This has been confusing me as to why it wasn't working. Just committed this. Will be in version 1.4.

Thanks!!

Travis.

travist’s picture

Status: Needs review » Fixed

Fixed in version 1.4! Thanks everyone for the efforts!

Status: Fixed » Closed (fixed)

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