With GP 10 and earlier, there was a single method called "eConnect_EntryPoint". Simple and easy, it accepted all of the eConnect document types.
With GP 2010, there are now two different methods. There is now a method called "CreateEntity", and a second called "CreateTransactionEntity".
Although I have one guess, I haven't asked and don't really know why this change was made, or what the rationale was behind implementing two methods--I just hope that some thought was put into it, and I hope that there was some benefit behind the scenes.
I hope that because it's slightly annoying to have to make two different calls now. By no means a big deal, but just slightly annoying, especially if you are upgrading from eConnect 9 or 10. You have to review your code and make sure that your code to create customers calls CreateEntity, and your code that imports invoices calls CreateTransactionEntity.
The eConnect 2010 Programmer's Guide provides this mediocre and slightly vague explanation the two new methods:
CreateEntity: Creates a record using information from an eConnect XML document. Use CreateEntity to add data entities like customers or vendors. You specify the type of record to create with a parameter that represents an eConnect XML document. If the operation succeeds, the method returns True as a boolean value.
CreateTransactionEntity: Create a transaction using information from an eConnect XML document. Use CreateTransactionEntity for transaction documents like sales orders or inventory transactions. You specify the type of transaction to create with a parameter that represents an eConnect XML document. If the operation succeeds, the method returns an XML string that represents the eConnect XML document that was created.
So, this is fine for obvious record types like a vendor vs. a voucher, but what if I create a new batch using SMTransactionBatchType? Is a batch an "Entity", or a "TransactionEntity"? That starts to become an existential question, because the current eConnect documentation offers no guidance when discussing each of the transaction types. You're stuck trying both methods to figure out which one to use (hint--the one that causes your app to crash is the wrong one).
With the griping aside, there is one last thing to pay attention to. Okay, it's really one last gripe.
Unless you paid very careful attention, you probably missed one critical difference between CreateEntity and CreateTransactionEntity. Do you know what it is?
They have different names obviously, but they both accept the same parameters, and they both seem to function in the same way. Read the last sentence of each of the paragraphs from the help file quoted above.
"If the operation succeeds, the method returns True as a boolean value."
"If the operation succeeds, the method returns an XML string that represents the eConnect XML document that was created."
WHY????
Despite having done at least a dozen eConnect 2010 integrations, I apparently hadn't paid close enough attention to that, so I discovered this the hard way: By having to submit a support case. I had upgraded an integration from GP 10, and when I had to create my two eConnect methods, I created a CreateEntity method, then copied that to do a CreateTransactionEntity method. I just changed the eConnect method name and that was it. Except that it wasn't.
I forgot to change my return variable from a bool to a string. When you make that mistake, you will get this error:
Unexpected error in InsertTransaction: Conversion from string " < eConnect xmlns:xsi="http://www." to type 'Boolean' is not valid.
After inserting the transaction, eConnect returns the XML string, but the code fails when it tries to send that into your boolean return variable.
Strangely, the .NET compiler didn't catch the mistake, so I thank Chris Roehrich at GP support for pointing out my mistake.
UPDATE: I just figured out why the .NET compiler didn't catch the problem. This particular eConnect upgrade was the first time I had upgraded an old VB eConnect integration to GP 2010. It seems that the VB compiler does not detect the type mismatch. I just tried the same thing in C#, and the compiler picks it up right away. All of my previous eConnect 2010 integrations and upgrades have been in C#, which is why I never had the issue before.
Why they couldn't just make both methods have the same return type...well, again, I hope there was a some thought put into it and there is a reason and a benefit...
Steve Endow is a Dynamics GP Certified Trainer and Dynamics GP Certified IT Professional in Los Angeles. He is also the owner of Precipio Services, which provides Dynamics GP integrations, customizations, and automation solutions.
http://www.precipioservices.com
3 comments:
Lesson learned... Don't use VB for commercial grade applications :-))
MG.-
Mariano Gomez, MVP
"dont' use VB for commercial grade apps."
Thats not a helpful comment.
consider many clients have home baked solutions that the VARs inherit when the 'code guy/gal' leaves the organization, and most organizations don't want to spend money to convert the vb apps to c# "just cause".
I am afraid I agree with the blog authors unspoken criticism of the API change. It does not appear to be changed for the better and only serves to necessitate recompiles and code changes for no obvious benefit.
Pretty old already but just read this and thought I'd chime in.
"Why they couldn't just make both methods have the same return type...well, again, I hope there was a some thought put into it and there is a reason and a benefit..."
The int return method is the "old way" and the new "return xml of document" was added pretty much for web services. Users would call the "CreateSalesInvoice" method and not supply the doc number and let WS/eConnect assign it. But then they'd want to know the document number just created and there wasn't a way. Well now there is.
As for having the 2 methods, there was thought to have some functionality changes behind this (other than return xml string) but that didn't happen and I think that the one method internally calls the other if i remember correctly.
Post a Comment