Bug 1540855 - [UI][RHOS] - remove Edit and Delete actions when in the SDN list view
Summary: [UI][RHOS] - remove Edit and Delete actions when in the SDN list view
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - OPS
Version: 5.8.0
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: GA
: 5.9.0
Assignee: Harpreet Kataria
QA Contact: Pavol Kotvan
URL:
Whiteboard: ui:rhos
Depends On:
Blocks: 1544488
TreeView+ depends on / blocked
 
Reported: 2018-02-01 07:46 UTC by Pavol Kotvan
Modified: 2018-07-09 12:36 UTC (History)
10 users (show)

Fixed In Version: 5.9.0.20
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1544488 (view as bug list)
Environment:
Last Closed: 2018-03-06 15:08:36 UTC
Category: Bug
Cloudforms Team: Openstack
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Comment 5 Dave Johnson 2018-02-01 16:08:52 UTC
Pavol, does 5.9 show the same behavior?

Comment 6 Martin Povolny 2018-02-01 17:41:21 UTC
Marek, what happened here is that the functionality was implemented for OpenStack. It works for OpenStack.

But it was enabled for both the single-item case (with proper checking if the particular action supports given action) and also for the list of items case.

In the list of items case no logic that would check the item type and support for the particular action was added.

That means that when you have a mix of items e.g. OpenStack + EC2 the functionality is broken. The existing code crashes on EC2.

To fix that we removed the buttons from the list case and left them just for the single-item case.

Examples:
https://github.com/ManageIQ/manageiq-ui-classic/pull/3349
https://github.com/ManageIQ/manageiq-ui-classic/pull/2234

To enable the buttons in the list requires implementing the missing checking of items. We (me and Ladas) decided not to do that because of how late in the release.

This would not be about fixing bugs but about implementing missing functionality.

Furthermore the code is ugly. It needs cleanup so that it can be worked with.

I a going to PTO for a week I'll refer you to Ladas if further explanation is needed.

I suggest against enabling the buttons for gaprindashvili. Seeing the level of cut-n-paste by the implementors of the problematic functionality I think that the core team needs to provide a generic way of checking the available actions for the list of items case. We can work on that but it's at least a sprint of work.

I suggest closing this BZ as WONTFIX.


Note You need to log in before you can comment on or make changes to this bug.