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

Previous Next

Package: emacs;

Reported by: Thomas Fitzsimmons <fitzsim <at> fitzsim.org>

Date: Fri, 5 May 2023 06:04:01 UTC

Severity: normal

Tags: patch

Found in version 30.0.50

Done: Eli Zaretskii <eliz <at> gnu.org>

Bug is archived. No further changes may be made.

Full log


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

From: Thomas Fitzsimmons <fitzsim <at> fitzsim.org>
To: Mauro Aranda <maurooaranda <at> gmail.com>
Cc: 63290 <at> debbugs.gnu.org
Subject: Re: bug#63290: 30.0.50; Customize UI shows extra fields for (choice
 (const ...) (alist ...))
Date: Wed, 09 Aug 2023 11:51:00 -0400
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




This bug report was last modified 1 year and 330 days ago.

Previous Next


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