jro_apps/redmine Commit - r4588:88d847a1d512

Fixes TimeEntry#spent_on= so that datetimes don't get stored in SQLite3 (#7258)....

Jean-Philippe Lang -

r4588:88d847a1d512

parent child

Context file:

r4588:88d847a1d512

Collapse all files

app/models/time_entry.rb +3 0

             # redMine - project management software
             # Copyright (C) 2006-2008  Jean-Philippe Lang
             #
             # This program is free software; you can redistribute it and/or
             # modify it under the terms of the GNU General Public License
             # as published by the Free Software Foundation; either version 2
             # of the License, or (at your option) any later version.
             #
             # This program is distributed in the hope that it will be useful,
             # but WITHOUT ANY WARRANTY; without even the implied warranty of
             # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
             # GNU General Public License for more details.
             #
             # You should have received a copy of the GNU General Public License
             # along with this program; if not, write to the Free Software
             # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
             class TimeEntry < ActiveRecord::Base
               # could have used polymorphic association
               # project association here allows easy loading of time entries at project level with one database trip
               belongs_to :project
               belongs_to :issue
               belongs_to :user
               belongs_to :activity, :class_name => 'TimeEntryActivity', :foreign_key => 'activity_id'
               attr_protected :project_id, :user_id, :tyear, :tmonth, :tweek
               acts_as_customizable
               acts_as_event :title => Proc.new {|o| "#{l_hours(o.hours)} (#{(o.issue || o.project).event_title})"},
                             :url => Proc.new {|o| {:controller => 'timelog', :action => 'index', :project_id => o.project, :issue_id => o.issue}},
                             :author => :user,
                             :description => :comments
               acts_as_activity_provider :timestamp => "#{table_name}.created_on",
                                         :author_key => :user_id,
                                         :find_options => {:include => :project}
               validates_presence_of :user_id, :activity_id, :project_id, :hours, :spent_on
               validates_numericality_of :hours, :allow_nil => true, :message => :invalid
               validates_length_of :comments, :maximum => 255, :allow_nil => true
               def after_initialize
                 if new_record? && self.activity.nil?
                   if default_activity = TimeEntryActivity.default
                     self.activity_id = default_activity.id
                   end
                   self.hours = nil if hours == 0
                 end
               end
               def before_validation
                 self.project = issue.project if issue && project.nil?
               end
               def validate
                 errors.add :hours, :invalid if hours && (hours < 0 || hours >= 1000)
                 errors.add :project_id, :invalid if project.nil?
                 errors.add :issue_id, :invalid if (issue_id && !issue) || (issue && project!=issue.project)
               end
               def hours=(h)
                 write_attribute :hours, (h.is_a?(String) ? (h.to_hours || h) : h)
               end
               # tyear, tmonth, tweek assigned where setting spent_on attributes
               # these attributes make time aggregations easier
               def spent_on=(date)
                 super
+                if spent_on.is_a?(Time)
+                  self.spent_on = spent_on.to_date
+                end
                 self.tyear = spent_on ? spent_on.year : nil
                 self.tmonth = spent_on ? spent_on.month : nil
                 self.tweek = spent_on ? Date.civil(spent_on.year, spent_on.month, spent_on.day).cweek : nil
               end
               # Returns true if the time entry can be edited by usr, otherwise false
               def editable_by?(usr)
                 (usr == user && usr.allowed_to?(:edit_own_time_entries, project)) || usr.allowed_to?(:edit_time_entries, project)
               end
               def self.visible_by(usr)
                 with_scope(:find => { :conditions => Project.allowed_to_condition(usr, :view_time_entries) }) do
                   yield
                 end
               end
               def self.earilest_date_for_project(project=nil)
                 finder_conditions = ARCondition.new(Project.allowed_to_condition(User.current, :view_time_entries))
                 if project
                   finder_conditions << ["project_id IN (?)", project.hierarchy.collect(&:id)]
                 end
                 TimeEntry.minimum(:spent_on, :include => :project, :conditions => finder_conditions.conditions)
               end
               def self.latest_date_for_project(project=nil)
                 finder_conditions = ARCondition.new(Project.allowed_to_condition(User.current, :view_time_entries))
                 if project
                   finder_conditions << ["project_id IN (?)", project.hierarchy.collect(&:id)]
                 end
                 TimeEntry.maximum(:spent_on, :include => :project, :conditions => finder_conditions.conditions)
               end
             end

