Filesystem Class for ftpext does not perform chmod for directories

[st3ff3n]

When I try to install or update a plugin with ftp, which comes with its own directory, the directory gets created with permission 400. The ftp_chmod function is not performed for created directories. wp-admin is not able, to acces this plugin. I don't know, if this is the desired behaviour, but I worked around this with the attached patch. The exists function always returns false for directories, so I changed the line to also check, if the argument is a directory.

Perhaps there are better ways, to do this.

Best regards,
Steffen Hau

[DD32]

I'll look into this.

Can I get some examples however?

So, In short:

• When you connect via FTP, The folders are created as chmod 400 (This would not be an issue alone to WP, It should happen when you connect via FTP to the host as well)
• What chmod do you expect the folders to be?
• What chmod are the created files/what do you expect them to be?
• Have you defined FS_CHMOD_DIR / FS_CHMOD_FILE to see if that fixes your problem?
  • Note, You'd define those as such in your wp-config.php file: define('FS_CHMOD_DIR', 0660);(Note the leading zeo): [10050]

[DD32]

Havn't actually tested this, But looking at the code it seems like its definitely a possibility. The patch cant do any harm as well, So it might as well be commited.

Some extra look will be needed to see if ::exists() is returning true only for files.

[ryan]

(In [11082]) Check if not exists and not is_dir before bailing. Props st3ff3n. fixes #9274