eeschema: shortcut 'C' (Copy Component) asks to Clarify select Field Value or Component

[Eldar Khayrullin]

If a value field and a component placed near if push 'C' eeschema asks to make clarify.
Copy operations doesn't apply to a Value Field.
Clarify Select pop-up menu is odd.

Application: eeschema
Version: 0.201604162031+6694~44~ubuntu15.10.1-product, release build
Libraries: wxWidgets 3.0.2
           libcurl/7.43.0 OpenSSL/1.0.2d zlib/1.2.8 libidn/1.28 librtmp/2.3
Platform: Linux 4.2.0-35-generic x86_64, 64 bit, Little endian, wxGTK
- Build Info -
wxWidgets: 3.0.2 (wchar_t,wx containers,compatible with 2.8)
Boost: 1.58.0
Curl: 7.43.0
KiCad - Compiler: GCC 5.2.1 with C++ ABI 1009
        Settings: USE_WX_GRAPHICS_CONTEXT=OFF
                  USE_WX_OVERLAY=OFF
                  KICAD_SCRIPTING=ON
                  KICAD_SCRIPTING_MODULES=ON
                  KICAD_SCRIPTING_WXPYTHON=ON
                  USE_FP_LIB_TABLE=HARD_CODED_ON
                  BUILD_GITHUB_PLUGIN=ON
OS: Ubuntu 15.10

[Novak Tamas (novak-7)]

Probably at first there is a code part which chooses objects involved and this part fires up listbox to "clarify selection". Next code part receives already selected object(s) to achieve the move/copy/... function.
It would be nice if object chooser part distinguished on objects based on the function to achieve later... but it is only for wishlist.

[Eldar Khayrullin (eldar)]

WHy invalid? Maybe should be other Status

[Chris Pavlina (pavlina-chris)]

Agreed, Invalid is not the right status for this.

I also don't agree that it's a Wishlist item, it looks like a legitimate bug to me. Fields can't receive the Copy operation ---- except when they overlap components, in which case it tries. That's wrong.

[Eldar Khayrullin (eldar)]

Agreed with Chris Pavlina. The code implementation it isn't reason to set a bug to Wishlist

[Novak Tamas (novak-7)]

Okay.
In my opinion it is not a bug. The "clarify.." question is silly and unnecesary, but definitly NOT a bug. If I choose Value or Reference from Clarify list, nothing happens for C-copy --as expected. No crash, no bad moves etc.

If something is not a bug, but a "would be so fine" then it's going to Wishlist. If Importance=Wishlist, Status shouldn't be left New, as it seems to be untouched yet. If case is regarded as not-a-bug, then it's an invalid bug report. (Not an invalid report or invalid wish, but an invalid bugreport.)
I don't fight, I leave it as a low-importance bug, but I keep disagreeing.

[Chris Pavlina (pavlina-chris)]

Any time something doesn't work correct, it's a bug. Bug and crash, or bug and data corruption, are not synonymous. No worky = bug.

[Eldar Khayrullin (eldar)]

Bug it is a wrong behaviour. Here 'clarify' question is wrong behaviour of shortcut to 'Copy' command

[Eldar Khayrullin (eldar)]

And bug it is a wrong representation (text's typo, for example) too.

[KiCad Janitor (kicad-janitor)]

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