Can't generate 'create view' sql

Backup and migrate doesn't support MySQL views. Are there any Drupal modules that use them? Drupal is supposed to work with MySQL 4.1 so this should not be an issue with Drupal data, but if there are any contributed modules using them then I'll have to either support views or warn the user about not supporting them.

If you are backup up data from tools other than Drupal you would be safest to use a more complete tool like phpMyAdmin.

I used this contrib module http://drupal.org/project/uc_views
I have created issue for this module http://drupal.org/node/448038

Since many of the uc_views users are reporting this problem, I have created a patch for backup_migrate that allows db views to be backed up and restored.

the patch is made against the head branch.

hope you can use it.

By the way I noticed that you pass $exclude_tables and $nodata_tables as parameters for _backup_migrate_dump_tables and _backup_migrate_get_dump_sql but you don't seem to use the values since they are overwritten within the functions. Therefor I have avoided changing the signature of those functions, even though I have a $exclude_views equivalent to $exclude_tables.

And the patch attachement :-S

Thanks for the patch. I'll review and port this as soon as possible. I may not add it to the 1.x branch as I'm trying to make that branch obsolete and am approaching a stable release for the 2.x branch.

Today I had the chance to try out my patch in an installation without db-views and I realized that I am instantiating the default values of the lists containing the views and the view names wrong.

So rather than setting
$out = "";
it really should be
$out = array();
in both

Sorry for not testing properly the first time.

I LOVE this module. It has saved my behind more than once.

I ran into this problem recently where a view table is not created. If you use ubercart at all, I think you will run into this issue.

The patch doesn't seem to work it all fails.

A

How does the patch fail?