This Bugzilla instance is deprecated, and most Eclipse projects now use GitHub or Eclipse GitLab. Please see the deprecation plan for details.
Bug 543218 - Launch configuration - view filter affects working copy state
Summary: Launch configuration - view filter affects working copy state
Status: VERIFIED FIXED
Alias: None
Product: PDE
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.11   Edit
Hardware: All All
: P3 major (vote)
Target Milestone: 4.11 M3   Edit
Assignee: Julian Honnen CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 543416 (view as bug list)
Depends on:
Blocks:
 
Reported: 2019-01-07 11:00 EST by Lars Vogel CLA
Modified: 2019-03-21 13:07 EDT (History)
5 users (show)

See Also:


Attachments
screencap (1.15 MB, video/mp4)
2019-01-08 03:00 EST, Julian Honnen CLA
no flags Details
Example project (15.37 KB, application/x-zip-compressed)
2019-01-11 04:38 EST, Lars Vogel CLA
no flags Details
Screencast on Linux demonstrating the issue (3.12 MB, image/gif)
2019-01-11 04:54 EST, Lars Vogel CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Lars Vogel CLA 2019-01-07 11:00:14 EST
Eclipse 2018-12 allows to use the "Add Required Plug-ins" button, even if a view filter is set. That is great!

Currently the "Add Required Plug-ins" does consider the view filter, which I think is counter-intuitiv. 

In my example, I filtered by "jface" and added it. Afterwards I pressed "Add Required Plug-ins" and expected to have swt added.

I suggest to consider all available plug-ins in "Add Required Plug-ins".
Comment 1 Lars Vogel CLA 2019-01-07 11:00:55 EST
Julian, WDYT?
Comment 2 Julian Honnen CLA 2019-01-07 11:09:08 EST
The plug-in tree improvements are not in 4.10 (-> Bug 441684). The add required button is still disabled there with an active filter.

What build are you running? Please try an I-Build, the button should behave exactly like you want.
Comment 3 Lars Vogel CLA 2019-01-07 11:57:16 EST
(In reply to Julian Honnen from comment #2)
> The plug-in tree improvements are not in 4.10 (-> Bug 441684). The add
> required button is still disabled there with an active filter.
> 
> What build are you running? Please try an I-Build, the button should behave
> exactly like you want.

Eclipse SDK
Version: 2019-03 (4.11)
Build id: I20190106-1800
OS: Windows 10, v.10.0, x86_64 / win32
Java version: 1.8.0_191

To test:

Create a new e4 RCP application (using the e4 RCP template)
Start it via the generated product
Open the launch configuration and remove SWT
Filter for jface and press Add Required Plug-ins -> SWT is not addded
Comment 4 Lars Vogel CLA 2019-01-08 02:57:40 EST
Julian, are you able to reproduce?
Comment 5 Julian Honnen CLA 2019-01-08 03:00:26 EST
Created attachment 277088 [details]
screencap

No, those exact steps work for me on that build (see attached screencap). Anything you're doing differently?
Comment 6 Lars Vogel CLA 2019-01-11 04:38:24 EST
Created attachment 277132 [details]
Example project

Still does not work for me. I'm testing on Windows (no idea how to do a screencast here) but will repeat soon the test on Linux (for which I can provide a screencast).

Test I do:

Import attached project and start via the product
Go to launch configuration and remove SWT and press validate -> errors are reported
Add filter for jface -> Press add required plug-ins 
Press Validate again -> Still errors

If I remove the filter and press Add required plug-in the validate does not report problems anymore.
Comment 7 Lars Vogel CLA 2019-01-11 04:54:52 EST
Created attachment 277133 [details]
Screencast on Linux demonstrating the issue
Comment 8 Julian Honnen CLA 2019-01-11 07:30:18 EST
It seems like the result of the validation depends on the visible plugins. I get different problems depending on the filter.

Add required seems to work in your screencap. The selected plugin count goes back up to 293.
Comment 9 Julian Honnen CLA 2019-01-14 08:00:59 EST
Saving into working copy broke with the new tree implementation from Bug 441684: only selected *and visible* plugins are written to working copy.

-> PluginBlock#savePluginState
Comment 10 Julian Honnen CLA 2019-01-14 08:03:56 EST
*** Bug 543416 has been marked as a duplicate of this bug. ***
Comment 11 Eclipse Genie CLA 2019-01-14 08:15:55 EST
New Gerrit change created: https://git.eclipse.org/r/135040
Comment 12 Andrey Loskutov CLA 2019-01-14 08:32:36 EST
Blocker due bug 441684 regression.
Comment 14 Lars Vogel CLA 2019-01-14 09:01:43 EST
Thanks, Julian for the fast fix.
Comment 15 Vikas Chandra CLA 2019-02-15 04:49:36 EST
Can bug reporter or bug assignee please verify this defect for 4.11M3?
Comment 16 Lars Vogel CLA 2019-02-15 04:53:47 EST
Works for me.