Update sanitize script to include CiviCRM db backups if they exist #2
Conversation
index.php
Outdated
| } | ||
| } | ||
| } | ||
|
|
||
| /** | ||
| * Print the HTML for a single table row. | ||
| */ | ||
| function create_row($name, $db_path, $file_path) { | ||
| function create_row($name, $db_path, $db_civi_path, $file_path) { |
There was a problem hiding this comment.
can we make the new argument the last one? We won't have civi for most of our sites, and it would be easier not to need to pass NULL when backing up the docs site (for example).
There was a problem hiding this comment.
Sure, happy to swap the arguments (see revised PR). I will note that when we pass '' (not NULL), that happens automatically (or not) based on whether there's a CiviCRM db backup in place, so that this works currently for all four properties.
And then if we decide to add CiviCRM to, for example, the events site (to do event registrations), all we'd have to do, backup/sanitation-wise, is add the *_CIVI settings to the config.ini file on events.b.org, use the current sql-dump-sanitize.php script, and add the symlink to the sanitized_civi directory, and this function will start showing the Civi backups automatically as they start to be created.
Fixes #1.