yahoo-eng-team team mailing list archive

[Kirill Zaitsev <kzaitsev@xxxxxxxxxxxx>]

Public bug reported:


The offending code is found in horizon/static/horizon/js/horizon.tables.js

      if($row.find('.table-row-multi-select')[0].checked)

In case the table is multi_valued=False there would be no '.table-row-
multi-select' element. [0] would return undefined and accessing .checked
would raise an error.

This was introduced by https://review.openstack.org/#/c/252161/ Before
this commit this access was guarded by jquery helper method.

** Affects: horizon
     Importance: Undecided
         Status: New


** Tags: js

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/1555752

Title:
  AJAX Row updates are borken for non-multi-select tables

Status in OpenStack Dashboard (Horizon):
  New

Bug description:
  
  The offending code is found in horizon/static/horizon/js/horizon.tables.js

        if($row.find('.table-row-multi-select')[0].checked)

  In case the table is multi_valued=False there would be no '.table-row-
  multi-select' element. [0] would return undefined and accessing
  .checked would raise an error.

  This was introduced by https://review.openstack.org/#/c/252161/ Before
  this commit this access was guarded by jquery helper method.

To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1555752/+subscriptions