Closed (fixed)
Project:
Drupal.org security advisory coverage applications
Component:
module
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
1 Mar 2012 at 08:25 UTC
Updated:
1 Aug 2012 at 10:11 UTC
This module extends the backup functionality of Aegir. It adds a tab to the site content type. The tab shows the backups and enables per backup actions (Restore, Delete and Get). There is currently, as far as we know, no similar plugin or module that adds this functionality to Aegir.
Sandbox page: http://drupal.org/sandbox/gerbenspil/1459830
Git clone command: git clone --branch 6.x-1.x http://git.drupal.org/sandbox/gerbenspil/1459830.git hosting_site_backup_manager
This module is only for Drupal 6 as Aegir is running on Drupal 6.
Comments
Comment #1
patrickd commentedwelcome,
while waiting for an in-depht review of your module you can start out fixing some coding style issues detected by automated tools:
http://ventral.org/pareview/httpgitdrupalorgsandboxgerbenspil1459830git
You can also get a review bonus and we will come back to your application sooner.
Comment #2
Gerben Spil commentedThanks patrickd, i thought I had them all with coder module. Will use the ventral.org website as well next time to check it.
Comment #3
Gerben Spil commentedRan the module through ventral.org untill all found issues were fixed.
Comment #4
helmo commentedThis is a nice addition to the Aegir platform.
The option to download a backup has been opted before, e.g. in #1047992: exportable backups
I have one concern about file permissions, which we've already discussed privately and filed under #1468444: File permissions on backups
Comment #5
Gerben Spil commentedCurrently implementing #4.
Comment #6
helmo commentedI've become co-maintainer of this module.
All open issues have been resolved and I've ran in through ventral.org again.
Would I be able to promote this to a full project If I had 'Edit project' privileges?
Comment #7
helmo commentedTo answer my own question: Yes.
And since I've just received that privilege, I've taken the liberty to promote.
Calling this fixed.
Comment #8.0
(not verified) commentedAdded a 6.x-1.x branch to git and edited the application accordingly.