search

hicknhack-software / redmine_time_tracker

A time tracker plugin for Redmine
Other
106 stars 32 forks source link

The Running Time in overview page disappeared (updated) #111

Closed shuozhifenxi closed 10 years ago

shuozhifenxi commented 10 years ago

Hi,

It is still the error 110. I copied the two lines in the opposite position in yesterday conversation. The right description should be

In app/view/time_trackers/_timer.js.erb, it used to be

var start = new Date($('#time_tracker_date').val().replace(/-/ig, '/') + ...

But now it is

var start = new Date();

the browser is complaining #time_tracker_date is unrecognized expression.

shuozhifenxi commented 10 years ago

And I did not change the code by myself, I am sure I am using your master branch in the repository.

Also I confirmed the commit b19de071929c43b886f0a810d873ffba1754c948 is the last working one, and the next one 991075d02d63d6baaa7d517cd54233f2fc45557e brings the problem.

shuozhifenxi commented 10 years ago

Ok, It seems it is a only misspelling introduced by 991075d

in you code of app/views/time_trackers/_time_tracker_control.html.erb line 65

<%= javascript_tag("$(function() { $('#time_tracker_date}').datepicker(locDatepickerOptions);

should be

<%= javascript_tag("$(function() { $('#time_tracker_date').datepicker(locDatepickerOptions);

There is one extra "}"

And after removing the extra "}", the running time is back.

jume-dev commented 10 years ago

ah yeah, i see. strangeley i didnt got an error with this. Thanks for finding this typo, will fix when i am in office.