Incomplete backup

[autosurf]

Hello,

Before the update version 1.0,
I looked at the backup made ​​by VL, it is different from a backup made ​​by conventional mysql.

In fact, all the tables are not saved by VL, see sources/admin/backup_database.php,
missing tables 10 tables 24 ^^

It would be nice to have a full backup and perhaps the ability to restore directly under VL?

 

[Basti]

missing tables 10 tables 24 ^^

and which shall these be? Vl by default has no 24 tables... so please name which are missing. Might as well be a plugin issue ( my bet ) where the plugin made a new db table but is not using the admin backup hook so that table gets backed up

 

[autosurf]

actually, there is not 24 but 18 tables tables
I counted my management for advertisements ^^

i see in the file :

$tables_to_backup = array(
"{$CONF['sql_prefix']}_settings" => true,
"{$CONF['sql_prefix']}_bad_words" => true,
"{$CONF['sql_prefix']}_ban" => true,
"{$CONF['sql_prefix']}_custom_pages" => true,
"{$CONF['sql_prefix']}_etc" => true,
"{$CONF['sql_prefix']}_categories" => true,
"{$CONF['sql_prefix']}_ip_log" => false,
"{$CONF['sql_prefix']}_reviews" => true,
"{$CONF['sql_prefix']}_sessions" => false,
"{$CONF['sql_prefix']}_sites" => true,
"{$CONF['sql_prefix']}_sites_edited" => true,
"{$CONF['sql_prefix']}_stats" => true,
"{$CONF['sql_prefix']}_menu" => true,
"{$CONF['sql_prefix']}_menus" => true
);

i have : VL_join_fields, VL_langs, VL_osbanners, VL_screens, VL_settings

it seems to me that not all of the tables for plugins?

 

[Basti]

You did not updated correctly at one point. That file is old. Make sure you have the newest file as found in 1.0 ( actually 0.9 was the same, so your file is reaaaaly old )

And if you have the correct file, your tables created through plugins ( like VL_screens ) get handled also. Yours does not cos you also miss the plugin hook

So the correct version feature 15 tables to backup. So still thanks for the headups, as "VL_langs" is missing nonetheless .
I will post a patch with this and some other small things later.

 

[autosurf]

I especially noticed the file VL_langs indeed my VL is French ^^
Thank you for the patch and perhaps the ability to restore directly under VL ?

 

[Basti]

Thank you for the patch and perhaps the ability to restore directly under VL ?

at some point maybe yes, but no within the patch