Changes the representation of journal details in issue API.

git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@4483 e93f8b46-1217-0410-a6f0-8f06a7374b81

Changes the representation of journal details in issue API.
git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@4483 e93f8b46-1217-0410-a6f0-8f06a7374b81
ab6a93b0 · Jean-Philippe Lang · 252e4983 · ab6a93b0 · ab6a93b0
Commit ab6a93b0 authored Dec 10, 2010 by Jean-Philippe Lang
Hide whitespace changes
Inline Side-by-side

Showing with 38 additions and 1 deletion

show.api.rsb app/views/issues/show.api.rsb +4 -1

issues_test.rb test/integration/api_test/issues_test.rb +34 -0

No files found.
--- a/app/views/issues/show.api.rsb
+++ b/app/views/issues/show.api.rsb
@@ -50,7 +50,10 @@ api.issue do
  			api.notes journal.notes
  			api.array :details do
  				journal.details.each do |detail|
-  					api.detail :property => detail.property, :name => detail.prop_key, :old => detail.old_value, :new => detail.value
+  					api.detail :property => detail.property, :name => detail.prop_key do
+  					  api.old_value detail.old_value
+  					  api.new_value detail.value
+  					end
  				end
  			end
  		end

--- a/test/integration/api_test/issues_test.rb
+++ b/test/integration/api_test/issues_test.rb
@@ -91,6 +91,40 @@ class ApiTest::IssuesTest < ActionController::IntegrationTest
  end
  context "GET /issues/:id" do
+    context "with journals" do
+      context ".xml" do
+        should "display journals" do
+          get '/issues/1.xml'
+          assert_tag :tag => 'issue',
+            :child => {
+              :tag => 'journals',
+              :attributes => { :type => 'array' },
+              :child => {
+                :tag => 'journal',
+                :attributes => { :id => '1'},
+                :child => {
+                  :tag => 'details',
+                  :attributes => { :type => 'array' },
+                  :child => {
+                    :tag => 'detail',
+                    :attributes => { :name => 'status_id' },
+                    :child => {
+                      :tag => 'old_value',
+                      :content => '1',
+                      :sibling => {
+                        :tag => 'new_value',
+                        :content => '2'
+                      }
+                    }
+                  }
+                }
+              }
+            }
+        end
+      end
+    end
    context "with custom fields" do
      context ".xml" do
        should "display custom fields" do