Bug 1283745 - UI ERROR When bulk deleting copied methods.
Summary: UI ERROR When bulk deleting copied methods.
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - OPS
Version: 5.5.0
Hardware: All
OS: All
unspecified
high
Target Milestone: GA
: 5.5.0
Assignee: Martin Hradil
QA Contact: Milan Falešník
URL:
Whiteboard:
: 1284926 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-11-19 17:26 UTC by Josh Carter
Modified: 2018-03-13 10:45 UTC (History)
11 users (show)

Fixed In Version: 5.5.0.13
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-12-08 13:49:24 UTC
Category: ---
Cloudforms Team: ---
Target Upstream Version:
Embargoed:
dajohnso: automate_bug?


Attachments (Terms of Use)
Automate blowup (87.86 KB, image/png)
2015-11-25 11:13 UTC, Pete Savage
no flags Details
Automate blowup2 (48.16 KB, image/png)
2015-11-25 11:14 UTC, Pete Savage
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHSA-2015:2551 0 normal SHIPPED_LIVE Moderate: CFME 5.5.0 bug fixes and enhancement update 2015-12-08 17:58:09 UTC

Description Josh Carter 2015-11-19 17:26:30 UTC
Description of problem:

When attempting to delete a group of methods that were copied from another automate domain causes a UI blow up. 

