|
Medz
|
Posted Thu 22 Nov 2012
|
|
Group: Administrators
Last Active: Tue 3 Dec 2013
Posts: 99,
Visits: 1,400
|
Hi John, attached is the updated Kartris Interfaces DLL and SagePay DLL that contain these changes. The interface DLL goes to the 'bin' folder of your Kartris root and the actual sagepay DLL to 'Plugins/SagePay/'. These DLLs are the ones that will be included in the upcoming Kartris 2.0 release. Since you're still using Kartris v1.xxx, you need to add the line below to your checkout script in order for it to work properly ->
objOrder.ShippingExTaxPrice = CurrenciesBLL.FormatCurrencyPrice(CUR_ID, objBasket.ShippingPrice.ExTax, False) You need to insert the new line before or after this line in Checkout.aspx.vb -> objOrder.ShippingPrice = CurrenciesBLL.FormatCurrencyPrice(CUR_ID, objBasket.ShippingPrice.IncTax, False) Let me know if it works for you. |
|
|
|
|
JBeardon
|
Posted Thu 15 Nov 2012
|
|
Group: Awaiting Activation
Last Active: Thu 13 Dec 2012
Posts: 9,
Visits: 18
|
UPDATE - IMPORTANTI have now eventually spoken to someone within Sagepay who has been honest and truthful regarding this issue. The Sagepay integration guide is generic and provides no details whatsoever regarding the Sage/Sagepay integration. They will be correcting this in the future as unsurprisingly it is causing a lot of issues! He has provided the definitive format of what needs to be included for the delivery line to import into Sage correctly. This is what is produced by the DLL you amended for us to include the product code in square brackets and separate the delivery. Basket items | Quantity | Item value | Item tax | Item total | Line total | [TT123]Bag | 4 | 7.08 | 1.42 | 8.5 | 34 | Delivery | - | - | - | - | 4.95 |
This is what it should look like, each field needs to have a value in it, no dashes (-): Basket items | Quantity | Item value | Item tax | Item total | Line total | [TT123]Bag | 4 | 7.08 | 1.42 | 8.5 | 34 | Delivery | 1 | 4.95 | 0 | 4.95 | 4.95 |
This format will still work for generic sagepay integration, but will also have the benefit of allowing Sage Line 50 users to download SagePay transactions into their accounts package, integrate with Sage stock management and also handle the delivery correctly.
If you could amend the DLL to produce this for us we would be eternally grateful and it would also be of great assistance to other Sage users.
Thanks
John
|
|
|
|
|
JBeardon
|
Posted Thu 8 Nov 2012
|
|
Group: Awaiting Activation
Last Active: Thu 13 Dec 2012
Posts: 9,
Visits: 18
|
Hi Medz Sorry for the delay on getting back to you on this. We have been battling with SagePay and Sage to try and get someone to give us a definitive answer regarding the format required, which apparently is not what is published in their integration guide. The latest DLL you kindly produced is almost there, but for the Delivery line, they also require a quantity and the Tax amounts to be shown. Apologies for this, but Sage and SagePay are proving to be a pain to deal with on this and have only just come back with this answer. If there is anything you can do to amend the dll and include the info required, it would be greatly appreciated. |
|
|
|
|
JBeardon
|
Posted Tue 2 Oct 2012
|
|
Group: Awaiting Activation
Last Active: Thu 13 Dec 2012
Posts: 9,
Visits: 18
|
Sorry for the delay Medz.We should be testing this tomorrow so will get back to you with results. Many thanks for your help John
|
|
|
|
|
Medz
|
Posted Fri 7 Sep 2012
|
|
Group: Administrators
Last Active: Tue 3 Dec 2013
Posts: 99,
Visits: 1,400
|
Ok. This one separates the shipping charges and puts it in a separate line as "Delivery". It then puts the remaining amount (if theres any) ->
order total - (total basket items value + shipping price) as "Discounts and other adjustments" line. Can you check if this one does it? |
|
|
|
|
JBeardon
|
Posted Thu 6 Sep 2012
|
|
Group: Awaiting Activation
Last Active: Thu 13 Dec 2012
Posts: 9,
Visits: 18
|
That worked Guys but we do have another problem :-(
When the shipping cost is being passed to SagePay, it appears on a line titled 'Shipping, discounts and other adjustments' - see attached screen shot.
This doesn't cause an issue apart from if the user wants to import transactions from SagePay into Sage Line 50. From what I can gather, Sage Line 50 is expecting to see the line title to be Delivery instead. The problem this causes is that when the import takes place, Sage actually sees the shipping value as a discount and takes it from the order total.
I am seeing if the accountant can work out if this is a Sage configuration issue but looking at page 22 of the SagePay/Sage integration guide, I don't think this is the case. This page also mentions the product code in square brackets so I've attached it for your reference.
Will report back regarding the Delivery charges/Sage import. |
|
|
|
|
JBeardon
|
Posted Thu 6 Sep 2012
|
|
Group: Awaiting Activation
Last Active: Thu 13 Dec 2012
Posts: 9,
Visits: 18
|
Many thanks Medz. I will give it a go and report back.
|
|
|
|
|
Medz
|
Posted Wed 5 Sep 2012
|
|
Group: Administrators
Last Active: Tue 3 Dec 2013
Posts: 99,
Visits: 1,400
|
Hi John, attached is an updated SagePay.dll that includes the Version CodeNumber in the product description field passed to SagePay. Can you let me know if this works for you? I've just appended the version codenumber inside square brackets at the beginning of the product description field. Its not mentioned anywhere in their documentation so I just did it based on whats written in your post.
[codenumber]productdescription
|
|
|
|
|
Paul
|
Posted Sat 1 Sep 2012
|
|
Group: Administrators
Last Active: Tue 10 Sep 2024
Posts: 807,
Visits: 2,748
|
The sagepay plugin does format an order description, which is a text built up of the order contents, although this doesn't include the version code, only the version name and product name (or just one, if it is a single version type product, since these are the same).
I'll see if we can get this added into the plugin...
--
If my post solves your issue, can you 'Mark as Answer' so it's easier for other users to find in future.
If you would like to be informed of new features, new releases, developments and occasional special bonuses, please sign up to our mailing list: http://bit.ly/19sKMZb
|
|
|
|
|
JBeardon
|
Posted Wed 29 Aug 2012
|
|
Group: Awaiting Activation
Last Active: Thu 13 Dec 2012
Posts: 9,
Visits: 18
|
I was quite hoping for a response from the Kartris guys regarding this.
|
|
|
|