Currently, when installing the drupalorg_testing profile, afterwards there will be about 50 errors about how "File attachments are disabled". These are all due to the fact that the /files/issues directory is not set up properly. This patch does a few things:
1. It makes sure that the files directory exists and is writable early on, and creates it if possible. If it can't create the directory, then it bails on the rest of the installation and provides a warning message to the user.
2. Automatically creates the files/issues directory by executing the project_issue_settings_form. Unfortunately, doing this causes a problem if the Anonymous user is configured to be the user used for automatically closing issues. Because of this, the patch also sets issues to not be automatically closed. I don't think this is a big deal because my guess is that most of the time these test sites don't get cron set up anyway or don't last for 2 weeks in the first place. When we upgrade this to D6 we'll be able to get around this problem. However, the user is warned that this is the case on the post-installation page and is given a link to set it back.
Please see the long code comment in the patch for a more detailed explanation of why this is a problem.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | dot_pi_file_dir_2.patch | 5.13 KB | aclight |
| dot_pi_file_dir_0.patch | 5.21 KB | aclight |
Comments
Comment #1
agentrickardPotentially more efficient patch here: http://drupal.org/node/50789#comment-953185
Comment #2
aclight commentedReroll to eliminate fuzz and offset. Restested and it still works.
Comment #3
hunmonk commentedtested and committed the fix in #2.