Automatic Storm time tracking node creation.

the above code has security vulnerability as it doesn't do validation of data before node_save().

Updating files. Code cleanup has been done in the module file. Please ignore the files in comment #4.
EDIT NOTE: i meant ignore files in comment #1. Sorry for the confusion.

Dear drclaw,

i think if you expose the $time variable in your function function time_tracker_time_entry_submit($form, &$form_state) in "time_tracker.module" file, those who want to save the data into custom tables ( like the one am trying to do) would be able to use it.

Possibly some hook if implemented in our custom module will be invoked. Thus all you have to do is module_invoke_all( '[hook_name]', $time ); after you have updated your tables in function time_tracker_time_entry_submit($form, &$form_state) .

A similar hook for deleting and updating can be decided upon.

NOTE: For some reason that i dont know my patch files are not being uploaded properly. So am pasting the patch file that was uploaded in comment #4 here.

diff --git a/time_tracker.module b/time_tracker.module
index 846cea3..24385e6 100644
--- a/time_tracker.module
+++ b/time_tracker.module
@@ -1095,6 +1095,7 @@ function time_tracker_time_entry_submit($form, &$form_state) {
   else {
     drupal_set_message(t('Time Entry Recorded'));
   }
+	time_tracker_storm_store_time( $time );
 }
 
 function time_tracker_time_entry_delete($form, &$form_state) {

If this small patch is committed into the latest dev, then i can carry out further developement of the module, maybe as a separate project.

the patch in comment #6 actually calls a module_invoke_all('hook_name', $time);

Attaching the patch mentioned in comment #6 without the square brackets. Hopes it works.

Just in case it doesn't am pasting the patch here below.

Index: time_tracker.module
===================================================================
--- time_tracker.module	(revision 36)
+++ time_tracker.module	(working copy)
@@ -1095,6 +1095,7 @@
   else {
     drupal_set_message(t('Time Entry Recorded'));
   }
+	module_invoke_all('tt_store_time', $time);
 }
 
 function time_tracker_time_entry_delete($form, &$form_state) {

Noops the above patch wont show up cause of this #284899: Drupal url problem with clean urls.

Renaming the patch file again.

Hi, i have created a sandbox project that is waiting for the patch to be commited .
This is the path to the sandbox project.

Thank you.

any updates?

i appreciate your willingness to include some hooks.

Will be anxiously waiting for its release. Thank you.

Hi codi Welcome,

Time tracker integrates nicely with atrium and i can see it plays well with case tracker too.

I think there might be people (like me) who want to use timetracker with other project management application. It would be a nice feature to include a hook like what i had proposed earlier (#9).

If some hook like that is implemented, i could use a hook like that to update Storm using Time tracker storm update

Thank you.