My 5 Cents:
Change that else for:
elif not "firmware_msg" in obj:
That will refuse to change the setting in the firmware_msg setting, both lines.
73
El 13/02/16 a las 22:17, Jim Unroe via chirp_devel escribió:
On Sat, Feb 13, 2016 at 9:24 PM, Jim Unroe rock.unroe@gmail.com wrote:
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:" ?
My tester reported back "I just tried it and when I edit the squelch/service settings, I get an error message that says, "Error in setting value: 'str' object has no attribute'_mutable'""
Jim _______________________________________________ chirp_devel mailing list chirp_devel@intrepid.danplanet.com http://intrepid.danplanet.com/mailman/listinfo/chirp_devel Developer docs: http://chirp.danplanet.com/projects/chirp/wiki/Developers