I have added a few of the requested features and done an overall re-factoring of the custom reports module.
I would like to purpose a 2.0 version or perhaps to merge some of the functionality back into the custom reports 1.0 release.
I have uploaded the module here for review and feedback. (I tried to create a patch however I can't seem to figure out how to have the patch create directories that did not previously exist.)
The main features include;
- Ability to add a temporary table field to the customreports content type.
- Run custom reports in the browser with a new tab titled 'Run reports'.
- More granular access controls to the various customreports tabs.
Some differences include;
- The customreports content type is created when the module is enabled. (If it doesn't already exist)
- The query code is output into collapsible field-sets and includes a theme template file for easy overrides.
- The customreports content type is generated using the CCK content copy method.
Using the temporary table fields feature you can now write more complex and optimized queries, then reference your temporary tables in the main report query.
I have not done much with the access control but I think it should be considered as well. see #859380: node grants fails
Please review the code and let's discuss!
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | customreports-enhanced-900690.patch | 31.93 KB | Architeck |
| #2 | customreports.zip | 21.57 KB | Architeck |
| customreports2.zip | 18.08 KB | Architeck |
Comments
Comment #1
a_c_m commentedComment #2
Architeck commentedPlease review the updated module attached to this comment.
I have done some code cleanup and resolved a few issues.
Comment #3
a_c_m commentedPatches are much better than full module exports, because as soon as we apply another patch, your full module export becomes a backward step.
http://drupal.org/patch/create
Comment #4
a_c_m commentedComment #5
Architeck commentedPlease see the attached patch of the functionality mentioned above.
The patch should be applied to the customreports 6.x dev release and should be run against the customreports directory.
Run the patch from the parent directory that contains your customreports module.
patch -p0 < customreports-enhanced-900690.patch