Great module! I wanted to extend it to use also current user's session id. To do this, I had to override hashing function. I think it's good idea to include drupal alter hook into filefield_stats module in order to allow third-party modules to extend this module.

Proposed patch attached

CommentFileSizeAuthor
filefield_stats.module.alterhook.patch565 bytespavel.karoukin
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

infojunkie’s picture

infojunkie
Location Vancouver
CreditAttribution: infojunkie commented
Project: Filefield Styles » Filefield Stats

Reassigning to correct module.

pavel.karoukin’s picture

pavel.karoukin CreditAttribution: pavel.karoukin commented

Oh. I am really sorry. Somehow I post it into wrong issue queue. Thank you infojunkie!

fietserwin’s picture

fietserwin
Location Le Mont-Dore
CreditAttribution: fietserwin commented

@Scrypt, I'm going currently through this issue queue.

I assume that this is a copy of #986568: Allow third-party modules to plug into files dispatch process? But, looking at your code, I can't see any actual usage of this alter hook and, frankly speaking, I don't see why you would want to change the URL. You might want to be informed about the constructed URL, but not change it. Or am I missing something?