GNU bug report logs - #63290
30.0.50; Customize UI shows extra fields for (choice (const ...) (alist ...))

Previous Next

Full log

Message #28 received at 63290 <at> debbugs.gnu.org (full text, mbox):

From: Mauro Aranda <maurooaranda <at> gmail.com>
To: Thomas Fitzsimmons <fitzsim <at> fitzsim.org>
Cc: 63290 <at> debbugs.gnu.org
Subject: Re: bug#63290: 30.0.50; Customize UI shows extra fields for (choice
 (const ...) (alist ...))
Date: Wed, 9 Aug 2023 15:03:24 -0300

Thomas Fitzsimmons <fitzsim <at> fitzsim.org> writes:

> Hi Mauro,
>
> Mauro Aranda <maurooaranda <at> 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.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.