Missing argument 1 for backup_migrate_crud_ui_edit() in /sites/all/modules/backup_migrate/includes/crud.inc [#689080]

Use case only occurs with admin_menu 6.x-1.5 module active with profiles applied.
I did not test this with the dev version because of http://drupal.org/node/689044.
The admin_menu module gives me some extra superfish menu like interface possibilities to navigate the backup_migrate ui. Some links target functions without supplying an argument (profile name). This results in errors. These errors are:

Backup and Migrate > Profiles > List Profiles > Delete Profile
Error message:
Fatal error: Call to a member function delete_confirm_message() on a non-object in /sites/all/modules/backup_migrate/includes/crud.inc on line 147
Backup and Migrate > Profiles > List Profiles > Edit Profile
Silent failure, warning message in log:
Missing argument 1 for backup_migrate_crud_ui_edit() in /sites/all/modules/backup_migrate/includes/crud.inc
Backup and Migrate > Profiles > List Profiles > Export Profile
Silent failure, warning message in log:
Missing argument 1 for backup_migrate_crud_ui_export() in /sites/all/modules/backup_migrate/includes/crud.inc on line 165.

Comments

Comment #1

joachim CreditAttribution: joachim commented 24 March 2010 at 07:14

I'm seeing this too.

Comment #2

ronan CreditAttribution: ronan commented 25 March 2010 at 03:58

This is because of the way I've defined the menu items for these. I made these sub functions local_menu_items (technically tabs) because otherwise the Backup and Migrate tabs disappear when you're editing (or deleting or exporting) an item which can be disorienting. Unfortunately admin menu exposed these as available options even though they don't work without an ID specified.

I modified the code so you should no longer get fatal errors, but ideally I'd like to remove the useless links from admin menu altogether but I'd like to find a way to do so without losing the tabs when you're editing things.

Comment #3

joachim CreditAttribution: joachim commented 25 March 2010 at 07:22

> Unfortunately admin menu exposed these as available options even though they don't work without an ID specified.

Isn't the way to do that to specify a % in the path in hook_menu?

Look at how admin/content/taxonomy/edit/vocabulary/VID works, say. With no VID, it's just a 404.

Comment #4

aiphes

French

CreditAttribution: aiphes commented 25 March 2010 at 08:01

the strange things are that submenu to delete/export/edit aren't displayed on the classic admin page....so the access is only by the admin menu...good luck to fix this

Comment #5

sime

any/any

Melbourne

CreditAttribution: sime commented 2 April 2010 at 07:52

I modified the code so you should no longer get fatal errors,

hi ronan, so this is fixed in the dev version?

Comment #6

ronan CreditAttribution: ronan commented 9 May 2010 at 04:54

Kind of. The fatal errors should be gone but the non working links will still show up in admin menu. I still don't have a way to remove these without screwing up my subtabs.

Comment #7

giorgio79 CreditAttribution: giorgio79 commented 8 July 2010 at 09:39

Priority:

Minor

» Critical

For some reason my default db setting is shown as missing, and when I click on override I get the WSOD with this error message after clicking the back button. I am bumping this to critical, as I cannot use the module without it.

Comment #8

jzornig CreditAttribution: jzornig commented 7 September 2010 at 00:24

subscribe. I can no longer get ot any of the Backup and Migrate admin pages. Always WSOD with "/list/delete" added to the url.

Comment #9

tbenice CreditAttribution: tbenice commented 11 September 2010 at 19:44

Having the same issue. Started with a different error: see http://drupal.org/node/906136.

Now I have the crud_edit() error when trying to view profile lists and get the white screen. Clearing cache does not help.
Started when I tried to edit the default profile to send email on success/failure.