test/unit/time_entry_test.rb +30 0

             # redMine - project management software
             # Copyright (C) 2006-2008  Jean-Philippe Lang
             #
             # This program is free software; you can redistribute it and/or
             # modify it under the terms of the GNU General Public License
             # as published by the Free Software Foundation; either version 2
             # of the License, or (at your option) any later version.
             #
             # This program is distributed in the hope that it will be useful,
             # but WITHOUT ANY WARRANTY; without even the implied warranty of
             # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
             # GNU General Public License for more details.
             #
             # You should have received a copy of the GNU General Public License
             # along with this program; if not, write to the Free Software
             # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
             require File.expand_path('../../test_helper', __FILE__)
             class TimeEntryTest < ActiveSupport::TestCase
               fixtures :issues, :projects, :users, :time_entries
               def test_hours_format
                 assertions = { "2"      => 2.0,
                                "21.1"   => 21.1,
                                "2,1"    => 2.1,
                                "1,5h"   => 1.5,
                                "7:12"   => 7.2,
                                "10h"    => 10.0,
                                "10 h"   => 10.0,
                                "45m"    => 0.75,
                                "45 m"   => 0.75,
                                "3h15"   => 3.25,
                                "3h 15"  => 3.25,
                                "3 h 15"   => 3.25,
                                "3 h 15m"  => 3.25,
                                "3 h 15 m" => 3.25,
                                "3 hours"  => 3.0,
                                "12min"    => 0.2,
                               }
                 assertions.each do |k, v|
                   t = TimeEntry.new(:hours => k)
                   assert_equal v, t.hours, "Converting #{k} failed:"
                 end
               end
               def test_hours_should_default_to_nil
                 assert_nil TimeEntry.new.hours
               end
+              def test_spent_on_with_blank
+                c = TimeEntry.new
+                c.spent_on = ''
+                assert_nil c.spent_on
+              end
+              def test_spent_on_with_nil
+                c = TimeEntry.new
+                c.spent_on = nil
+                assert_nil c.spent_on
+              end
+              def test_spent_on_with_string
+                c = TimeEntry.new
+                c.spent_on = "2011-01-14"
+                assert_equal Date.parse("2011-01-14"), c.spent_on
+              end
+              def test_spent_on_with_date
+                c = TimeEntry.new
+                c.spent_on = Date.today
+                assert_equal Date.today, c.spent_on
+              end
+              def test_spent_on_with_time
+                c = TimeEntry.new
+                c.spent_on = Time.now
+                assert_equal Date.today, c.spent_on
+              end
               context "#earilest_date_for_project" do
                 setup do
                   User.current = nil
                   @public_project = Project.generate!(:is_public => true)
                   @issue = Issue.generate_for_project!(@public_project)
                   TimeEntry.generate!(:spent_on => '2010-01-01',
                                       :issue => @issue,
                                       :project => @public_project)
                 end
                 context "without a project" do
                   should "return the lowest spent_on value that is visible to the current user" do
                     assert_equal "2007-03-12", TimeEntry.earilest_date_for_project.to_s
                   end
                 end
                 context "with a project" do
                   should "return the lowest spent_on value that is visible to the current user for that project and it's subprojects only" do
                     assert_equal "2010-01-01", TimeEntry.earilest_date_for_project(@public_project).to_s
                   end
                 end
               end
               context "#latest_date_for_project" do
                 setup do
                   User.current = nil
                   @public_project = Project.generate!(:is_public => true)
                   @issue = Issue.generate_for_project!(@public_project)
                   TimeEntry.generate!(:spent_on => '2010-01-01',
                                       :issue => @issue,
                                       :project => @public_project)
                 end
                 context "without a project" do
                   should "return the highest spent_on value that is visible to the current user" do
                     assert_equal "2010-01-01", TimeEntry.latest_date_for_project.to_s
                   end
                 end
                 context "with a project" do
                   should "return the highest spent_on value that is visible to the current user for that project and it's subprojects only" do
                     project = Project.find(1)
                     assert_equal "2007-04-22", TimeEntry.latest_date_for_project(project).to_s
                   end
                 end
               end
             end

General Comments 0

Write
Preview

You need to be logged in to leave comments. Login now

No TODOs yet

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
g 0-9	Goto bookmarked items from 0-9
n r	New repository page
n g	New gist page

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository access permissions settings
t s	Toggle sidebar on some pages