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!

Comments

a_c_m’s picture

Status: Active » Needs review
Architeck’s picture

StatusFileSize
new21.57 KB

Please review the updated module attached to this comment.

I have done some code cleanup and resolved a few issues.

a_c_m’s picture

Patches 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

a_c_m’s picture

Status: Needs review » Needs work
Architeck’s picture

Status: Needs work » Needs review
StatusFileSize
new31.93 KB

Please 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