Closed (fixed)
Project:
Drupal.org security advisory coverage applications
Component:
module
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Anonymous (not verified)
Created:
1 Mar 2012 at 08:25 UTC
Updated:
9 Sep 2026 at 04:30 UTC
Jump to comment: Most recent
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
Anonymous (not verified) 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
Anonymous (not verified) 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
Anonymous (not verified) 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.