Opened 7 years ago

Closed 7 years ago

#2589 closed defect (bug) (fixed)

WP DB Backup core tables should reflect new and deleted tables

Reported by: MichaelH Owned by: anonymous
Priority: normal Milestone: 2.1
Component: Administration Version: 2.1
Severity: normal Keywords: bg|has-patch bg|tested bg|commit bg|2nd-opinion
Cc:

Description

Add the new 'link2cat' table and delete the old 'linkcategories' table from the "These core WordPress tables will always be backed up:" list in WordPress Database Backup Plugin (wp-db-backup.php).

Attachments (3)

db-backup-plugin_updatedtables.diff (685 bytes) - added by abhay 7 years ago.
db-backup-plugin_updated_tables2.diff (1.2 KB) - added by abhay 7 years ago.
2589.diff (1.7 KB) - added by westi 7 years ago.
New patch updated to use a single array

Download all attachments as: .zip

Change History (12)

  • Keywords bg|has-patch added

Good catch and easy to fix. Patch to come.

Thanks abhay, the fix works. I didn't test with Skippy's WP-Cron plugin, but notice there's a reference to tables in the wp-cron-daily function in that plugin. Should that be changed also?

MichaelH: sorry I can't find the reference. Could you give me a "grep --line-number" ?

Should have done that to begin with--line 869.

  • Keywords bg|needs-testing added

It's alright. I should have caught it. New patch is up.

westi7 years ago

New patch updated to use a single array

  • Keywords bg|tested bg|commit added; bg|needs-testing removed

I have tested this and it works fine for me.

The cron list of tables was still missing "link2cat" so i have added that in a new patch and moved this list of tables to a class attribute storing them in an array near the top of the file so they are easy to find for updates and only in one place!

I think this is good to go for commit

  • Keywords bg|2nd-opinion added

While testing westi's fix on a 2.0.2 to 2.1-alpha1 upgrade, I was reminded that the linkcategories table is being left, intact, on upgrades. With this fix, that table will show in the "You may choose to include any of the following tables" column of WP DB Backup.

Any idea on what to do there, if anything?

Also, any opinion about bumping the Version from 1.7 to 1.7.something?

comment:8   ryan7 years ago

Eventually, upgrade will drop the linkcategories table. I've been holding off on dropping the table until the bookmark design settled. That has quieted down though, so maybe the time to drop is near.

comment:9   ryan7 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [3657]) Update table list in backup plugin. Props abhay and westi. fixes #2589

Note: See TracTickets for help on using tickets.