Bug 1732489

Summary: Custom Button: button with dialog on Container Provider after execution lands on Infrastructure Providers page
Product: Red Hat CloudForms Management Engine Reporter: Nikhil Dhandre <ndhandre>
Component: UI - OPSAssignee: Martin Maroši <mmarosi>
Status: CLOSED CURRENTRELEASE QA Contact: Nikhil Dhandre <ndhandre>
Severity: high Docs Contact: Red Hat CloudForms Documentation <cloudforms-docs>
Priority: medium    
Version: 5.10.7CC: bmidwood, dmetzger, hkataria, lavenel, mpovolny, obarenbo, simaishi
Target Milestone: GAKeywords: TestOnly, ZStream
Target Release: 5.11.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: 5.11.0.17 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1733376 (view as bug list) Environment:
Last Closed: 2019-12-13 14:57:59 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: Bug
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: CFME Core Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1733376    

Description Nikhil Dhandre 2019-07-23 12:53:17 UTC 
Description of problem:


Version-Release number of selected component (if applicable):
510 and 511

How reproducible:
always


Steps to Reproduce:
1. Create simple textbox dialog
2. Create a button with dialog on Provider
3. Now just execute button on Container Provider


Actual results:
land on infra provider page

Expected results:
-----------------
proper flash message and land on details or all page

Additional info:
similar to: 
https://bugzilla.redhat.com/show_bug.cgi?id=1732436
Comment 4 CFME Bot 2019-07-25 08:51:28 UTC 
https://github.com/ManageIQ/manageiq-ui-classic/pull/5869
Comment 5 CFME Bot 2019-07-25 10:41:38 UTC 
New commit detected on ManageIQ/manageiq-ui-classic/master:

https://github.com/ManageIQ/manageiq-ui-classic/commit/15f7d0896eda6d9d28d1266a6cdc52d1680c0811
commit 15f7d0896eda6d9d28d1266a6cdc52d1680c0811
Author:     Martin Marosi <marvusm.mmi>
AuthorDate: Thu Jul 25 04:38:19 2019 -0400
Commit:     Martin Marosi <marvusm.mmi>
CommitDate: Thu Jul 25 04:38:19 2019 -0400

    Fixed container provider custom button dialog redirect.

    fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1732489

 app/services/dialog_local_service.rb | 2 +
 1 file changed, 2 insertions(+)
Comment 7 CFME Bot 2019-07-25 20:56:48 UTC 
New commit detected on ManageIQ/manageiq-ui-classic/ivanchuk:

https://github.com/ManageIQ/manageiq-ui-classic/commit/7f2d89105a339b82677c8b515f19cde018fb1124
commit 7f2d89105a339b82677c8b515f19cde018fb1124
Author:     Milan Zázrivec <mzazrivec>
AuthorDate: Thu Jul 25 06:39:28 2019 -0400
Commit:     Milan Zázrivec <mzazrivec>
CommitDate: Thu Jul 25 06:39:28 2019 -0400

    Merge pull request #5869 from Hyperkid123/fix-emx-container-dialog-redirect

    Fixed container provider custom button dialog redirect.

    (cherry picked from commit 705928522644c1adec4fc63bd94b57869b4085d3)

    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1732489

 app/services/dialog_local_service.rb | 2 +
 1 file changed, 2 insertions(+)
Comment 8 Nikhil Dhandre 2019-08-08 07:49:12 UTC 
Tested with 5.11.0.18; working fine landing on container provider page after dialog execution.