Opened 16 years ago

Closed 16 years ago

#1872 closed New Feature (fixed)

Add option to change permissions applied with chmod commmand

Reported by: Wiktor Walc Owned by: Wiktor Walc
Priority: Normal Milestone: FCKeditor 2.6
Component: Server : Perl Version:
Keywords: Confirmed Review+ Cc:

Description

For more info please take a look at: #950

Attachments (1)

1872.patch (2.0 KB) - added by Wiktor Walc 16 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 Changed 16 years ago by Wojciech Olchawa

Keywords: Confirmed added

Changed 16 years ago by Wiktor Walc

Attachment: 1872.patch added

comment:2 Changed 16 years ago by Wiktor Walc

Keywords: Review? added

comment:3 Changed 16 years ago by Frederico Caldeira Knabben

Keywords: Review- added; Review? removed

Is there any real advantage of having two different settings, one for folder creation, and another for file upload? I think it is ok to have a single generic setting for all our chmod needs.

Maybe I'm wrong, in that case, feel free to Review? it again.

comment:4 Changed 16 years ago by Wiktor Walc

Keywords: Review? added; Review- removed

The most commonly used permissions are 755 for folders and 644 for files, so making it configurable with just one variable may be a wrong idea.

So I think it's better to stay with separate settings for folders and files.

comment:5 Changed 16 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

Ok... the same approach should be applied to the other languages at this point.

Please add a changelog for it on commit.

comment:6 Changed 16 years ago by Wiktor Walc

Resolution: fixed
Status: newclosed

Fixed with [1626].

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy