Skip to content

Commit 6c75609

Browse files
jon-signaljon-signal
committed
fix!: add BigDecimal getters for prices and deprecate double getters
1 parent 3ab9289 commit 6c75609

10 files changed

+223
-23
lines changed

‎src/main/java/com/twilio/type/InboundCallPrice.java

+43-4
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
import com.twilio.converter.Promoter;
77
import lombok.ToString;
88

9+
import java.math.BigDecimal;
910
import java.util.Objects;
1011

1112
/**
@@ -41,8 +42,8 @@ public static Type forValue(final String value) {
4142
}
4243
}
4344

44-
private final double basePrice;
45-
private final double currentPrice;
45+
private final BigDecimal basePrice;
46+
private final BigDecimal currentPrice;
4647
private final Type type;
4748

4849
/**
@@ -53,19 +54,57 @@ public static Type forValue(final String value) {
5354
* @param type type of phone number
5455
*/
5556
@JsonCreator
56-
public InboundCallPrice(@JsonProperty("base_price") final double basePrice,
57-
@JsonProperty("current_price") final double currentPrice,
57+
public InboundCallPrice(@JsonProperty("base_price") final BigDecimal basePrice,
58+
@JsonProperty("current_price") final BigDecimal currentPrice,
5859
@JsonProperty("number_type") final Type type) {
5960
this.basePrice = basePrice;
6061
this.currentPrice = currentPrice;
6162
this.type = type;
6263
}
6364

65+
/**
66+
* Returns the retail price per minute to receive a call to this phone number type. The value returned by this
67+
* method is represented as a {@code double}, which may result in loss of precision.
68+
*
69+
* @return the retail price per minute to receive a call to this phone number type
70+
*
71+
* @deprecated please use {{@link #getBasePriceDecimal()}} instead for a lossless representation of the price
72+
*/
73+
@Deprecated
6474
public double getBasePrice() {
75+
return basePrice.doubleValue();
76+
}
77+
78+
/**
79+
* Returns the retail price per minute to receive a call to this phone number type.
80+
*
81+
* @return the retail price per minute to receive a call to this phone number type
82+
*/
83+
public BigDecimal getBasePriceDecimal() {
6584
return basePrice;
6685
}
6786

87+
/**
88+
* Returns the current price per minute (which accounts for any volume or custom price discounts) to receive a call
89+
* to this phone number type. The value returned by this method is represented as a {@code double}, which may result
90+
* in loss of precision.
91+
*
92+
* @return the current price per minute to receive a call to this phone number type
93+
*
94+
* @deprecated please use {{@link #getCurrentPriceDecimal()} instead for a lossless representation of the price
95+
*/
96+
@Deprecated
6897
public double getCurrentPrice() {
98+
return currentPrice.doubleValue();
99+
}
100+
101+
/**
102+
* Returns the current price per minute (which accounts for any volume or custom price discounts) to receive a call
103+
* to this phone number type.
104+
*
105+
* @return the current price per minute to receive a call to this phone number type
106+
*/
107+
public BigDecimal getCurrentPriceDecimal() {
69108
return currentPrice;
70109
}
71110

‎src/main/java/com/twilio/type/InboundSmsPrice.java

+41-4
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
import com.twilio.converter.Promoter;
77
import lombok.ToString;
88

9+
import java.math.BigDecimal;
910
import java.util.Objects;
1011

1112
/**
@@ -42,8 +43,8 @@ public static Type forValue(final String value) {
4243
}
4344
}
4445

45-
private final double basePrice;
46-
private final double currentPrice;
46+
private final BigDecimal basePrice;
47+
private final BigDecimal currentPrice;
4748
private final Type type;
4849

4950
/**
@@ -54,19 +55,55 @@ public static Type forValue(final String value) {
5455
* @param type type of phone number
5556
*/
5657
@JsonCreator
57-
public InboundSmsPrice(@JsonProperty("base_price") final double basePrice,
58-
@JsonProperty("current_price") final double currentPrice,
58+
public InboundSmsPrice(@JsonProperty("base_price") final BigDecimal basePrice,
59+
@JsonProperty("current_price") final BigDecimal currentPrice,
5960
@JsonProperty("number_type") final Type type) {
6061
this.basePrice = basePrice;
6162
this.currentPrice = currentPrice;
6263
this.type = type;
6364
}
6465

66+
/**
67+
* Returns the retail price to receive a message. The value returned by this method is represented as a
68+
* {@code double}, which may result in loss of precision.
69+
*
70+
* @return the retail price to receive a message
71+
*
72+
* @deprecated please use {{@link #getBasePriceDecimal()}} instead for a lossless representation of the price
73+
*/
74+
@Deprecated
6575
public double getBasePrice() {
76+
return basePrice.doubleValue();
77+
}
78+
79+
/**
80+
* Returns the retail price to receive a message.
81+
*
82+
* @return the retail price to receive a message
83+
*/
84+
public BigDecimal getBasePriceDecimal() {
6685
return basePrice;
6786
}
6887

88+
/**
89+
* Returns the current price (which accounts for any volume or custom price discounts) to receive a message. The
90+
* value returned by this method is represented as a {@code double}, which may result in loss of precision.
91+
*
92+
* @return the current price to receive a message
93+
*
94+
* @deprecated please use {{@link #getCurrentPriceDecimal()} instead for a lossless representation of the price
95+
*/
96+
@Deprecated
6997
public double getCurrentPrice() {
98+
return currentPrice.doubleValue();
99+
}
100+
101+
/**
102+
* Returns the current price (which accounts for any volume or custom price discounts) to receive a message.
103+
*
104+
* @return the current price to receive a message
105+
*/
106+
public BigDecimal getCurrentPriceDecimal() {
70107
return currentPrice;
71108
}
72109

‎src/main/java/com/twilio/type/OutboundCallPrice.java

+43-4
Original file line numberDiff line numberDiff line change
@@ -5,26 +5,65 @@
55
import com.fasterxml.jackson.annotation.JsonProperty;
66
import lombok.ToString;
77

8+
import java.math.BigDecimal;
89
import java.util.Objects;
910

1011
@JsonIgnoreProperties(ignoreUnknown = true)
1112
@ToString
1213
public class OutboundCallPrice {
13-
private final double basePrice;
14-
private final double currentPrice;
14+
private final BigDecimal basePrice;
15+
private final BigDecimal currentPrice;
1516

1617
@JsonCreator
17-
public OutboundCallPrice(@JsonProperty("base_price") final double basePrice,
18-
@JsonProperty("current_price") final double currentPrice) {
18+
public OutboundCallPrice(@JsonProperty("base_price") final BigDecimal basePrice,
19+
@JsonProperty("current_price") final BigDecimal currentPrice) {
1920
this.basePrice = basePrice;
2021
this.currentPrice = currentPrice;
2122
}
2223

24+
/**
25+
* Returns the retail price per minute to make a call from this phone number type. The value returned by this
26+
* method is represented as a {@code double}, which may result in loss of precision.
27+
*
28+
* @return the retail price per minute to make a call from this phone number type
29+
*
30+
* @deprecated please use {{@link #getBasePriceDecimal()}} instead for a lossless representation of the price
31+
*/
32+
@Deprecated
2333
public double getBasePrice() {
34+
return basePrice.doubleValue();
35+
}
36+
37+
/**
38+
* Returns the retail price per minute to make a call from this phone number type.
39+
*
40+
* @return the retail price per minute to make a call from this phone number type
41+
*/
42+
public BigDecimal getBasePriceDecimal() {
2443
return basePrice;
2544
}
2645

46+
/**
47+
* Returns the current price per minute (which accounts for any volume or custom price discounts) to make a call
48+
* from this phone number type. The value returned by this method is represented as a {@code double}, which may
49+
* result in loss of precision.
50+
*
51+
* @return the current price per minute to make a call from this phone number type
52+
*
53+
* @deprecated please use {{@link #getCurrentPriceDecimal()} instead for a lossless representation of the price
54+
*/
55+
@Deprecated
2756
public double getCurrentPrice() {
57+
return currentPrice.doubleValue();
58+
}
59+
60+
/**
61+
* Returns the current price per minute (which accounts for any volume or custom price discounts) to make a call
62+
* from this phone number type.
63+
*
64+
* @return the current price per minute to make a call from this phone number type
65+
*/
66+
public BigDecimal getCurrentPriceDecimal() {
2867
return currentPrice;
2968
}
3069

‎src/main/java/com/twilio/type/OutboundPrefixPrice.java

+43-4
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55
import com.fasterxml.jackson.annotation.JsonProperty;
66
import lombok.ToString;
77

8+
import java.math.BigDecimal;
89
import java.util.List;
910
import java.util.Objects;
1011

@@ -22,8 +23,8 @@ public class OutboundPrefixPrice {
2223

2324
private final List<String> prefixes;
2425
private final String friendlyName;
25-
private final double basePrice;
26-
private final double currentPrice;
26+
private final BigDecimal basePrice;
27+
private final BigDecimal currentPrice;
2728

2829
/**
2930
* Initialize an OutboundPrefixPrice.
@@ -36,8 +37,8 @@ public class OutboundPrefixPrice {
3637
@JsonCreator
3738
public OutboundPrefixPrice(@JsonProperty("prefixes") final List<String> prefixes,
3839
@JsonProperty("friendly_name") final String friendlyName,
39-
@JsonProperty("base_price") final double basePrice,
40-
@JsonProperty("current_price") final double currentPrice) {
40+
@JsonProperty("base_price") final BigDecimal basePrice,
41+
@JsonProperty("current_price") final BigDecimal currentPrice) {
4142
this.prefixes = prefixes;
4243
this.friendlyName = friendlyName;
4344
this.basePrice = basePrice;
@@ -52,11 +53,49 @@ public String getFriendlyName() {
5253
return friendlyName;
5354
}
5455

56+
/**
57+
* Returns the retail price per minute to make a call to numbers matching this prefix list. The value returned by
58+
* this method is represented as a {@code double}, which may result in loss of precision.
59+
*
60+
* @return the retail price per minute to make a call to numbers matching this prefix list
61+
*
62+
* @deprecated please use {{@link #getBasePriceDecimal()}} instead for a lossless representation of the price
63+
*/
64+
@Deprecated
5565
public double getBasePrice() {
66+
return basePrice.doubleValue();
67+
}
68+
69+
/**
70+
* Returns the retail price per minute to make a call to numbers matching this prefix list.
71+
*
72+
* @return the retail price per minute to make a call to numbers matching this prefix list
73+
*/
74+
public BigDecimal getBasePriceDecimal() {
5675
return basePrice;
5776
}
5877

78+
/**
79+
* Returns the current price per minute (which accounts for any volume or custom price discounts) to make a call to
80+
* numbers matching this prefix list. The value returned by this method is represented as a {@code double}, which
81+
* may result in loss of precision.
82+
*
83+
* @return the current price per minute to make a call to numbers matching this prefix list
84+
*
85+
* @deprecated please use {{@link #getCurrentPriceDecimal()} instead for a lossless representation of the price
86+
*/
87+
@Deprecated
5988
public double getCurrentPrice() {
89+
return currentPrice.doubleValue();
90+
}
91+
92+
/**
93+
* Returns the current price per minute (which accounts for any volume or custom price discounts) to make a call to
94+
* numbers matching this prefix list.
95+
*
96+
* @return the current price per minute to make a call to numbers matching this prefix list
97+
*/
98+
public BigDecimal getCurrentPriceDecimal() {
6099
return currentPrice;
61100
}
62101

‎src/main/java/com/twilio/type/PhoneNumberPrice.java

+32-6
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
import com.twilio.converter.Promoter;
77
import lombok.ToString;
88

9+
import java.math.BigDecimal;
910
import java.util.Objects;
1011

1112
/**
@@ -56,8 +57,8 @@ public static Type forValue(final String value) {
5657
}
5758
}
5859

59-
private final double basePrice;
60-
private final double currentPrice;
60+
private final BigDecimal basePrice;
61+
private final BigDecimal currentPrice;
6162
private final Type type;
6263

6364
/**
@@ -68,29 +69,54 @@ public static Type forValue(final String value) {
6869
* @param type type of phone number
6970
*/
7071
@JsonCreator
71-
public PhoneNumberPrice(@JsonProperty("base_price") final double basePrice,
72-
@JsonProperty("current_price") final double currentPrice,
72+
public PhoneNumberPrice(@JsonProperty("base_price") final BigDecimal basePrice,
73+
@JsonProperty("current_price") final BigDecimal currentPrice,
7374
@JsonProperty("number_type") final Type type) {
7475
this.basePrice = basePrice;
7576
this.currentPrice = currentPrice;
7677
this.type = type;
7778
}
7879

7980
/**
80-
* Returns the base price of the phone number.
81+
* Returns the base price of the phone number. The value returned by this method is represented as a {@code double},
82+
* which may result in loss of precision.
8183
*
8284
* @return the base price of the phone number
85+
*
86+
* @deprecated please use {{@link #getBasePriceDecimal()} instead for a lossless representation of the price
8387
*/
88+
@Deprecated
8489
public double getBasePrice() {
90+
return basePrice.doubleValue();
91+
}
92+
93+
/**
94+
* Returns the base price of the phone number.
95+
*
96+
* @return the base price of the phone number
97+
*/
98+
public BigDecimal getBasePriceDecimal() {
8599
return basePrice;
86100
}
87101

88102
/**
89-
* Returns the current price of the phone number.
103+
* Returns the current price of the phone number. The value returned by this method is represented as a
104+
* {@code double}, which may result in loss of precision.
90105
*
91106
* @return the current price of the phone number
107+
*
108+
* @deprecated please use {{@link #getCurrentPriceDecimal()} instead for a lossless representation of the price
92109
*/
110+
@Deprecated
93111
public double getCurrentPrice() {
112+
return currentPrice.doubleValue();
113+
}
114+
115+
/**
116+
* Returns the current price of the phone number.
117+
*
118+
* @return the current price of the phone number
119+
*/ public BigDecimal getCurrentPriceDecimal() {
94120
return currentPrice;
95121
}
96122

0 commit comments

Comments
 (0)