issaved() doesn't work when called directly

ID:1878028
 
Jun 20 2015, 3:44 am
BYOND Version:508
Operating System:Windows 7 Home Basic 64-bit
Web Browser:Chrome 43.0.2357.124
Applies to:Dream Maker
Status: Open
Issue hasn't been assigned a status value.
Descriptive Problem Summary:
issaved(const_var) fails to compile with the message: "cannot change constant value".

Code Snippet (if applicable) to Reproduce Problem:
/mob/var/const/A = "a letter"

/mob/verb/issaved_test()
    src << issaved(src.A)


When does the problem NOT occur?
When doing issaved(vars["const_var"])

Did the problem NOT occur in any earlier versions? If so, what was the last version that worked?
Not tested. Might be really old, might be new to 508.1288
Jun 20 2015, 3:48 am
Constant vars are never saved, but nice find.
Jun 22 2015, 10:03 am
Ah, I found out what's happening here. I don't have a fix right away, but I did find this note under issaved():

//use save idx even though we are getting value because that handles lists correctly
//TODO: should make this work for consts and other non-writable values

I don't understand why lists wouldn't be handled correctly without this, so that's something I'd have to investigate.