Question

How do I correct the code for calculating the total for a Purchase Order?

Forms

Updated October 10, 2017

asked on April 15, 2014

I have set up a table following the instructions on the help page: Customizing Your Form with JavaScript: Examples for a Purchase Order. Using the code provided, it does not correctly add the total. Instead of adding the tax and shipping once to the subtotal collected from each line total, it adds the tax and shipping to each line. For example:

A form with the following values:

Qty Price Total Price

2 $5.00 $10.00

3 $5.00 $15.00

Shipping $1.00

Tax $1.00

Will produce a Total of $29.00, instead of $27.00

Can anyone tell me how to adjust the code to properly calculate the total?

Thanks,

0 0

Answer

SELECTED ANSWER

replied on April 15, 2014 • Show version history

Add a new Currency field to the form and give it the grandTotal CSS class.

The total field will now only show the table subtotal. The grand total field will show the total plus shipping and tax.

Replace the JavaScript from the example with the following:

$(document).ready(function () {
$('.cf-table-block').on('blur', 'input', sumtotal);
$('.tax').on('blur', 'input', sumtotal);
$('.shipping').on('blur', 'input', sumtotal);
function sumtotal() {
var sum = 0;
$('.cf-table-block tbody tr').each(function () {
var s = 0;
s = parseNumber($(this).find('.price input').val()) * parseNumber($(this).find('.quantity input').val());
$(this).find('.subtotal input').val((s).toFixed(2));
sum += s; 
});
$('.total input').val((sum).toFixed(2));
$('.grandTotal input').val((sum + parseNumber($('.tax input').val()) + parseNumber($('.shipping input').val())).toFixed(2));  
}
function parseNumber(n) {
var f = parseFloat(n); //Convert to float number.
return isNaN(f) ? 0 : f; //treat invalid input as 0;
}
});

Note: This code further enhances the online help example by restricting the values to two decimal places using the .toFixed() method.

To add a little more explanation: the sum variable keeps the sum of each table row. In the version of this code shown in the online help, this value was added to the shipping and tax amount before being placed into the total field.

To change the code, we placed only the sum value into the total field, and added a new grand total field that takes the total amount plus shipping and tax.

0 0

Replies

replied on April 15, 2014

Hi Eric,

Yes, I would like the total field to reflect the total before tax and shipping. I would also like to have a grand total field that adds the total plus tax and shipping. I am fairly new to JavaScript and having trouble changing the code to do this.

Thanks,

0 0

replied on April 15, 2014 • Show version history

Edited to avoid confusion. If you don't want to restrict currency values to two decimal places, you can use the following code:

$(document).ready(function () {
$('.cf-table-block').on('blur', 'input', sumtotal);
$('.tax').on('blur', 'input', sumtotal);
$('.shipping').on('blur', 'input', sumtotal);
function sumtotal() {
var sum = 0;
$('.cf-table-block tbody tr').each(function () {
var s = 0;
s = parseNumber($(this).find('.price input').val()) * parseNumber($(this).find('.quantity input').val());
$(this).find('.subtotal input').val(s);
sum += s; 
});
$('.total input').val(sum);
$('.grandTotal input').val(sum + parseNumber($('.tax input').val()) + parseNumber($('.shipping input').val()));  
}
function parseNumber(n) {
var f = parseFloat(n); //Convert to float number.
return isNaN(f) ? 0 : f; //treat invalid input as 0;
}
});

0 0

SELECTED ANSWER

replied on April 15, 2014 • Show version history

Add a new Currency field to the form and give it the grandTotal CSS class.

The total field will now only show the table subtotal. The grand total field will show the total plus shipping and tax.

Replace the JavaScript from the example with the following:

$(document).ready(function () {
$('.cf-table-block').on('blur', 'input', sumtotal);
$('.tax').on('blur', 'input', sumtotal);
$('.shipping').on('blur', 'input', sumtotal);
function sumtotal() {
var sum = 0;
$('.cf-table-block tbody tr').each(function () {
var s = 0;
s = parseNumber($(this).find('.price input').val()) * parseNumber($(this).find('.quantity input').val());
$(this).find('.subtotal input').val((s).toFixed(2));
sum += s; 
});
$('.total input').val((sum).toFixed(2));
$('.grandTotal input').val((sum + parseNumber($('.tax input').val()) + parseNumber($('.shipping input').val())).toFixed(2));  
}
function parseNumber(n) {
var f = parseFloat(n); //Convert to float number.
return isNaN(f) ? 0 : f; //treat invalid input as 0;
}
});

Note: This code further enhances the online help example by restricting the values to two decimal places using the .toFixed() method.

To change the code, we placed only the sum value into the total field, and added a new grand total field that takes the total amount plus shipping and tax.

0 0

replied on April 15, 2014

Hi Eric,

This worked perfectly. Thanks for your quick response and the additional code for restricting the values to two decimal places.

Thanks!

Jen

0 0

replied on April 15, 2014

You're welcome!

0 0

replied on October 10, 2017

I just came across this error in the Laserfiche Administration Help Guide (which still needs to be corrected). Here is an alternative way to accomplish this without adding a new field:

Hope this helps someone down the road.

Thanks

0 0

replied on April 15, 2014

As written, the code adds the values from the Shipping and Tax fields directly to the subtotal to arrive at the total amount. So, the total amount is the the sum of each row subtotal plus shipping plus tax.

Do you want the total field to reflect the total before tax and shipping?

0 1

You are not allowed to follow up in this post.

Question

Question

How do I correct the code for calculating the total for a Purchase Order?

Answer

Replies

Sign in to reply to this post.