rconomic 0.5.2 → 0.6.0

data/spec/economic/proxies/invoice_proxy_spec.rb

@@ -1,7 +1,6 @@
-require './spec/spec_helper'
+require "./spec/spec_helper"
 
 describe Economic::InvoiceProxy do
-
   let(:session) { make_session }
   subject { Economic::InvoiceProxy.new(session) }
 
@@ -27,12 +26,16 @@ describe Economic::InvoiceProxy do
 
   describe ".find" do
     it "gets invoice data from API" do
-      mock_request('Invoice_GetData', {'entityHandle' => {'Number' => 42}}, :success)
+      mock_request(
+        "Invoice_GetData",
+        {"entityHandle" => {"Number" => 42}},
+        :success
+      )
       subject.find(42)
     end
 
     it "returns Invoice object" do
-      stub_request('Invoice_GetData', nil, :success)
+      stub_request("Invoice_GetData", nil, :success)
       expect(subject.find(42)).to be_instance_of(Economic::Invoice)
     end
   end
@@ -42,26 +45,37 @@ describe Economic::InvoiceProxy do
     let(:unto) { Time.now }
 
     it "should be able to return a single current invoice" do
-      mock_request('Invoice_FindByDateInterval', {'first' => from.iso8601, 'last' => unto.iso8601}, :single)
-      mock_request('Invoice_GetDataArray', :any, :single)
+      mock_request(
+        "Invoice_FindByDateInterval",
+        {"first" => from.iso8601, "last" => unto.iso8601},
+        :single
+      )
+      mock_request("Invoice_GetDataArray", :any, :single)
       results = subject.find_by_date_interval(from, unto)
       expect(results.size).to eq(1)
       expect(results.first).to be_instance_of(Economic::Invoice)
     end
 
     it "should be able to return multiple invoices" do
-      mock_request('Invoice_FindByDateInterval', {'first' => from.iso8601, 'last' => unto.iso8601}, :many)
-      mock_request('Invoice_GetDataArray', :any, :multiple)
+      mock_request(
+        "Invoice_FindByDateInterval",
+        {"first" => from.iso8601, "last" => unto.iso8601},
+        :many
+      )
+      mock_request("Invoice_GetDataArray", :any, :multiple)
       results = subject.find_by_date_interval(from, unto)
       expect(results.size).to eq(2)
       expect(results.first).to be_instance_of(Economic::Invoice)
     end
 
     it "should be able to return nothing" do
-      mock_request('Invoice_FindByDateInterval', {'first' => from.iso8601, 'last' => unto.iso8601}, :none)
+      mock_request(
+        "Invoice_FindByDateInterval",
+        {"first" => from.iso8601, "last" => unto.iso8601},
+        :none
+      )
       results = subject.find_by_date_interval(from, unto)
       expect(results.size).to eq(0)
     end
-
   end
 end

data/spec/economic/proxies/order_proxy_spec.rb

@@ -1,7 +1,6 @@
-require './spec/spec_helper'
+require "./spec/spec_helper"
 
 describe Economic::OrderProxy do
-
   let(:session) { make_session }
   subject { Economic::OrderProxy.new(session) }
 
@@ -27,12 +26,12 @@ describe Economic::OrderProxy do
 
   describe ".find" do
     it "gets order data from API" do
-      mock_request('Order_GetData', {'entityHandle' => {'Id' => 42}}, :success)
+      mock_request("Order_GetData", {"entityHandle" => {"Id" => 42}}, :success)
       subject.find(42)
     end
 
     it "returns Order object" do
-      stub_request('Order_GetData', nil, :success)
+      stub_request("Order_GetData", nil, :success)
       expect(subject.find(42)).to be_instance_of(Economic::Order)
     end
   end
@@ -42,26 +41,37 @@ describe Economic::OrderProxy do
     let(:unto) { Time.now }
 
     it "should be able to return a single current order" do
-      mock_request('Order_FindByDateInterval', {'first' => from.iso8601, 'last' => unto.iso8601}, :single)
-      mock_request('Order_GetDataArray', :any, :single)
+      mock_request(
+        "Order_FindByDateInterval",
+        {"first" => from.iso8601, "last" => unto.iso8601},
+        :single
+      )
+      mock_request("Order_GetDataArray", :any, :single)
       results = subject.find_by_date_interval(from, unto)
       expect(results.size).to eq(1)
       expect(results.first).to be_instance_of(Economic::Order)
     end
 
     it "should be able to return multiple orders" do
