Hmm

nlapiGetCurrentLineItemIndex()

is returning a string for me. I swear it wasn't before, and the docs say it returns an integer. Can anyone else verify?

Maybe just wrap it in Number() ?

Well, yeah. More coming from the angle that I have scripts deployed that use it & have been working OK for ages & now aren't 😄

imho NS is notorious for using strings where they could have done better

I've opened a defect anyway