I have the "your active times" block active for every page. When I am clocked in, it gives me the "You have an unsaved timer on NODE NAME" link. This link works when clocked in, but if I stop the clock, it will not direct me to the correct url. It takes me to /node/2300%23edit-time-wrapper, which does not exist.

Works fine if the clock is still ticking though.

Comments

dev25 created an issue. See original summary.

dev25’s picture

Priority: Normal » Critical
ron.du’s picture

I have same issue with both 1.5 and 1.6rc version

dev25’s picture

I went into the module code, keyword searched for the "2300%23edit-time-", and removed it. Seemed to work.

andrewoke’s picture

Hi All,

That link is due to incorrect formatting. I've rolled a diff you can use to correct it.

diff --git a/modules/time_tracker_timer/time_tracker_timer.module b/modules/time_tracker_timer/time_tracker_timer.module
index a670648..d33093d 100644
--- a/modules/time_tracker_timer/time_tracker_timer.module
+++ b/modules/time_tracker_timer/time_tracker_timer.module
@@ -1054,7 +1054,7 @@ function theme_time_tracker_timer_active_timers_block($variables) {
         // since it still exists (should be deleted when stopped and saved).
         // Set the timer state to 'stopped' state.
         $timer_state = 'stopped';
-        $timer_links = l($entity->title, $uri['path'] . '#edit-time-wrapper');
+       $timer_links = l($entity->title, $uri['path'], array('fragment' => 'edit-time-wrapper'));
         $total_time = $timer->total ? $timer->total : $timer->stop - $timer->start;
         $timer_msg = t('You have an unsaved timer on !links.', array('!links' => $timer_links));
         $path = drupal_get_path('module', 'time_tracker') . '/css/images/time_tracker_stopwatch_icon_white.png';