-      mock_request('Order_FindByDateInterval', {'first' => from.iso8601, 'last' => unto.iso8601}, :many)
-      mock_request('Order_GetDataArray', :any, :multiple)
+      mock_request(
+        "Order_FindByDateInterval",
+        {"first" => from.iso8601, "last" => unto.iso8601},
+        :many
+      )
+      mock_request("Order_GetDataArray", :any, :multiple)
       results = subject.find_by_date_interval(from, unto)
       expect(results.size).to eq(2)
       expect(results.first).to be_instance_of(Economic::Order)
     end
 
     it "should be able to return nothing" do
-      mock_request('Order_FindByDateInterval', {'first' => from.iso8601, 'last' => unto.iso8601}, :none)
+      mock_request(
+        "Order_FindByDateInterval",
+        {"first" => from.iso8601, "last" => unto.iso8601},
+        :none
+      )
       results = subject.find_by_date_interval(from, unto)
       expect(results.size).to eq(0)
     end
-
   end
 end

data/spec/economic/proxies/product_proxy_spec.rb

@@ -0,0 +1,86 @@
+require "./spec/spec_helper"
+
+describe Economic::ProductProxy do
+  let(:session) { make_session }
+  subject { Economic::ProductProxy.new(session) }
+
+  describe "new" do
+    it "stores session" do
+      expect(subject.session).to equal(session)
+    end
+  end
+
+  describe "find" do
+    it "gets product data from API" do
+      mock_request(
+        "Product_GetData",
+        {"entityHandle" => {"Number" => 42}},
+        :success
+      )
+      subject.find(42)
+    end
+
+    it "returns Product object" do
+      stub_request("Product_GetData", nil, :success)
+      expect(subject.find(42)).to be_instance_of(Economic::Product)
+    end
+  end
+
+  describe "find_by_number" do
+    it "can find a product" do
+      mock_request("Product_FindByNumber", {"number" => "1"}, :found)
+      result = subject.find_by_number("1")
+      expect(result).to be_instance_of(Economic::Product)
+      expect(result.number).to eq("1")
+      expect(result.partial).to be_truthy
+      expect(result.persisted).to be_truthy
+      expect(result.handle).to eq(Economic::Entity::Handle.new(:number => "1"))
+    end
+
+    it "returns nil when there is no product" do
+      mock_request("Product_FindByNumber", {"number" => "1"}, :not_found)
+      result = subject.find_by_number("1")
+      expect(result).to be_nil
+    end
+  end
+
+  describe "build" do
+    subject { session.products.build }
+
+    it "instantiates a new Product" do
+      expect(subject).to be_instance_of(Economic::Product)
+    end
+
+    it "assigns the session to the Product" do
+      expect(subject.session).to equal(session)
+    end
+  end
+
+  describe "#entity_class" do
+    it "should return Economic::Product" do
+      expect(Economic::ProductProxy.entity_class).to eq(Economic::Product)
+    end
+  end
+
+  describe ".all" do
+    it "returns a single product" do
+      stub_request("Product_GetAll", nil, :single)
+      mock_request(
+        "Product_GetData",
+        {"entityHandle" => {"Number" => "1"}},
+        :success
+      )
+      all = subject.all
+      expect(all.size).to eq(1)
+      expect(all.first).to be_instance_of(Economic::Product)
+    end
+
+    it "returns multiple products" do
+      mock_request("Product_GetAll", nil, :multiple)
+      stub_request("Product_GetDataArray", nil, :multiple)
+      all = subject.all
+      expect(all.size).to eq(2)
+      expect(all.first).to be_instance_of(Economic::Product)
+    end
+  end
+end

data/spec/economic/session_spec.rb

@@ -1,32 +1,43 @@
-require './spec/spec_helper'
+require "./spec/spec_helper"
 
 describe Economic::Session do
-  let(:credentials) { [123456, 'api', 'passw0rd'] }
+  let(:credentials) { [123_456, "api", "passw0rd"] }
   subject { Economic::Session.new }
 
   let(:endpoint) { subject.endpoint }
 
   describe "new" do
     describe "legacy connect" do
-      subject { Economic::Session.new *credentials }
+      subject { Economic::Session.new(*credentials) }
       it "should store authentication details" do
