Opened 9 years ago

Closed 9 years ago

#1872 closed New Feature (fixed)

Add option to change permissions applied with chmod commmand

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


For more info please take a look at: #950

Attachments (1)

1872.patch (2.0 KB) - added by wwalc 9 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 Changed 9 years ago by w.olchawa

  • Keywords Confirmed added

Changed 9 years ago by wwalc

comment:2 Changed 9 years ago by wwalc

  • Keywords Review? added

comment:3 Changed 9 years ago by fredck

  • 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 9 years ago by wwalc

  • 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 9 years ago by fredck

  • 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 9 years ago by wwalc

  • Resolution set to fixed
  • Status changed from new to closed

Fixed with [1626].

Note: See TracTickets for help on using tickets.
© 2003 – 2017 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy