It is good idea to alter the column headings while displaying the results.
Also, it may be good idea to display the image files rather than giving link to files.

Comments

noel.rivas’s picture

Component: Miscellaneous » Code
Assigned: Unassigned » noel.rivas
Status: Active » Needs review
StatusFileSize
new336 bytes

I added a drupal_alter call at the end of the function that builds the report data. This will allow other modules to modify both headers and rows in several ways. They'll just have to implement module_webform_report_data_alter(&$data).

noel.rivas’s picture

StatusFileSize
new343 bytes
jimbullington’s picture

I don't have a problem with this - but, can we just call the hook 'webform_report' - since it is the only one? And add a bit of doc...

// allow report data to be altered through a module
// module must implement a {module}_webform_report_alter(&$report, $node) function
drupal_alter('webform_report', $report, $node);
noel.rivas’s picture

StatusFileSize
new487 bytes

Doc added and type simplified.

Thanks ;)

jimbullington’s picture

Thanks. Pushed to repo.

jimbullington’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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