-        expect(subject.agreement_number).to eq(123456)
-        expect(subject.user_name).to eq('api')
-        expect(subject.password).to eq('passw0rd')
+        expect(subject.agreement_number).to eq(123_456)
+        expect(subject.user_name).to eq("api")
+        expect(subject.password).to eq("passw0rd")
+      end
+
+      it "can also store an app_id" do
+        session = Economic::Session.new(*credentials, "app_id")
+        expect(session.app_identifier).to eq("app_id")
       end
     end
 
     it "yields the endpoint if a block is given" do
       endpoint_mock = double("Endpoint")
-      Economic::Endpoint.stub(:new).and_return(endpoint_mock)
-      expect{|b|
-        Economic::Session.new(123456, 'api', 'passw0rd', &b)
+      allow(Economic::Endpoint).to receive(:new).and_return(endpoint_mock)
+      expect {|b|
+        Economic::Session.new(123_456, "api", "passw0rd", &b)
       }.to yield_with_args(endpoint_mock)
     end
   end
 
   describe "connect" do
-    let(:authentication_details) { {:agreementNumber => 123456, :userName => 'api', :password => 'passw0rd'} }
+    let(:authentication_details) {
+      {
+        :agreementNumber => 123_456,
+        :userName => "api",
+        :password => "passw0rd"
+      }
+    }
 
     it "can connect old-style" do
       mock_request(:connect, authentication_details, :success)
@@ -41,30 +52,38 @@ describe Economic::Session do
 
     it "stores the authentication token for later requests" do
       response = {
-        :headers => {'Set-Cookie' => 'cookie value from e-conomic'},
+        :headers => {"Set-Cookie" => "cookie value from e-conomic"},
         :body => fixture(:connect, :success)
       }
-      stub_request('Connect', authentication_details, response)
+      stub_request("Connect", authentication_details, response)
 
       subject.connect_with_credentials(*credentials)
 
-      expect(subject.authentication_token.collect { |cookie|
+      expect(subject.authentication_cookies.collect { |cookie|
         cookie.name_and_value.split("=").last
       }).to eq(["cookie value from e-conomic"])
     end
 
     it "updates the authentication token for new sessions" do
-      stub_request("Connect", nil, {:headers => {"Set-Cookie" => "authentication token"}})
+      stub_request(
+        "Connect",
+        nil,
+        :headers => {"Set-Cookie" => "authentication token"}
+      )
       subject.connect_with_credentials(*credentials)
 
-      stub_request('Connect', nil, {:headers => {"Set-Cookie" => "another token"}})
+      stub_request(
+        "Connect",
+        nil,
+        :headers => {"Set-Cookie" => "another token"}
+      )
       other_session = Economic::Session.new
-      other_session.connect_with_credentials(123456, 'api', 'passw0rd')
+      other_session.connect_with_credentials(123_456, "api", "passw0rd")
 
-      expect(subject.authentication_token.collect { |cookie|
+      expect(subject.authentication_cookies.collect { |cookie|
         cookie.name_and_value.split("=").last
       }).to eq(["authentication token"])
-      expect(other_session.authentication_token.collect { |cookie|
+      expect(other_session.authentication_cookies.collect { |cookie|
         cookie.name_and_value.split("=").last
       }).to eq(["another token"])
     end
@@ -77,7 +96,12 @@ describe Economic::Session do
 
   describe "connecting with access ID" do
     # As per http://www.e-conomic.com/developer/tutorials
-    let(:authentication_details) { {:appToken => 'the_private_app_id', :token => 'the_access_id_you_got_from_the_grant'} }
+    let(:authentication_details) {
+      {
+        :appToken => "the_private_app_id",
+        :token => "the_access_id_you_got_from_the_grant"
+      }
+    }
     let(:private_app_id) { authentication_details[:appToken] }
     let(:access_id) { authentication_details[:token] }
     subject { Economic::Session.new }
@@ -88,36 +112,47 @@ describe Economic::Session do
 
     it "stores the authentication token for later requests" do
       response = {
-        :headers => {'Set-Cookie' => 'cookie value from e-conomic'},
+        :headers => {"Set-Cookie" => "cookie value from e-conomic"},
         :body => fixture(:connect_with_token, :success)
       }
-      stub_request('ConnectWithToken', authentication_details, response)
+      stub_request("ConnectWithToken", authentication_details, response)
 
       subject.connect_with_token private_app_id, access_id
 
-      expect(subject.authentication_token.collect { |cookie|
+      expect(subject.authentication_cookies.collect { |cookie|
         cookie.name_and_value.split("=").last
       }).to eq(["cookie value from e-conomic"])
     end
 
     it "updates the authentication token for new sessions" do
-      stub_request("ConnectWithToken", nil, {:headers => {"Set-Cookie" => "authentication token"}})
+      stub_request(
+        "ConnectWithToken",
+        nil,
+        :headers => {"Set-Cookie" => "authentication token"}
+      )
       subject.connect_with_token private_app_id, access_id
 
-      stub_request('Connect', nil, {:headers => {"Set-Cookie" => "another token"}})
+      stub_request(
+        "Connect",
+        nil,
+        :headers => {"Set-Cookie" => "another token"}
+      )
       other_session = Economic::Session.new
-      other_session.connect_with_credentials(123456, 'api', 'passw0rd')
+      other_session.connect_with_credentials(123_456, "api", "passw0rd")
 
-      expect(subject.authentication_token.collect { |cookie|
+      expect(subject.authentication_cookies.collect { |cookie|
         cookie.name_and_value.split("=").last
       }).to eq(["authentication token"])
-      expect(other_session.authentication_token.collect { |cookie|
+      expect(other_session.authentication_cookies.collect { |cookie|
         cookie.name_and_value.split("=").last
       }).to eq(["another token"])
     end
 
     it "doesn't use existing authentication details when connecting" do
-      expect(endpoint).to receive(:call).with(:connect_with_token, instance_of(Hash))
+      expect(endpoint).to receive(:call).with(
+        :connect_with_token,
+        instance_of(Hash)
+      )
       subject.connect_with_token private_app_id, access_id
     end
   end
@@ -146,7 +181,9 @@ describe Economic::Session do
 
   describe "current_invoices" do
     it "returns an CurrentInvoiceProxy" do
-      expect(subject.current_invoices).to be_instance_of(Economic::CurrentInvoiceProxy)
+      expect(subject.current_invoices).to be_instance_of(
+        Economic::CurrentInvoiceProxy
+      )
     end
 
     it "memoizes the proxy" do
@@ -186,12 +223,14 @@ describe Economic::Session do
 
     it "sends data if given" do
       mock_request(:current_invoice_get_all, {:bar => :baz}, :none)
-      subject.request(:current_invoice_get_all, {:bar => :baz})
+      subject.request(:current_invoice_get_all, :bar => :baz)
     end
 
     it "returns a hash with data" do
       stub_request(:current_invoice_get_all, nil, :single)
-      expect(subject.request(:current_invoice_get_all)).to eq({:current_invoice_handle => {:id => "1"}})
+      expect(subject.request(:current_invoice_get_all)).to eq(
+        :current_invoice_handle => {:id => "1"}
+      )
     end
 
     it "returns an empty hash if no data returned" do
@@ -204,22 +243,22 @@ describe Economic::Session do
     let(:endpoint) { double("Endpoint") }
 
     before :each do
-      Economic::Endpoint.stub(:new).and_return(endpoint)
+      allow(Economic::Endpoint).to receive(:new).and_return(endpoint)
     end
 
     it "sets the log_level option of the endpoint" do
-      endpoint.should_receive(:log_level=).with(:info)
+      expect(endpoint).to receive(:log_level=).with(:info)
       subject.log_level = :info
     end
 
     it "sets the log option of the endpoint" do
-      endpoint.should_receive(:log=).with(true)
+      expect(endpoint).to receive(:log=).with(true)
       subject.log = true
     end
 
     it "sets the logger option of the boolean" do
       logger = double("MyLogger")
-      endpoint.should_receive(:logger=).with(logger)
+      expect(endpoint).to receive(:logger=).with(logger)
       subject.logger = logger
     end
   end

data/spec/fixtures/account_get_all/multiple.xml

@@ -0,0 +1,15 @@
+<?xml version="1.0" encoding="utf-8"?>
+<soap:Envelope xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:xsd="http://www.w3.org/2001/XMLSchema" xmlns:soap="http://schemas.xmlsoap.org/soap/envelope/">
+  <soap:Body>
+    <Account_GetAllResponse xmlns="http://e-conomic.com">
+      <Account_GetAllResult>
+        <AccountHandle>
+          <Number>37</Number>
+        </AccountHandle>
+        <AccountHandle>
+          <Number>42</Number>
+        </AccountHandle>
+      </Account_GetAllResult>
+    </Account_GetAllResponse>
+  </soap:Body>
+</soap:Envelope>

data/spec/fixtures/account_get_all/none.xml

@@ -0,0 +1,8 @@
+<?xml version="1.0" encoding="utf-8"?>
+<soap:Envelope xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:xsd="http://www.w3.org/2001/XMLSchema" xmlns:soap="http://schemas.xmlsoap.org/soap/envelope/">
+  <soap:Body>
+    <Account_GetAllResponse xmlns="http://e-conomic.com">
+      <Account_GetAllResult />
+    </Account_GetAllResponse>
+  </soap:Body>
+</soap:Envelope>

data/spec/fixtures/account_get_all/single.xml

@@ -0,0 +1,12 @@
+<?xml version="1.0" encoding="utf-8"?>
+<soap:Envelope xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:xsd="http://www.w3.org/2001/XMLSchema" xmlns:soap="http://schemas.xmlsoap.org/soap/envelope/">
+  <soap:Body>
+    <Account_GetAllResponse xmlns="http://e-conomic.com">
+      <Account_GetAllResult>
+        <AccountHandle>
+          <Number>1</Number>
+        </AccountHandle>
+      </Account_GetAllResult>
+    </Account_GetAllResponse>
+  </soap:Body>
+</soap:Envelope>

data/spec/fixtures/account_get_data_array/multiple.xml

@@ -0,0 +1,69 @@
+<?xml version="1.0" encoding="utf-8"?>
+<soap:Envelope xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:xsd="http://www.w3.org/2001/XMLSchema" xmlns:soap="http://schemas.xmlsoap.org/soap/envelope/">
+  <soap:Body>
+    <Account_GetDataArrayResponse xmlns="http://e-conomic.com">
+      <Account_GetDataArrayResult>
+        <AccountData>
+          <Handle>
+            <Number>37</Number>
+          </Handle>
+          <Number>37</Number>
+          <Name>Some account</Name>
+          <Type>ProfitAndLoss or Status or TotalFrom or Heading or HeadingStart or SumInterval or SumAlpha</Type>
+          <DebitCredit>Debit or Credit</DebitCredit>
+          <IsAccessible>boolean</IsAccessible>
+          <BlockDirectEntries>boolean</BlockDirectEntries>
+          <VatAccountHandle>
+            <VatCode>string</VatCode>
+          </VatAccountHandle>
+          <ContraAccountHandle>
+            <Number>int</Number>
+          </ContraAccountHandle>
+          <OpeningAccountHandle>
+            <Number>int</Number>
+          </OpeningAccountHandle>
+          <TotalFromHandle>
+            <Number>int</Number>
+          </TotalFromHandle>
+          <Balance>decimal</Balance>
+          <DepartmentHandle>
+            <Number>int</Number>
+          </DepartmentHandle>
+          <DistributionKeyHandle>
+            <Number>int</Number>
+          </DistributionKeyHandle>
+        </AccountData>
+        <AccountData>
+          <Handle>
+            <Number>42</Number>
+          </Handle>
+          <Number>42</Number>
+          <Name>Another account</Name>
+          <Type>ProfitAndLoss or Status or TotalFrom or Heading or HeadingStart or SumInterval or SumAlpha</Type>
+          <DebitCredit>Debit or Credit</DebitCredit>
+          <IsAccessible>boolean</IsAccessible>
+          <BlockDirectEntries>boolean</BlockDirectEntries>
+          <VatAccountHandle>
+            <VatCode>string</VatCode>
+          </VatAccountHandle>
+          <ContraAccountHandle>
+            <Number>int</Number>
+          </ContraAccountHandle>
+          <OpeningAccountHandle>
+            <Number>int</Number>
+          </OpeningAccountHandle>
+          <TotalFromHandle>
+            <Number>int</Number>
+          </TotalFromHandle>
+          <Balance>decimal</Balance>
+          <DepartmentHandle>
+            <Number>int</Number>
+          </DepartmentHandle>
+          <DistributionKeyHandle>
+            <Number>int</Number>
+          </DistributionKeyHandle>
+        </AccountData>
+      </Account_GetDataArrayResult>
+    </Account_GetDataArrayResponse>
+  </soap:Body>
+</soap:Envelope>

data/spec/fixtures/company_get/success.xml

@@ -0,0 +1,10 @@
+<?xml version="1.0" encoding="utf-8"?>
+<soap12:Envelope xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:xsd="http://www.w3.org/2001/XMLSchema" xmlns:soap12="http://www.w3.org/2003/05/soap-envelope">
+  <soap12:Body>
+    <Company_GetResponse xmlns="http://e-conomic.com">
+      <Company_GetResult>
+        <Number>string</Number>
+      </Company_GetResult>
+    </Company_GetResponse>
+  </soap12:Body>
+</soap12:Envelope>

data/spec/fixtures/company_get_data/success.xml

@@ -0,0 +1,32 @@
+<?xml version="1.0" encoding="utf-8"?>
+<soap12:Envelope xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:xsd="http://www.w3.org/2001/XMLSchema" xmlns:soap12="http://www.w3.org/2003/05/soap-envelope">
+  <soap12:Body>
+    <Company_GetDataResponse xmlns="http://e-conomic.com">
+      <Company_GetDataResult>
+        <Handle>
+          <Number>string</Number>
+        </Handle>
+        <Number>string</Number>
+        <BaseCurrencyHandle>
+          <Code>string</Code>
+        </BaseCurrencyHandle>
+        <Name>string</Name>
+        <Address1>string</Address1>
+        <Address2>string</Address2>
+        <PostalCode>string</PostalCode>
+        <City>string</City>
+        <County>string</County>
+        <Country>string</Country>
+        <TelephoneNumber>string</TelephoneNumber>
+        <FaxNumber>string</FaxNumber>
+        <MobileNumber>string</MobileNumber>
+        <Contact>string</Contact>
+        <WebSite>string</WebSite>
+        <Email>string</Email>
+        <CINumber>string</CINumber>
+        <VatNumber>string</VatNumber>
+        <SignUpDate>dateTime</SignUpDate>
+      </Company_GetDataResult>
+    </Company_GetDataResponse>
+  </soap12:Body>
+</soap12:Envelope>

data/spec/fixtures/current_invoice_line_create_from_data/success.xml

@@ -1,42 +1,11 @@
 <?xml version="1.0" encoding="utf-8"?>
 <soap:Envelope xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:xsd="http://www.w3.org/2001/XMLSchema" xmlns:soap="http://schemas.xmlsoap.org/soap/envelope/">
   <soap:Body>
-    <CurrentInvoiceLine_CreateFromData xmlns="http://e-conomic.com">
-      <data>
-        <Handle>
-          <Id>int</Id>
-          <Number>int</Number>
-        </Handle>
-        <Id>int</Id>
-        <Number>int</Number>
-        <InvoiceHandle>
-          <Id>int</Id>
-        </InvoiceHandle>
-        <Description>string</Description>
-        <DeliveryDate>dateTime</DeliveryDate>
-        <UnitHandle>
-          <Number>int</Number>
-        </UnitHandle>
-        <ProductHandle>
-          <Number>string</Number>
-        </ProductHandle>
-        <Quantity>decimal</Quantity>
-        <UnitNetPrice>decimal</UnitNetPrice>
-        <DiscountAsPercent>decimal</DiscountAsPercent>
-        <UnitCostPrice>decimal</UnitCostPrice>
-        <TotalNetAmount>decimal</TotalNetAmount>
-        <TotalMargin>decimal</TotalMargin>
-        <MarginAsPercent>decimal</MarginAsPercent>
-        <DepartmentHandle>
-          <Number>int</Number>
-        </DepartmentHandle>
-        <DistributionKeyHandle>
-          <Number>int</Number>
-        </DistributionKeyHandle>
-        <InventoryLocationHandle>
-          <Number>int</Number>
-        </InventoryLocationHandle>
-      </data>
-    </CurrentInvoiceLine_CreateFromData>
+    <CurrentInvoiceLine_CreateFromDataResponse xmlns="http://e-conomic.com">
+      <CurrentInvoiceLine_CreateFromDataResult>
+        <Id>42</Id>
+        <Number>42</Number>
+      </CurrentInvoiceLine_CreateFromDataResult>
+    </CurrentInvoiceLine_CreateFromDataResponse>
   </soap:Body>
 </soap:Envelope>