czech_post_b2b_client 1.2.4 → 1.2.5

Sign up to get free protection for your applications and to get access to all the features.
@@ -50,9 +50,7 @@ module CzechPostB2bClient
50
50
  end
51
51
 
52
52
  def response_root_node
53
- if response_service_data.keys.include?(response_root_node_name)
54
- return response_service_data.dig(response_root_node_name)
55
- end
53
+ return response_service_data.dig(response_root_node_name) if response_service_data.keys.include?(response_root_node_name)
56
54
 
57
55
  errors.add(:xml, "Cannot find `#{response_root_node_name}` in `serviceData` node.")
58
56
  fail_on_structure_parsing
@@ -48,9 +48,7 @@ module CzechPostB2bClient
48
48
  # more parcel_codes for one parcel_id => probably errors
49
49
  old_p_code = value[:parcel_code]
50
50
  new_p_code = pd_hash[:parcel_code]
51
- if old_p_code != new_p_code
52
- raise "Two different parcel_codes [#{old_p_code}, #{new_p_code}] for parcel_id:'#{parcel_id}'"
53
- end
51
+ raise "Two different parcel_codes [#{old_p_code}, #{new_p_code}] for parcel_id:'#{parcel_id}'" if old_p_code != new_p_code
54
52
 
55
53
  value
56
54
  end
@@ -71,9 +71,7 @@ module CzechPostB2bClient
71
71
  # more parcel_codes for one parcel_id => probably errors
72
72
  old_p_code = value[:parcel_code]
73
73
  new_p_code = pd_hash[:parcel_code]
74
- if old_p_code != new_p_code
75
- raise "Two different parcel_codes [#{old_p_code}, #{new_p_code}] for parcel_id:'#{parcel_id}'"
76
- end
74
+ raise "Two different parcel_codes [#{old_p_code}, #{new_p_code}] for parcel_id:'#{parcel_id}'" if old_p_code != new_p_code
77
75
 
78
76
  value
79
77
  end
@@ -55,7 +55,8 @@ module CzechPostB2bClient
55
55
  request = Net::HTTP::Post.new service_uri.request_uri, headers
56
56
  request.body = request_xml
57
57
 
58
- debug_msg = "CzechPost B2B REQUEST: #{request} to #{service_uri.request_uri} with body:\n#{request.body}"
58
+ debug_msg = "CzechPost B2B REQUEST: #{request} to #{service_uri.request_uri}" \
59
+ " with headers: #{headers}\n and body:\n#{request.body}"
59
60
  CzechPostB2bClient.logger.debug(debug_msg)
60
61
 
61
62
  request
@@ -106,7 +107,8 @@ module CzechPostB2bClient
106
107
  return 'error code not found in XML' unless error_match
107
108
 
108
109
  error_code = error_match[1].to_i
109
- error = CzechPostB2bClient::B2BErrors.new_by_code(error_code)
110
+ error_details = result.xml.match(%r{<(?:\w+\:)?errorDescription>(.*)</(?:\w+\:)?errorDescription>})
111
+ error = CzechPostB2bClient::B2BErrors.new_by_code(error_code, error_details[1])
110
112
  return "error code [#{error_code}] is unknown" unless error
111
113
 
112
114
  error.message
@@ -63,7 +63,7 @@ module CzechPostB2bClient
63
63
  response_code = response_state[:code]
64
64
  next if response_code == CzechPostB2bClient::ResponseCodes::Ok.code
65
65
 
66
- errors.add(:parcels, "Parcel[#{parcel_id}] => #{CzechPostB2bClient::ResponseCodes.new_by_code(response_code)}") # rubocop:disable Layout/LineLength
66
+ errors.add(:parcels, "Parcel[#{parcel_id}] => #{CzechPostB2bClient::ResponseCodes.new_by_code(response_code)}")
67
67
  end
68
68
  end
69
69
  end
@@ -80,7 +80,7 @@ module CzechPostB2bClient
80
80
  response_code = response_state[:code]
81
81
  next if response_code == CzechPostB2bClient::ResponseCodes::Ok.code
82
82
 
83
- errors.add(:parcels, "Parcel[#{parcel_id}] => #{CzechPostB2bClient::ResponseCodes.new_by_code(response_code)}") # rubocop:disable Layout/LineLength
83
+ errors.add(:parcels, "Parcel[#{parcel_id}] => #{CzechPostB2bClient::ResponseCodes.new_by_code(response_code)}")
84
84
  end
85
85
  end
86
86
  end
@@ -1,5 +1,5 @@
1
1
  # frozen_string_literal: true
2
2
 
3
3
  module CzechPostB2bClient
4
- VERSION = '1.2.4'
4
+ VERSION = '1.2.5'
5
5
  end
metadata CHANGED
@@ -1,7 +1,7 @@
1
1
  --- !ruby/object:Gem::Specification
2
2
  name: czech_post_b2b_client
3
3
  version: !ruby/object:Gem::Version
4
- version: 1.2.4
4
+ version: 1.2.5
5
5
  platform: ruby
6
6
  authors:
7
7
  - Petr Mlčoch