Make sure that the tmgmt_file folder is writable for the file translator [#2860782]

Problem

If the tmgmt_file folder exists, but it is not writable, the export will fail. The reason is that the file translator, when requesting the translation and calling file_prepare_directory() does not try to make the folder writable if it finds already one.

Proposed solution

Add the FILE_MODIFY_PERMISSIONS flag to the file_prepare_directory() call.

Comment	File	Size	Author
#4	tmgmt_file_writable_2860782_3.patch	881 bytes	vasi1186
#4	7.x-1.x: PHP 5.5 & MySQL 5.5, D7 141 pass, 9 fail
#2	tmgmt_file_writable_2860782_2.patch	876 bytes	vasi1186
#2	7.x-1.x: PHP 5.5 & MySQL 5.5, D7 Patch Failed to Apply

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

15 March 2017 at 12:19

vasi1186 created an issue. See original summary.

Comment #2

vasi1186 CreditAttribution: vasi1186 at Amazee Labs commented 15 March 2017 at 12:22

Status:

Active

» Needs review

File	Size
tmgmt_file_writable_2860782_2.patch	876 bytes
7.x-1.x: PHP 5.5 & MySQL 5.5, D7 Patch Failed to Apply

I attached a patch that should fix the issue. Should we maybe also show a message in case this operation fails? At the moment you will not see any message on the screen. You will just be redirected to the job form without knowing what happened.