bug#63290: 30.0.50; Customize UI shows extra fields for (choice (const ...) (alist ...))

[Mauro Aranda <maurooaranda@gmail.com>]

Thomas Fitzsimmons <fitzsim@fitzsim.org> writes:

 > Hi Mauro,
 >
 > Mauro Aranda <maurooaranda@gmail.com> writes:
 >
 >> tags 63290 patch quit  I ended up adding a custom :default-get
 >> function for the list widget, to make it respect a nil value as the
 >> :value.  This should be backward compatible with other widgets, and
 >> should fix these "ghost" elements insertions. I also added a test
 >> for cus-edit-tests.
 >
 > Can you try this patch with:
 >
 > M-x package-install RET excorporate RET
 >
 > Then:
 >
 > M-x customize-variable RET excorporate-configuration RET
 >
 > then select "Value Menu" and 3, which is "EWS URL OAuth 2.0 settings
 > (no autodiscovery)".  With your wis-edit.el patch applied I still get
 > empty values for:
 >
 >    INS DEL Argument name:             Argument value:
 >
 > and:
 >
 >    INS DEL OAuth 2.0 setting name:             OAuth 2.0 setting
 >   value:
 >
 > and when I apply the setting the value contains:   (... (... (#1#
 > . #1#))    (#1# . #1#))
 >
 > Maybe this is a more complicated case than the test case I provided
 > (which does now work for me with your patch)?
 >
 > Thanks,
 > Thomas

I took a look and this seems to have something to do with how the alist
widget (and almost surely the plist widget) get created when they have
:options.  I'm not sure yet, and I don't totally understand what's
happening as of now, but I wanted to report back.

I'll try to investigate and see how this can be fixed, but it's going to
take me some time.