-
Notifications
You must be signed in to change notification settings - Fork 132
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix PHP 8.2 and Other Related Issues #640
base: master
Are you sure you want to change the base?
Changes from 1 commit
76dc113
591629c
b81fb0b
fad9d56
f4e69f4
c6138da
d580082
959b1a7
475e87f
dd38b0f
6da6e4f
6c17547
a096ea6
ba9a468
96d67ae
c1f8164
ce7d988
87bfb35
c31570d
a12582a
aa87640
156a26d
089bff7
964ee7e
596cca5
cfe79c5
d77e72b
85d3080
27520ac
d3f2448
6dc14ea
0149fb7
f205667
275dfda
87f0929
ed80f9f
6b9f10f
f5303e0
3f3e135
91ca58a
586b770
0268830
beac573
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
- Loading branch information
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -60,6 +60,32 @@ class PayIn extends Transaction | |
*/ | ||
public $RecurringPayinRegistrationId; | ||
|
||
/** | ||
* The IP address of the end user initiating the transaction, in IPV4 or IPV6 format. | ||
* @var string | ||
*/ | ||
public $IpAddress; | ||
|
||
/** | ||
* Information about the browser used by the end user (author) to perform the payment. | ||
* @var BrowserInfo | ||
*/ | ||
public $BrowserInfo; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. What is the reason for adding this to the PayIn class? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This property was identified in both the documentation and PHP SDK. I verified its existence through debug logs and implemented it accordingly. |
||
|
||
/** | ||
* Information about the end user billing address. If left empty, the default values will be automatically taken into account. | ||
* Default values: FirstName, LastName, and Address information of the Shipping object if any, otherwise the user (author). | ||
* @var Billing | ||
*/ | ||
public $Billing; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. What is the reason for adding this to the PayIn class? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This property was identified in both the documentation and PHP SDK. I verified its existence through debug logs and implemented it accordingly. |
||
|
||
/** | ||
* Information about the end user’s shipping address. If left empty, the default values will be automatically taken into account. | ||
* Default values: FirstName, LastName, and Address information of the Billing object, if supplied, otherwise of the user (author). | ||
* @var Shipping | ||
*/ | ||
public $Shipping; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. What is the reason for adding this to the PayIn class? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This property was identified in both the documentation and PHP SDK. I verified its existence through debug logs and implemented it accordingly. |
||
|
||
/** | ||
* Get array with mapping which property depends on other property | ||
* @return array | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is the reason for adding this to the PayIn class?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This property was identified in both the documentation and PHP SDK. I verified its existence through debug logs and implemented it accordingly.