download tracking by user

Okay, I will see if I can get something to you by tomorrow afternoon.

Regards

Okay, here is my first pass at making a data table that will track a download by user. I modified two files, the .install and the .module file. In the former, I created an update section as well as the table definitions in the install section. In the .module file, I created a function that simply populates that table with a tracking row. I am willing to maintain this additional feature in the hopes of improving this great module. I did not update the .info module as I was pretty sure that I was not supposed to do that.

Go easy on me, this was my very first patch :)

all I did at this point was to make the logging occur in the data base. I don't understand how views work well enough to expose the tables and columns in the views module yet, and I need this working well enough that I can collect and pull some raw data asap. It looked to me that someone was working on integrating webfm with views, as there are several files that indicated that to me, so I am hoping that piece will be forthcoming soon and I can integrate with that soon enough.

Is that helpful?

just use the permissions interface. I have it set up the same way.

Okay, I got carried away on some other business, but have come back to this. My question is this... If I have created an /admin/reports interface that reads table data from the table that I created, do I need to send in a second, partial patch, apart from the one that I sent you earlier, or do you want a completely new patch with all my changes in it.

I can do either. I have not added much by way of statistics, only the fields that I had there before. That is all that I need, but I would be willing to track and maintain issues with that feature, in effect, co-support this module, if you like.

Let me know what works best for you and I will do it.

Thanks,

Okay, I have a patch ready for testing.

What it does:

An additional table has been added to the schema.

mysql> desc webfm_statistics;
+-------- -+---------------------+------+-----+----------------------------+--------------------+
| Field | Type | Null | Key | Default | Extra |
+----------+---------------------+------+-----+----------------------------+--------------------+
| sid | int(10) unsigned | NO | PRI | NULL | auto_increment |
| uid | int(10) unsigned | NO | | NULL | |
| fid | int(11) | NO | | 0 | |
| dl_time | datetime | NO | | 0000-00-00 00:00:00 | |
+----------+---------------------+------+-----+----------------------------+---------------------+
4 rows in set (0.01 sec)

The report query calls left outer joins against 5 tables to get necessary data.

$query = "SELECT a.sid, a.uid, a.fid, a.dl_time, b.name, b.mail, c.fpath, d.nid, e.title
FROM webfm_statistics a
LEFT OUTER JOIN users b ON a.uid = b.uid
LEFT OUTER JOIN webfm_file c ON a.fid = c.fid
LEFT OUTER JOIN webfm_attach d ON a.fid = c.fid
LEFT OUTER JOIN node e ON d.nid = e.nid";

This gives the ability to build links to the user page, the node if the file is attached to one, and the user's email as well as display the physical file path on disk. All of this information was requested by my client. The purpose of the email was to easily contact a user after the user downloaded a file.

I have created a .csv export function that end users are able to pull, as well as a manual delete function that will clean data out of the table. If desired, I am thinking that automatic report saving (maybe sending via email) and report table cleaning functions that could be executed via cron (my current client )has not asked for that, but I can see how that may be desirable.

Please import these features into the core of the module for drupal 6. Let me know when this is done and I will be happy to port this feature into Drupal 7 as well.

Thanks!

All excellent feedback. I will make the necessary changes and resubmit. Thank you so much for your helpful insight.

Regards,

1. Done
2. I didn't see any tabs in the file, but I did clean out some of my non-drupal conforming spacing. I hope I got it all...
3. optional: maybe at a later time
4. I expect to expand on this feature as time goes on, so I am thinking that a more generic name is appropriate at this time.
5. Done
6. Done
7. DONE
8. done
9. done

Thanks again... Do let me know if you see anything else.

I have a slight error in my query... attached is the revised patch.

I don't know how to use the module_invoke_all hook, but I will look into it. My hope was to simply have a bit of feedback for who has downloaded each file. To me it seems better placed within the webfm module itself. I don't really know how the modules sub directory process works, so if you want me to do that, I will need you to tell me how to go about it.

Thanks,

I have my module constructed, I simply need the changes that you would need to make in your module so that I can know when webfm_file_send function is called. Thanks,

Thanks Rob,

I have linked it up and it works as it did before. I have requested a CVS account, if it gets approved, I think we will be done here.

I have released this project in development, pending feedback from a few users/testers. If you are interested in using it and can offer some feedback as to its applications, please let me know...

the module can be found here:

http://drupal.org/project/webfm_statistics

Thanks,