Modify

Opened 2 years ago

Closed 2 years ago

#14597 closed defect (fixed)

[patch] All Inside [testing] invalid selection bug

Reported by: bagage Owned by: team
Priority: normal Milestone:
Component: Plugin utilsplugin2 Version:
Keywords: template_report Cc:

Description (last modified by Don-vip)

What steps will reproduce the problem?

  1. Select a city boundary relation
  2. Add a filter "type:node" and enable it so that nodes are not selectable
  3. Use "Select all inside [testing]" (Alt+Shift+I)
  4. Filter out (Ctrl+F) with "type:way" and tick "find in selection"

What is the expected result?

Selection contains only ways.

What happens instead?

Selection contains ways and some residual nodes (for me 750 ways, 1500 nodes). I cannot see to whose these nodes belong to.

Please provide any additional information below. Attach a screenshot if possible.

First thing, it's important to have nodes filtered out to reproduce the bug.
Second thing, you can "fix" the issue by disable the node filter, then reenable it (this will "refresh" the selection and removes nodes).
Third thing, if you have 2 layers, you can also "fix" the issue by switching between the 2 layers.

Basically anything that "refresh" the selection fix the problem.

screenshot of the issue

Build-Date:2017-03-30 17:49:38
Revision:11772
Is-Local-Build:true

Identification: JOSM/1.5 (11772 SVN en) Linux Debian GNU/Linux 9.0 (stretch)
Memory Usage: 844 MB / 1760 MB (572 MB allocated, but free)
Java version: 1.8.0_121-8u121-b13-4-b13, Oracle Corporation, OpenJDK 64-Bit Server VM
Screen: :0.0 1920x1080
Maximum Screen Size: 1920x1080
Java package: openjdk-8-jre:amd64-8u121-b13-4
Java ATK Wrapper package: libatk-wrapper-java:all-0.33.3-13
Dataset consistency test: No problems found

Plugins:
+ PicLayer (33148)
+ buildings_tools (33004)
+ conflation (0.5.2)
+ jts (32699)
+ rex (26)
+ todo (30000)
+ utilsplugin2 (33182)

Last errors/warnings:
- W: No configuration settings found.  Using hardcoded default values for all pools.
- W: Cannot start IPv4 remotecontrol server on port 8111: Address already in use (Bind failed)
- W: Cannot start IPv6 remotecontrol server on port 8111: Address already in use (Bind failed)
- W: Cannot start IPv4 remotecontrol https server on port 8112: Address already in use (Bind failed)
- W: Cannot start IPv6 remotecontrol https server on port 8112: Address already in use (Bind failed)
- W: Old style SideButton usage for action org.openstreetmap.josm.plugins.todo.TodoDialog$SelectAction@68aa73d8
- W: Old style SideButton usage for action org.openstreetmap.josm.plugins.todo.TodoDialog$AddAction@7292c0c3
- W: Old style SideButton usage for action org.openstreetmap.josm.plugins.todo.TodoDialog$PassAction@be5edae
- W: Old style SideButton usage for action org.openstreetmap.josm.plugins.todo.TodoDialog$MarkAction@6e73bcbf

Attachments (3)

cap.png (1.2 MB) - added by bagage 2 years ago.
screenshot of the issue
execute-filters.patch (3.3 KB) - added by bagage 2 years ago.
patch proposal
0001-fix-2.patch (1.5 KB) - added by bagage 2 years ago.
patch proposal

Download all attachments as: .zip

Change History (9)

Changed 2 years ago by bagage

Attachment: cap.png added

screenshot of the issue

comment:1 Changed 2 years ago by Don-vip

Description: modified (diff)

comment:2 Changed 2 years ago by Don-vip

Component: CorePlugin utilsplugin2

Changed 2 years ago by bagage

Attachment: execute-filters.patch added

patch proposal

comment:3 Changed 2 years ago by bagage

I attached a proposal patch that fixes the problem by applying filters again after adding "all inside selection".

comment:4 Changed 2 years ago by bagage

Even simpler patch borrowed from SelectAllAction.java class :).

comment:5 Changed 2 years ago by bastiK

Summary: All Inside [testing] invalid selection bug[patch] All Inside [testing] invalid selection bug

Changed 2 years ago by bagage

Attachment: 0001-fix-2.patch added

patch proposal

comment:6 Changed 2 years ago by stoecker

Resolution: fixed
Status: newclosed

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.