> + // reverse the logic for the checkbox > + values.nowritecache = values.nowritecache == '0'?'1':'0'; I wonder why we compare to a string '0'?? AFAIK nowritecache is an boolean/integer?