zone connection of pad can't be set back to "From parent footprint"

[Frank Severinsen]

Steps to reproduce:
1. open a footprint in FP editor.
2. open pad properties -> Local Clearance and Settings
3. change "Pad connection to zones" to "Thermal relief"
4. save footprint.
5. open pad properties -> Local Clearance and Settings
6. change "Pad connection to zones" to "From parent footprint".
7. save footprint.
8. open pad properties -> Local Clearance and Settings
9. Notice the "Pad connection to zones" is still "thermal relief"

Application: ModEdit
Version: 5.1.3-br-unknown-c8278f3~84~ubuntu19.04.1, release build
Libraries:
    wxWidgets 3.0.4
    libcurl/7.64.0 OpenSSL/1.1.1b zlib/1.2.11 libidn2/2.0.5 libpsl/0.20.2 (+libidn2/2.0.5) libssh/0.8.6/openssl/zlib nghttp2/1.36.0 librtmp/2.3
Platform: Linux 5.0.0-21-generic x86_64, 64 bit, Little endian, wxGTK
Build Info:
    wxWidgets: 3.0.4 (wchar_t,wx containers,compatible with 2.8) GTK+ 3.24
    Boost: 1.67.0
    OpenCASCADE Community Edition: 6.9.1
    Curl: 7.64.0
    Compiler: GCC 8.3.0 with C++ ABI 1013

Build settings:
    USE_WX_GRAPHICS_CONTEXT=OFF
    USE_WX_OVERLAY=ON
    KICAD_SCRIPTING=ON
    KICAD_SCRIPTING_MODULES=ON
    KICAD_SCRIPTING_PYTHON3=ON
    KICAD_SCRIPTING_WXPYTHON=ON
    KICAD_SCRIPTING_WXPYTHON_PHOENIX=ON
    KICAD_SCRIPTING_ACTION_MENU=ON
    BUILD_GITHUB_PLUGIN=ON
    KICAD_USE_OCE=ON
    KICAD_USE_OCC=OFF
    KICAD_SPICE=ON

[Michael Kavanagh (michaelkavanagh)]

I tested this earlier (macOS) and couldn't reproduce it on master. I don't have an up to date 5.1 build at the moment.

Application: KiCad
Version: (5.1.0-1404-g28446be0b), debug build
Libraries:
    wxWidgets 3.0.4
    libcurl/7.54.0 LibreSSL/2.6.5 zlib/1.2.11 nghttp2/1.24.1
Platform: Mac OS X (Darwin 18.6.0 x86_64), 64 bit, Little endian, wxMac
Build Info:
    wxWidgets: 3.0.4 (wchar_t,STL containers,compatible with 2.8)
    Boost: 1.70.0
    OpenCASCADE Community Edition: 6.9.1
    Curl: 7.54.0
    Compiler: Clang 10.0.1 with C++ ABI 1002

Build settings:
    KICAD_SCRIPTING=OFF
    KICAD_SCRIPTING_MODULES=OFF
    KICAD_SCRIPTING_PYTHON3=OFF
    KICAD_SCRIPTING_WXPYTHON=OFF
    KICAD_SCRIPTING_WXPYTHON_PHOENIX=OFF
    KICAD_SCRIPTING_ACTION_MENU=OFF
    BUILD_GITHUB_PLUGIN=ON
    KICAD_USE_OCE=ON
    KICAD_USE_OCC=OFF
    KICAD_SPICE=ON

[Wayne Stambaugh (stambaughw)]

I cannot duplicate this on the 5.1 branch.

Application: ModEdit
Version: (5.1.4-4-g450dac4b1), debug build
Libraries:
    wxWidgets 3.0.4
    libcurl/7.65.1 OpenSSL/1.1.1c zlib/1.2.11 libidn2/2.2.0 libpsl/0.20.2 (+libidn2/2.0.5) libssh2/1.8.0 nghttp2/1.37.0 librtmp/2.3
Platform: Linux 4.19.0-5-amd64 x86_64, 64 bit, Little endian, wxGTK
Build Info:
    wxWidgets: 3.0.4 (wchar_t,wx containers,compatible with 2.8) GTK+ 3.24
    Boost: 1.62.0
    OpenCASCADE Community Edition: 6.9.1
    Curl: 7.65.1
    Compiler: Clang 7.0.1 with C++ ABI 1002

Build settings:
    USE_WX_GRAPHICS_CONTEXT=OFF
    USE_WX_OVERLAY=ON
    KICAD_SCRIPTING=ON
    KICAD_SCRIPTING_MODULES=ON
    KICAD_SCRIPTING_PYTHON3=ON
    KICAD_SCRIPTING_WXPYTHON=ON
    KICAD_SCRIPTING_WXPYTHON_PHOENIX=ON
    KICAD_SCRIPTING_ACTION_MENU=ON
    BUILD_GITHUB_PLUGIN=ON
    KICAD_USE_OCE=ON
    KICAD_USE_OCC=OFF
    KICAD_SPICE=ON

[Frank Severinsen (shack)]

Hmm Step 2 should probably be "double click on a pad to open pad properties".
Is this helping replicate the issue?

[Frank Severinsen (shack)]

alright, figured out whats going here (bug might be invalid)

When you change the FP (not pad) zone connection, to some other than "use zone settings", this setting overrides the pad (not FP) zone connection. so if you try to change the pad connection afterwards, it simply reverts to whatever the FP connection is set to.

I was guessing "From parent footprint" would simply point to the FP zone setting, but still be overridable by the pad setting. (say you wanted the footprint to be thermal relief, but override 1 pad for solid)

Perhaps the pad connection dropdown should be greyed out/disabled when the FP connection has been set?

[Jeff Young (jeyjey)]

That's a bug. Pad should win over footprint (which wins over zone).

[KiCad Janitor (kicad-janitor)]

Fixed in revision edc05b5d5440b72545cebbbffc4d918bdf77e4f1
https://git.launchpad.net/kicad/patch/?id=edc05b5d5440b72545cebbbffc4d918bdf77e4f1