Conversation
Issue: #45 Reproduced the issue by creating a sandbox account with zero customer profiles. Error for getCustomerProfileIds() : Notice: Undefined index: numericString in C:\..local_path...\authorizenet\authorizenet\lib\AuthorizeNetCIM.php on line 491 Fixed the issue by adding a check for empty array.
Ignore this PR.
Added Test for Community Issue - Error when Zero CIM profiles
Added Test for Community Issue - Error when Zero CIM profiles - Indent corrected
…Fields.php, removed unnecessay file_put_contents from HttpClient.php
modified json error message
Added logger, removed _log_file from ApiOperationBase.php
TO DO: Remove debug statements, Organize code
Updated README for consistency with the new model of the sdk.
…ngObjects Conflicts: lib/net/authorize/util/Log.php
Fixed Community Issue - Error when Zero CIM profiles - Thanks Naman
Removed references to the old model SDK. Also removed Apple Pay sample, as advised by @ramittal ('reference the sample-code so that it is always up-do date and verified functional'). But, I thought one sample ('Quick Usage Example' here) should be kept to illustrate the setup.
Removed details other than the sample code and its dependencies.
Correct formatting.
Trimmed usage example section to only keep the links to the dev center sample codes.
…eDataConfigType) rather than stdClass.
Conflicts: lib/net/authorize/util/ANetSensitiveFields.php
Added a quick usage example (charge credit card).
Section for logging details added. Necessary information about logging like defining regexes in configuration, file paths, and logging and masking usage is mentioned.
Update Readme [Future]
Add serializer for reading the json file, setECheck Fix #74
…to SDK Adds new features to retrieve ARB subscriptions and Customer payment profile list. New APIs to handle requests, responses and the necessary fields required for these features have been added. Also some other features have been updated *CreateCustomerPaymentProfile: Now response will also provide Customer Profile Id *CreateCustomerShippingAddress: Now response will also provide Customer Profile Id *GetCustomerPaymentProfile: Now request will require provide Unmask Expiration Date Signed-off-by: Srijan Misra <srmisra@visa.com>
Adds unit tests for *ARBGetSubscription *GetCustomerPaymentProfileList Signed-off-by: Srijan Misra <srmisra@visa.com>
Adding ARBGetSubscription and GetCustomerPaymentProfileList features
…leList Adds the necessary yml and controllers for * ARBGetSubscription * GetCustomerPaymentProfileList Signed-off-by: Srijan Misra <srmisra@visa.com>
brianmc
added a commit
that referenced
this pull request
Nov 20, 2015
Merging Future branch for November 2015 Release
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
7 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.