I'm not exactly sure how useful it will be to have the Pathauto pattern for files defaulted to file/[file:name] especially since when you install the module, [file:name] by default is the basename of the file on the system. Not exactly the most useful. Also this can create conflicts if I named a file '1' so that now the alias file/1 overrides the system path file/1.

Pathauto integration is good and useful, but I don't agree that it's useful enabled by default. Discuss?

Files: 
CommentFileSizeAuthor
#4 1959268-4-pathauto-defaults.patch736 bytesaaron
PASSED: [[SimpleTest]]: [MySQL] 874 pass(es).
[ View ]

Comments

Devin Carlson’s picture

My 2¢:

The content editors here always change a file's filename to something more friendly after it's uploaded. It makes finding existing files much easier (on the file listing page, in the Media Browser and when linking to files using CKEditor Link File).

In the event that they don't input a filename manually, they still prefer to pass around poorly formed URLs (example.com/file/new-memo-finaldoc) over plain file IDs since they still give some idea of what file they represent vs. file IDs.

How does pathauto deal with other entities when it comes to aliases conflicting with system paths and are there any ways around it?
Should a filename field be added to the file upload wizard so that users can give files a "proper" filename during upload?
Do any other entity types provide Pathauto integration but are disabled by default? If not, do you think that it will be confusing to users who expect that pathauto just works automatically out-of-the-box?

Dave Reid’s picture

pathauto deals with it by failing to save the alias. I'm just not sure how useful this is to be configured by default. As a pathauto maintainer, I don't really like that we ship with aliases by defaults since most people go in and have to change them anyway, and then have ended up with a bunch of aliases that they don't want or need that have already been generated. I would prefer if site builders made choosing the alias patterns something they had to think about first.

I'd like to get more input on this proposal.

Other things to note:
http://drupal.org/sandbox/damz/1332096 (pathauto integration for all entity types) uses empty string by default.

aaron’s picture

Status:Needs review» Active

I agree with both the points here. On the one hand, I think that it would be best to leave it blank, however, considering that pathauto currently sets some defaults, it might make sense to add a default for file entities as well. However, if we do that, I don't believe that it would be good practice to leave the first element of the URL alias as file/ because of the overriding issue. Perhaps it would be better to call it files/ instead. Otherwise I am neutral as to what direction we choose to go.

aaron’s picture

Status:Active» Needs review
StatusFileSize
new736 bytes
PASSED: [[SimpleTest]]: [MySQL] 874 pass(es).
[ View ]

This patch makes the pathauto pattern default to files/fid.

aaron’s picture

Status:Active» Needs review

Bumping for review and comment.

aaron’s picture

#4: 1959268-4-pathauto-defaults.patch queued for re-testing.

aaron’s picture

Status:Needs review» Fixed

Automatically closed -- issue fixed for 2 weeks with no activity.