This is a great little module... but once a user starts downloading multiple reports, it would be easier to quickly distinguish reports from one another if each report was named with the title, instead of "customreports-10-02-10.csv".

Additionally, I would like to recommend a date pattern other than "d-m-y", which is usually very confusing (as seen in my example above). "Y-m-d" would produce "2010-02-10".

Attached is a patch which accomplishes these two improvements.

CommentFileSizeAuthor
#1 customreports.module.patch1.77 KBjoelstein

Comments

joelstein’s picture

StatusFileSize
new1.77 KB

Sorry, here's the patch.

snufkin’s picture

Version: 6.x-1.0 » 6.x-1.x-dev
Status: Needs review » Fixed

Committed, thanks! It will be in the new stable release, so far its in the 1.x-dev branch.

Status: Fixed » Closed (fixed)

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