From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: Mauro Aranda Newsgroups: gmane.emacs.bugs Subject: bug#63290: 30.0.50; Customize UI shows extra fields for (choice (const ...) (alist ...)) Date: Wed, 9 Aug 2023 15:03:24 -0300 Message-ID: References: <136334d9-262d-e290-cff3-159c2f827b87@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="17296"; mail-complaints-to="usenet@ciao.gmane.io" User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.13.0 Cc: 63290@debbugs.gnu.org To: Thomas Fitzsimmons Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Wed Aug 09 20:04:13 2023 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane-mx.org Original-Received: from lists.gnu.org ([209.51.188.17]) by ciao.gmane.io with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1qTnXZ-0004DA-44 for geb-bug-gnu-emacs@m.gmane-mx.org; Wed, 09 Aug 2023 20:04:13 +0200 Original-Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1qTnXQ-0001dJ-28; Wed, 09 Aug 2023 14:04:04 -0400 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1qTnXO-0001ct-51 for bug-gnu-emacs@gnu.org; Wed, 09 Aug 2023 14:04:02 -0400 Original-Received: from debbugs.gnu.org ([2001:470:142:5::43]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1qTnXN-0000UV-TH for bug-gnu-emacs@gnu.org; Wed, 09 Aug 2023 14:04:01 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1qTnXN-00048A-OB for bug-gnu-emacs@gnu.org; Wed, 09 Aug 2023 14:04:01 -0400 X-Loop: help-debbugs@gnu.org Resent-From: Mauro Aranda Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Wed, 09 Aug 2023 18:04:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 63290 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: patch Original-Received: via spool by 63290-submit@debbugs.gnu.org id=B63290.169160421915847 (code B ref 63290); Wed, 09 Aug 2023 18:04:01 +0000 Original-Received: (at 63290) by debbugs.gnu.org; 9 Aug 2023 18:03:39 +0000 Original-Received: from localhost ([127.0.0.1]:40480 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1qTnWx-00047Q-6P for submit@debbugs.gnu.org; Wed, 09 Aug 2023 14:03:39 -0400 Original-Received: from mail-ot1-x32b.google.com ([2607:f8b0:4864:20::32b]:51255) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1qTnWv-00047D-OI for 63290@debbugs.gnu.org; Wed, 09 Aug 2023 14:03:34 -0400 Original-Received: by mail-ot1-x32b.google.com with SMTP id 46e09a7af769-6bd0a0a6766so68539a34.2 for <63290@debbugs.gnu.org>; Wed, 09 Aug 2023 11:03:33 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20221208; t=1691604208; x=1692209008; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=MBDcA2l/fW72ghUYYDCUdOKrv3e+amRz3AxrWxsWgns=; b=bjTjYu5XtjYo1gspZA606XP6KNGZ9O1GTPw123+PjiJ9vrU6A/TxEDJdiH5PacdSuL /isJnHQaRBZHu7VTXhIfm9PkThjpk2FJZvJSI6wJeAvCAW/YwisluaA9HY6H6m+NZuxE bEaiqEckBduHbapUjrBBQtedPAO2dpjhXI0N5V0HafbxR9ifS/etieflT8v+/0AhlJfH 6Q8AbFwVqPavRV4bRW8h36rqAuDEAfzigoxs6nPt99C73AzNYdoyGpqCXztBzBvgRzUk Rv2jHx7TRK9wDi1JbfceSXALJNUqP0gNQlRpxtkN6VJGV2zdp0X/Z3hPcPWHDbYYmLAM m1pg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1691604208; x=1692209008; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=MBDcA2l/fW72ghUYYDCUdOKrv3e+amRz3AxrWxsWgns=; b=ko4No4hmWl5qj9gsDfOFKRhz0CxAkUQGZazPWRqJKAZxR+mLvxE3B8fo9pNDyPSCIm n26oGmpBbZGRXvBjP6CazByTbXocx1lPbQYDfVLkhgxmc8bswHdFNTF0NFFtA/WAhewm E1+3iuHplvvEL5X65JWpIbbF0+tWjiDpBwux5j8gsCaqwdGtoH0VW5QkPv9vT6LH3pOH 1eutM2/OIR/Cdt2sS1Dq3g4kdmhA5DK8fadojKcOI0LLIj0x6XOZmKIjB7CuPRKPcUfd 329nMhaR5Y44YS9RycRFE/Zm9z/x2dD6lyrJdF8HlvcmsmykC8vmnXcbQlT10tCEIsAN BvBQ== X-Gm-Message-State: AOJu0YwTktjmPCRJKBfJ5kmTB7upAT6lajx6GW2LPxUgponqnuvBznbN KcNVi5Fp9rTOyvlfgOD6odECszVyjBI= X-Google-Smtp-Source: AGHT+IHoygLnFCX2971tiqalF/sdwP3OqHwjB91C77vTwhDsvLXJJ2QrCHPam8e8w+LGxXXlb30AuA== X-Received: by 2002:a05:6830:130f:b0:6b9:72be:671f with SMTP id p15-20020a056830130f00b006b972be671fmr3839480otq.16.1691604207699; Wed, 09 Aug 2023 11:03:27 -0700 (PDT) Original-Received: from [192.168.0.234] ([186.127.128.249]) by smtp.gmail.com with ESMTPSA id j21-20020a9d7f15000000b006b9ad7d0046sm7091567otq.57.2023.08.09.11.03.26 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 09 Aug 2023 11:03:27 -0700 (PDT) Content-Language: en-US In-Reply-To: X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Original-Sender: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Xref: news.gmane.io gmane.emacs.bugs:267058 Archived-At: Thomas Fitzsimmons writes: > Hi Mauro, > > Mauro Aranda 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.