intacct / intacct-sdk-php Goto Github PK
View Code? Open in Web Editor NEWOfficial repository of the Sage Intacct SDK for PHP
Home Page: https://developer.intacct.com/tools/sdk-php/
License: Apache License 2.0
Official repository of the Sage Intacct SDK for PHP
Home Page: https://developer.intacct.com/tools/sdk-php/
License: Apache License 2.0
Make sure to adjust MessageFormatter to redact cardnum
elements
Setting the logger config in the __construct is not passing it appropriately to any subsequent executions. You have to provide it as a param with each execution.
Already have Abstract and Create. Still need:
$client = IntacctClient();
should be $client = new IntacctClient();
Fix the USERLOCATIONS to support multiple entities like the following:
<USERINFO> <LOGINID>usertest9</LOGINID> <USERLOCATIONS> <LOCATIONID>123</LOCATIONID> </USERLOCATIONS> <USERLOCATIONS> <LOCATIONID>1</LOCATIONID> </USERLOCATIONS> </USERINFO>
Oops
For example, AccountsPayable and AccountsReceivable both have AbstractPayment classes. The shared names makes it difficult to work with.
PhpStorm 2016.3 complains about the namespace not being found in many of the phpunit tests.
Example: @covers Intacct\IntacctClient::__construct
To resolve this, the class needs to have a leading backslash: @covers \Intacct\IntacctClient::__construct
.
Also, instead of using the individual @covers
annotations per test function, it might be easier to just remove them and add @coversDefaultClass \Intacct\IntacctClient
to the top of the test class
Change the return to be Result[] instead of array.
Have the ErrorMessage::__construct
decode the XML values.
Get rid of the :
symbol too from the implode
function on line 46 and just use a space.
The fulfillmentstatus
XML element is not a basic string. It contains other elements per the schema:
deliverystatus
deliverydate
deferralstatus
kitstatus
(multiple)Passing an empty string to setAccountLabel('');
causes the writeXml function to end up writing the account label instead of the GL account.
if (!is_null($this->getAccountLabel())) {
needs to be
if (!empty($this->getAccountLabel())) {
This should be fixed on other *LineCreate classes. Please check LineUpdate is also functioning properly.
Probably missed this one with the online payment methods
Make sure to adjust MessageFormatter to redact accountnumber
elements
If the request's uniqueid = true, it should retry submitting the request. Server should enforce idempotence based on function's controlid attribute. Perhap the retry has a delay of X seconds
SDK already redacts password
elements with the MessageFormatter class. It should redact additional fields by default from any logs:
accountnumber
cardnum
ssn
achaccountnumber
wireaccountnumber
taxid
Need to make sure same fields from read
functions (usually in all CAPS) are redacted too.
This will remove IDE errors for unknown writeXml functions
A declarative, efficient, and flexible JavaScript library for building user interfaces.
๐ Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
An Open Source Machine Learning Framework for Everyone
The Web framework for perfectionists with deadlines.
A PHP framework for web artisans
Bring data to life with SVG, Canvas and HTML. ๐๐๐
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
Some thing interesting about web. New door for the world.
A server is a program made to process requests and deliver data to clients.
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
Some thing interesting about visualization, use data art
Some thing interesting about game, make everyone happy.
We are working to build community through open source technology. NB: members must have two-factor auth.
Open source projects and samples from Microsoft.
Google โค๏ธ Open Source for everyone.
Alibaba Open Source for everyone
Data-Driven Documents codes.
China tencent open source team.