Task #8778 (closed)
Bug: Inconsistent behaviour of Pasting in the Explore pane
| Reported by: | rleigh | Owned by: | wmoore |
|---|---|---|---|
| Priority: | critical | Milestone: | OMERO-4.4 |
| Component: | Web | Version: | n.a. |
| Keywords: | n.a. | Cc: | web-team@… |
| Resources: | n.a. | Referenced By: | n.a. |
| References: | n.a. | Remaining Time: | n.a. |
| Sprint: | n.a. |
Description
The copy and cut icons are greyed out when unavailable. However, the paste icon remains active even when pasting is not possible:
- Select image file and copy.
- Paste into different container types:
- Dataset (succeeds, unless it's already in the container)
- Project (fails silently, does nothing)
- Screen (fails silently, does nothing)
- Plate (fails with "Destination not supported" error)
- Select dataset and copy
- Paste into different container types:
- Dataset (copies dataset /contents/, not error as expected)
- Project (succeeds, unless duplicate)
- Screen (fails silently, does nothing)
- Plate (fails with "Destination not supported" error)
- Select screen and copy
- Does not support copying. Why can't I copy/move a screen into a Project?
- Select plate and copy
- Paste into different container types:
- Dataset (fails silently)
- Project (fails silently)
- Screen (succeeds, fails if duplicate)
- Plate (fails with error "Cannot move plate to plate")
So to summarise:
Failure is handled inconsistently, both by
- allowing paste where it will definitely fail
- allowing paste even when the icon is greyed out (it's still functional, or apparently functional), and the greying is sometimes inconsistent even for the same container type
- not producing errors in all cases where a failure to paste occurs
The behaviour should be consistent, understandable and give feedback to the user rather than silently failing.
Regards,
Roger
Change History (4)
comment:1 Changed 7 years ago by rleigh
- Summary changed from Bug: Inconsistent behavious of Pasting in the Explore pane to Bug: Inconsistent behaviour of Pasting in the Explore pane
comment:2 Changed 7 years ago by atarkowska
- Cc web-team@… added
- Owner changed from web-team@… to wmoore
comment:3 Changed 7 years ago by wmoore
- Resolution set to fixed
- Status changed from new to closed
comment:4 Changed 7 years ago by Will Moore <will@…>
(In [869972bd6c20f8ac837255c1b8ccf88ac40ae94d/ome.git] on branch develop) Only enable 'paste' if jsTree nodes are compatible. See #8778
Only allow paste on compatible types https://github.com/will-moore/openmicroscopy/commit/869972bd6c20f8ac837255c1b8ccf88ac40ae94d