When fishing out the value of a control by id in modern query you have to drill into an array of one item.

var defaultQtyPerItem = $("#DefaultQtyPerItem").val();

 
 

At some point what's above became what is below. Don't ask me when.

var defaultQtyPerItem = $("#DefaultQtyPerItem")[0].value;