Fix: Spent time CSV report returning hours as Rational instead of Float (#41895).

Patch by Go MAEDA (user:maeda).


git-svn-id: https://svn.redmine.org/redmine/trunk@23351 e93f8b46-1217-0410-a6f0-8f06a7374b81

app/helpers/queries_helper.rb

@@ -309,7 +309,7 @@ module QueriesHelper
     else
       format_object(value, html: false) do |value|
         case value.class.name
-        when 'Float'
+        when 'Float', 'Rational'
           sprintf("%.2f", value).gsub('.', l(:general_csv_decimal_separator))
         when 'IssueRelation'
           value.to_s(object)

test/functional/timelog_controller_test.rb

@@ -1740,6 +1740,16 @@ class TimelogControllerTest < Redmine::ControllerTest
       get :index, :params => {:format => 'csv'}
       assert_response :success
       assert_equal 'text/csv; header=present', response.media_type
+
+      parsed_csv = CSV.parse(response.body)
+      assert_equal %w[Project Date User Activity Issue Comment Hours], parsed_csv.first
+      assert_equal(
+        [
+          'eCookbook', '03/12/2007', 'Redmine Admin', 'Design',
+          'Bug #1: Cannot print recipes', '', '150.00'
+        ],
+        parsed_csv.last
+      )
     end
   end