FYI on the RP, seems we aren't allowed to add new ...
# suitescriptf
Felipe Chang
05/08/2018, 8:31 PMFYI on the RP, seems we aren't allowed to add new properties to the record object anymore:
var rec = context.newRecord;
//this is throwing an error
rec.myFunction = function(){
log.debug("hi")
}
rec.myFunction();
//this works
rec.proto.myFunction = function(){
log.debug("hi")
}
rec.myFunction();
👎 1
j
jkabot
05/08/2018, 8:56 PMYeah I was previously doing something like
rec.id = rec.save();rec.save()READ_ONLY_PROPERTYs
stalbert
05/08/2018, 9:08 PMthat particular case may be special @jkabot - that 'id' field is something that is defined in SS2. I am against the notion of adding to ANY netsuite controlled objects - that way be dragons.
stalbert
05/08/2018, 9:10 PMOn the other hand, no problem if one wants to wrap or otherwise contain a NS object, I just don't dare to modify it directly. I especially wouldn't modify proto . Heck
__proto__j
jkabot
05/08/2018, 9:11 PMYeah I thought the "fix" for my case was fair game since it was "documented" to be "read-only".
I was just annoyed that I'd have to wrap an object around the record if I wanted to pass around the saved record as well as its id
s
stalbert
05/08/2018, 9:13 PMFWIW, I think NFT does exactly this - since it wraps NS records anyway, it tries to keep the
id👍 1
2 Views