Re: [chirp_devel] Issue with Baofeng Firmware Version

So it sounds like we should be skipping any immutable fields during set_settings(). So around here:

...

                if element.has_apply_callback():
                    LOG.debug("Using apply callback")
                    element.run_apply_callback()
                else:
                    LOG.debug("Setting %s = %s" % (setting, element.value))
                    setattr(obj, setting, element.value)

What if we changed that else to "elif setting._mutable:" ?

Then it would never run the "set" on immutable fields, which is probably what we should have done in the first place.

Does that work?

Makes sense. I will have the person with the issue test it.

Thanks, Jim