[----] I, [2015-11-19T12:22:34.589499 #37164:ce7994]  INFO -- : Started POST "/miq_ae_class/tree_select/?id=aec-99r103" for 127.0.0.1 at 2015-11-19 12:22:34 -0500
[----] I, [2015-11-19T12:22:34.590979 #37164:ce7994]  INFO -- : Processing by MiqAeClassController#tree_select as JS
[----] I, [2015-11-19T12:22:34.591060 #37164:ce7994]  INFO -- :   Parameters: {"id"=>"aec-99r103"}
[----] I, [2015-11-19T12:22:34.777769 #37164:ce7994]  INFO -- :   Rendered layouts/_flash_msg.html.haml (0.1ms)
[----] I, [2015-11-19T12:22:34.783478 #37164:ce7994]  INFO -- :   Rendered miq_ae_class/_class_instances.html.haml (5.9ms)
[----] I, [2015-11-19T12:22:34.783854 #37164:ce7994]  INFO -- :   Rendered layouts/_flash_msg.html.haml (0.0ms)
[----] I, [2015-11-19T12:22:34.784457 #37164:ce7994]  INFO -- :   Rendered miq_ae_class/_class_methods.html.haml (0.7ms)
[----] I, [2015-11-19T12:22:34.784819 #37164:ce7994]  INFO -- :   Rendered layouts/_flash_msg.html.haml (0.0ms)
[----] I, [2015-11-19T12:22:34.785582 #37164:ce7994]  INFO -- :   Rendered miq_ae_class/_domain_overrides.html.haml (0.4ms)
[----] I, [2015-11-19T12:22:34.785676 #37164:ce7994]  INFO -- :   Rendered miq_ae_class/_class_props.html.haml (1.0ms)
[----] I, [2015-11-19T12:22:34.786019 #37164:ce7994]  INFO -- :   Rendered layouts/_flash_msg.html.haml (0.0ms)
[----] I, [2015-11-19T12:22:34.789517 #37164:ce7994]  INFO -- :   Rendered miq_ae_class/_class_fields.html.haml (3.7ms)
[----] I, [2015-11-19T12:22:34.789679 #37164:ce7994]  INFO -- :   Rendered miq_ae_class/_all_tabs.html.haml (12.8ms)
[----] I, [2015-11-19T12:22:34.802443 #37164:ce7994]  INFO -- : Completed 200 OK in 211ms (Views: 0.2ms | ActiveRecord: 14.0ms)
[----] I, [2015-11-19T12:22:37.973454 #37164:ce7994]  INFO -- : Started POST "/miq_ae_class/change_tab/?tab_id=methods" for 127.0.0.1 at 2015-11-19 12:22:37 -0500
[----] I, [2015-11-19T12:22:37.975093 #37164:ce7994]  INFO -- : Processing by MiqAeClassController#change_tab as JS
[----] I, [2015-11-19T12:22:37.975162 #37164:ce7994]  INFO -- :   Parameters: {"tab_id"=>"methods"}
[----] I, [2015-11-19T12:22:38.043310 #37164:ce7994]  INFO -- :   Rendered layouts/_flash_msg.html.haml (0.2ms)
[----] I, [2015-11-19T12:22:38.051818 #37164:ce7994]  INFO -- : Completed 200 OK in 77ms (Views: 3.0ms | ActiveRecord: 5.8ms)
[----] I, [2015-11-19T12:22:55.253383 #37164:ce7994]  INFO -- : Started POST "/miq_ae_class/x_button/99000000000103?pressed=miq_ae_method_delete" for 127.0.0.1 at 2015-11-19 12:22:55 -0500
[----] I, [2015-11-19T12:22:55.255013 #37164:ce7994]  INFO -- : Processing by MiqAeClassController#x_button as JS
[----] I, [2015-11-19T12:22:55.255085 #37164:ce7994]  INFO -- :   Parameters: {"miq_grid_checks"=>"", "pressed"=>"miq_ae_method_delete", "id"=>"99000000000103"}
[----] F, [2015-11-19T12:22:55.293035 #37164:ce7994] FATAL -- : Error caught: [NoMethodError] undefined method `class_id' for nil:NilClass
/var/www/miq/vmdb/app/controllers/miq_ae_class_controller.rb:1894:in `deletemethods'
/var/www/miq/vmdb/app/controllers/miq_ae_class_controller.rb:74:in `x_button'
/opt/rh/cfme-gemset/bundler/gems/rails-a37d0e57571f/actionpack/lib/action_controller/metal/implicit_render.rb:4:in `send_action'
/opt/rh/cfme-gemset/bundler/gems/rails-a37d0e57571f/actionpack/lib/abstract_controller/base.rb:198:in `process_action'
/opt/rh/cfme-gemset/bundler/gems/rails-a37d0e57571f/actionpack/lib/action_controller/metal/rendering.rb:10:in `process_action'
/opt/rh/cfme-gemset/bundler/gems/rails-a37d0e57571f/actionpack/lib/abstract_controller/callbacks.rb:20:in `block in process_action'
/opt/rh/cfme-gemset/bundler/gems/rails-a37d0e57571f/activesupport/lib/active_support/callbacks.rb:117:in `call'
/opt/rh/cfme-gemset/bundler/gems/rails-a37d0e57571f/activesupport/lib/active_support/callbacks.rb:117:in `call'
/opt/rh/cfme-gemset/bundler/gems/rails-a37d0e57571f/activesupport/lib/active_support/callbacks.rb:555:in `block (2 levels) in compile'
/opt/rh/cfme-gemset/bundler/gems/rails-a37d0e57571f/activesupport/lib/active_support/callbacks.rb:505:in `call'
/opt/rh/cfme-gemset/bundler/gems/rails-a37d0e57571f/activesupport/lib/active_support/callbacks.rb:505:in `call'
/opt/rh/cfme-gemset/bundler/gems/rails-a37d0e57571f/activesupport/lib/active_support/callbacks.rb:92:in `__run_callbacks__'
/opt/rh/cfme-gemset/bundler/gems/rails-a37d0e57571f/activesupport/lib/active_support/callbacks.rb:778:in `_run_process_action_callbacks'
/opt/rh/cfme-gemset/bundler/gems/rails-a37d0e57571f/activesupport/lib/active_support/callbacks.rb:81:in `run_callbacks'
/opt/rh/cfme-gemset/bundler/gems/rails-a37d0e57571f/actionpack/lib/abstract_controller/callbacks.rb:19:in `process_action'
/opt/rh/cfme-gemset/bundler/gems/rails-a37d0e57571f/actionpack/lib/action_controller/metal/rescue.rb:29:in `process_action'
/opt/rh/cfme-gemset/bundler/gems/rails-a37d0e57571f/actionpack/lib/action_controller/metal/instrumentation.rb:32:in `block in process_action'


Version-Release number of selected component (if applicable):
5.5.0.9 beta 2

How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 1 Josh Carter 2015-11-19 17:29:32 UTC
1. Create a new domain.
2. Create a namespace
3. Create a state machine namespace
4. Copied the class from the state machine namespace
5. Copied the instance from the state machine class 
6. Copied each method to that new state machine namespace
7. Bulk delete methods within that new state machine namespace triggers the ui blowup

Comment 3 Fedora Update System 2015-11-21 00:38:34 UTC
pesign-0.111-3.fc21 has been submitted as an update to Fedora 21. https://bodhi.fedoraproject.org/updates/FEDORA-2015-8a82827699

Comment 4 Fedora Update System 2015-11-21 23:23:26 UTC
pesign-0.111-3.fc21 has been pushed to the Fedora 21 testing repository. If problems still persist, please make note of it in this bug report.
If you want to test the update, you can install it with
$ su -c 'dnf --enablerepo=updates-testing update pesign'
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-8a82827699

Comment 5 Harpreet Kataria 2015-11-23 15:34:06 UTC
Josh,

Can you try this with latest build, i am unable to recreate the issue on latest upstream master. Let me know if you see this issue on latest build.

Thanks,
~Harpreet

Comment 6 Pete Savage 2015-11-25 11:09:19 UTC
This still happens for me I tried it multiple times, you can see the items on the top left of the screen shots, those were the methods I was trying to delete, each time I did so, I got a different answer, the first time it said they were deleted, and they weren't. The second time it said they were deleted and it appeared to be deleting another method/class entirely, the third time I got a UI blow up.

Comment 7 Pete Savage 2015-11-25 11:13:17 UTC
[----] I, [2015-11-25T06:07:12.089736 #2878:f7d988]  INFO -- : Started POST "/miq_ae_class/x_button/102?pressed=miq_ae_method_delete" for 127.0.0.1 at 2015-11-25 06:07:12 -0500
[----] I, [2015-11-25T06:07:12.091139 #2878:f7d988]  INFO -- : Processing by MiqAeClassController#x_button as JS
[----] I, [2015-11-25T06:07:12.091219 #2878:f7d988]  INFO -- :   Parameters: {"miq_grid_checks"=>"", "pressed"=>"miq_ae_method_delete", "id"=>"102"}
[----] F, [2015-11-25T06:07:12.132175 #2878:f7d988] FATAL -- : Error caught: [NoMethodError] undefined method `class_id' for nil:NilClass
/var/www/miq/vmdb/app/controllers/miq_ae_class_controller.rb:1894:in `deletemethods'
/var/www/miq/vmdb/app/controllers/miq_ae_class_controller.rb:74:in `x_button'
/opt/rh/cfme-gemset/gems/actionpack-4.2.5/lib/action_controller/metal/implicit_render.rb:4:in `send_action'
/opt/rh/cfme-gemset/gems/actionpack-4.2.5/lib/abstract_controller/base.rb:198:in `process_action'
/opt/rh/cfme-gemset/gems/actionpack-4.2.5/lib/action_controller/metal/rendering.rb:10:in `process_action'
/opt/rh/cfme-gemset/gems/actionpack-4.2.5/lib/abstract_controller/callbacks.rb:20:in `block in process_action'
/opt/rh/cfme-gemset/gems/activesupport-4.2.5/lib/active_support/callbacks.rb:117:in `call'
/opt/rh/cfme-gemset/gems/activesupport-4.2.5/lib/active_support/callbacks.rb:117:in `call'
/opt/rh/cfme-gemset/gems/activesupport-4.2.5/lib/active_support/callbacks.rb:555:in `block (2 levels) in compile'
/opt/rh/cfme-gemset/gems/activesupport-4.2.5/lib/active_support/callbacks.rb:505:in `call'
/opt/rh/cfme-gemset/gems/activesupport-4.2.5/lib/active_support/callbacks.rb:505:in `call'
/opt/rh/cfme-gemset/gems/activesupport-4.2.5/lib/active_support/callbacks.rb:92:in `__run_callbacks__'
/opt/rh/cfme-gemset/gems/activesupport-4.2.5/lib/active_support/callbacks.rb:778:in `_run_process_action_callbacks'
/opt/rh/cfme-gemset/gems/activesupport-4.2.5/lib/active_support/callbacks.rb:81:in `run_callbacks'
/opt/rh/cfme-gemset/gems/actionpack-4.2.5/lib/abstract_controller/callbacks.rb:19:in `process_action'
/opt/rh/cfme-gemset/gems/actionpack-4.2.5/lib/action_controller/metal/rescue.rb:29:in `process_action'
/opt/rh/cfme-gemset/gems/actionpack-4.2.5/lib/action_controller/metal/instrumentation.rb:32:in `block in process_action'
/opt/rh/cfme-gemset/gems/activesupport-4.2.5/lib/active_support/notifications.rb:164:in `block in instrument'
/opt/rh/cfme-gemset/gems/activesupport-4.2.5/lib/active_support/notifications/instrumenter.rb:20:in `instrument'
/opt/rh/cfme-gemset/gems/activesupport-4.2.5/lib/active_support/notifications.rb:164:in `instrument'
/opt/rh/cfme-gemset/gems/actionpack-4.2.5/lib/action_controller/metal/instrumentation.rb:30:in `process_action'
/opt/rh/cfme-gemset/gems/actionpack-4.2.5/lib/action_controller/metal/params_wrapper.rb:250:in `process_action'
/opt/rh/cfme-gemset/gems/activerecord-4.2.5/lib/active_record/railties/controller_runtime.rb:18:in `process_action'
/opt/rh/cfme-gemset/gems/actionpack-4.2.5/lib/abstract_controller/base.rb:137:in `process'
/opt/rh/cfme-gemset/gems/actionview-4.2.5/lib/action_view/rendering.rb:30:in `process'
/opt/rh/cfme-gemset/gems/actionpack-4.2.5/lib/action_controller/metal.rb:196:in `dispatch'
/opt/rh/cfme-gemset/gems/actionpack-4.2.5/lib/action_controller/metal/rack_delegation.rb:13:in `dispatch'
/opt/rh/cfme-gemset/gems/actionpack-4.2.5/lib/action_controller/metal.rb:237:in `block in action'
/opt/rh/cfme-gemset/gems/actionpack-4.2.5/lib/action_dispatch/routing/route_set.rb:76:in `call'
/opt/rh/cfme-gemset/gems/actionpack-4.2.5/lib/action_dispatch/routing/route_set.rb:76:in `dispatch'
/opt/rh/cfme-gemset/gems/actionpack-4.2.5/lib/action_dispatch/routing/route_set.rb:45:in `serve'
/opt/rh/cfme-gemset/gems/actionpack-4.2.5/lib/action_dispatch/journey/router.rb:43:in `block in serve'
/opt/rh/cfme-gemset/gems/actionpack-4.2.5/lib/action_dispatch/journey/router.rb:30:in `each'
/opt/rh/cfme-gemset/gems/actionpack-4.2.5/lib/action_dispatch/journey/router.rb:30:in `serve'
/opt/rh/cfme-gemset/gems/actionpack-4.2.5/lib/action_dispatch/routing/route_set.rb:817:in `call'
/opt/rh/cfme-gemset/gems/rack-1.6.4/lib/rack/etag.rb:24:in `call'
/opt/rh/cfme-gemset/gems/rack-1.6.4/lib/rack/conditionalget.rb:38:in `call'
/opt/rh/cfme-gemset/gems/rack-1.6.4/lib/rack/head.rb:13:in `call'
/opt/rh/cfme-gemset/gems/actionpack-4.2.5/lib/action_dispatch/middleware/params_parser.rb:27:in `call'
/opt/rh/cfme-gemset/gems/actionpack-4.2.5/lib/action_dispatch/middleware/flash.rb:260:in `call'
/opt/rh/cfme-gemset/gems/rack-1.6.4/lib/rack/session/abstract/id.rb:225:in `context'
/opt/rh/cfme-gemset/gems/rack-1.6.4/lib/rack/session/abstract/id.rb:220:in `call'
/opt/rh/cfme-gemset/gems/actionpack-4.2.5/lib/action_dispatch/middleware/cookies.rb:560:in `call'
/opt/rh/cfme-gemset/gems/activerecord-4.2.5/lib/active_record/query_cache.rb:36:in `call'
/opt/rh/cfme-gemset/gems/activerecord-4.2.5/lib/active_record/connection_adapters/abstract/connection_pool.rb:653:in `call'
/opt/rh/cfme-gemset/gems/actionpack-4.2.5/lib/action_dispatch/middleware/callbacks.rb:29:in `block in call'
/opt/rh/cfme-gemset/gems/activesupport-4.2.5/lib/active_support/callbacks.rb:88:in `__run_callbacks__'
/opt/rh/cfme-gemset/gems/activesupport-4.2.5/lib/active_support/callbacks.rb:778:in `_run_call_callbacks'
/opt/rh/cfme-gemset/gems/activesupport-4.2.5/lib/active_support/callbacks.rb:81:in `run_callbacks'
/opt/rh/cfme-gemset/gems/actionpack-4.2.5/lib/action_dispatch/middleware/callbacks.rb:27:in `call'
/opt/rh/cfme-gemset/gems/actionpack-4.2.5/lib/action_dispatch/middleware/remote_ip.rb:78:in `call'
/opt/rh/cfme-gemset/gems/actionpack-4.2.5/lib/action_dispatch/middleware/debug_exceptions.rb:17:in `call'
/opt/rh/cfme-gemset/gems/actionpack-4.2.5/lib/action_dispatch/middleware/show_exceptions.rb:30:in `call'
/opt/rh/cfme-gemset/gems/railties-4.2.5/lib/rails/rack/logger.rb:38:in `call_app'
/opt/rh/cfme-gemset/gems/railties-4.2.5/lib/rails/rack/logger.rb:22:in `call'
/opt/rh/cfme-gemset/gems/actionpack-4.2.5/lib/action_dispatch/middleware/request_id.rb:21:in `call'
/opt/rh/cfme-gemset/gems/rack-1.6.4/lib/rack/methodoverride.rb:22:in `call'
/opt/rh/cfme-gemset/gems/rack-1.6.4/lib/rack/runtime.rb:18:in `call'
/opt/rh/cfme-gemset/gems/activesupport-4.2.5/lib/active_support/cache/strategy/local_cache_middleware.rb:28:in `call'
/opt/rh/cfme-gemset/gems/rack-1.6.4/lib/rack/lock.rb:17:in `call'
/opt/rh/cfme-gemset/gems/actionpack-4.2.5/lib/action_dispatch/middleware/static.rb:116:in `call'
/opt/rh/cfme-gemset/gems/actionpack-4.2.5/lib/action_dispatch/middleware/static.rb:116:in `call'
/opt/rh/cfme-gemset/gems/rack-1.6.4/lib/rack/sendfile.rb:113:in `call'
/opt/rh/cfme-gemset/gems/railties-4.2.5/lib/rails/engine.rb:518:in `call'
/opt/rh/cfme-gemset/gems/railties-4.2.5/lib/rails/application.rb:165:in `call'
/opt/rh/cfme-gemset/gems/rack-1.6.4/lib/rack/content_length.rb:15:in `call'
/opt/rh/rh-ruby22/root/usr/share/gems/gems/thin-1.6.3/lib/thin/connection.rb:86:in `block in pre_process'
/opt/rh/rh-ruby22/root/usr/share/gems/gems/thin-1.6.3/lib/thin/connection.rb:84:in `catch'
/opt/rh/rh-ruby22/root/usr/share/gems/gems/thin-1.6.3/lib/thin/connection.rb:84:in `pre_process'
/opt/rh/rh-ruby22/root/usr/share/gems/gems/thin-1.6.3/lib/thin/connection.rb:53:in `process'
/opt/rh/rh-ruby22/root/usr/share/gems/gems/thin-1.6.3/lib/thin/connection.rb:39:in `receive_data'
/opt/rh/rh-ruby22/root/usr/share/gems/gems/eventmachine-1.0.7/lib/eventmachine.rb:187:in `run_machine'
/opt/rh/rh-ruby22/root/usr/share/gems/gems/eventmachine-1.0.7/lib/eventmachine.rb:187:in `run'
/opt/rh/rh-ruby22/root/usr/share/gems/gems/thin-1.6.3/lib/thin/backends/base.rb:73:in `start'
/opt/rh/rh-ruby22/root/usr/share/gems/gems/thin-1.6.3/lib/thin/server.rb:162:in `start'
/opt/rh/cfme-gemset/gems/rack-1.6.4/lib/rack/handler/thin.rb:19:in `run'
/opt/rh/cfme-gemset/gems/rack-1.6.4/lib/rack/server.rb:286:in `start'
/opt/rh/cfme-gemset/gems/railties-4.2.5/lib/rails/commands/server.rb:80:in `start'
/opt/rh/cfme-gemset/gems/railties-4.2.5/lib/rails/commands/commands_tasks.rb:80:in `block in server'
/opt/rh/cfme-gemset/gems/railties-4.2.5/lib/rails/commands/commands_tasks.rb:75:in `tap'
/opt/rh/cfme-gemset/gems/railties-4.2.5/lib/rails/commands/commands_tasks.rb:75:in `server'
/opt/rh/cfme-gemset/gems/railties-4.2.5/lib/rails/commands/commands_tasks.rb:39:in `run_command!'
/opt/rh/cfme-gemset/gems/railties-4.2.5/lib/rails/commands.rb:17:in `<top (required)>'
/var/www/miq/vmdb/bin/rails:4:in `require'
/var/www/miq/vmdb/bin/rails:4:in `<main>'

Comment 8 Pete Savage 2015-11-25 11:13:51 UTC
Created attachment 1098715 [details]
Automate blowup

Comment 9 Pete Savage 2015-11-25 11:14:29 UTC
Created attachment 1098717 [details]
Automate blowup2

Comment 10 Harpreet Kataria 2015-11-25 14:55:39 UTC
*** Bug 1284926 has been marked as a duplicate of this bug. ***

Comment 11 Harpreet Kataria 2015-11-25 15:07:01 UTC
Martin,

This seems to have been introduced either with grid conversion or by other refactoring in JS. When selecting methods form list of methods, it does not send up selected method's ids upto server.

Let me know if you have any questions & or need help.

Thanks,
~Harpreet

Comment 14 CFME Bot 2015-11-25 19:05:36 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/f3620989a281bbb8ea1ca06f389b50bc402b038e

commit f3620989a281bbb8ea1ca06f389b50bc402b038e
Author:     Martin Hradil <mhradil>
AuthorDate: Wed Nov 25 18:19:52 2015 +0000
Commit:     Martin Hradil <mhradil>
CommitDate: Wed Nov 25 18:19:52 2015 +0000

    Replace #miq_grid_checks with ManageIQ.gridChecks
    
    the miq_grid_checks logic failed when more than one list was present in the DOM at the same time (multiple #miq_grid_checks elements -> jquery retrieved random one)
    
    This changes the affected code to use a global variable instead of an element.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1283745

 app/assets/javascripts/miq_application.js | 6 +++---
 app/assets/javascripts/miq_global.js      | 1 +
 app/assets/javascripts/miq_grid.js        | 9 ++-------
 app/assets/javascripts/miq_list_grid.js   | 2 +-
 app/views/layouts/_list_grid.html.haml    | 6 ++----
 spec/javascripts/miq_grid_spec.js         | 9 ++-------
 6 files changed, 11 insertions(+), 22 deletions(-)

Comment 15 CFME Bot 2015-11-25 21:04:00 UTC
New commit detected on cfme/5.5.z:
https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=f40b66448931db4da7cc5323ee881c0a1a220705

commit f40b66448931db4da7cc5323ee881c0a1a220705
Merge: c1e0884 e4f2b79
Author:     Harpreet Kataria <hkataria>
AuthorDate: Wed Nov 25 15:19:44 2015 -0500
Commit:     Harpreet Kataria <hkataria>
CommitDate: Wed Nov 25 15:19:44 2015 -0500

    Merge branch 'bz1283745-ae-methods-55' into '5.5.z'
    
    Replace #miq_grid_checks with ManageIQ.gridChecks
    
    the miq_grid_checks logic failed when more than one list was present in the DOM at the same time (multiple #miq_grid_checks elements -> jquery retrieved random one)
    
    This changes the affected code to use a global variable instead of an element.
    
    https://github.com/ManageIQ/manageiq/pull/5604
    https://bugzilla.redhat.com/show_bug.cgi?id=1283745
    
    clean cherry pick
    
    See merge request !540

 app/assets/javascripts/miq_application.js  | 17 +++++++++--------
 app/assets/javascripts/miq_global.js       |  1 +
 app/assets/javascripts/miq_grid.js         |  9 ++-------
 app/assets/javascripts/miq_list_grid.js    |  2 +-
 app/controllers/application_controller.rb  |  2 +-
 app/controllers/miq_ae_class_controller.rb |  7 ++-----
 app/views/layouts/_list_grid.html.haml     |  6 ++----
 spec/javascripts/miq_grid_spec.js          |  9 ++-------
 8 files changed, 20 insertions(+), 33 deletions(-)

Comment 16 CFME Bot 2015-11-25 21:04:06 UTC
New commit detected on cfme/5.5.z:
https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=e4f2b79267a8cb2e48e2778f73029a241c38200e

commit e4f2b79267a8cb2e48e2778f73029a241c38200e
Author:     Martin Hradil <mhradil>
AuthorDate: Wed Nov 25 18:19:52 2015 +0000
Commit:     Martin Hradil <mhradil>
CommitDate: Wed Nov 25 19:01:46 2015 +0000

    Replace #miq_grid_checks with ManageIQ.gridChecks
    
    the miq_grid_checks logic failed when more than one list was present in the DOM at the same time (multiple #miq_grid_checks elements -> jquery retrieved random one)
    
    This changes the affected code to use a global variable instead of an element.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1283745
    (cherry picked from commit f3620989a281bbb8ea1ca06f389b50bc402b038e)

 app/assets/javascripts/miq_application.js | 6 +++---
 app/assets/javascripts/miq_global.js      | 1 +
 app/assets/javascripts/miq_grid.js        | 9 ++-------
 app/assets/javascripts/miq_list_grid.js   | 2 +-
 app/views/layouts/_list_grid.html.haml    | 6 ++----
 spec/javascripts/miq_grid_spec.js         | 9 ++-------
 6 files changed, 11 insertions(+), 22 deletions(-)

Comment 18 errata-xmlrpc 2015-12-08 13:49:24 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2015:2551


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