← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1327276] Re: filter function doesn't handle some special characters

 

Reviewed:  https://review.openstack.org/113932
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=0ce5c85f9ee124fd69deff57d3cf365903b0a299
Submitter: Jenkins
Branch:    master

commit 0ce5c85f9ee124fd69deff57d3cf365903b0a299
Author: Justin Pomeroy <jpomero@xxxxxxxxxxxxxxxxxx>
Date:   Wed Aug 13 10:14:38 2014 -0500

    Escape RegEx characters from table quick search text
    
    This fix escapes any regular expression characters from the table
    quick search text so that the filter works as the user would
    expect.
    
    A new string utility module is created for the legacy javascript
    string utils and this new function for escaping regex chars is
    placed there along with the existing function for escaping HTML
    chars.
    
    Closes-Bug: #1327276
    Change-Id: I514bbe7ae9e1f319ced693e9a3da2881a86d4f57


** Changed in: horizon
       Status: In Progress => Fix Released

-- 
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/1327276

Title:
  filter function doesn't handle some special characters

Status in OpenStack Dashboard (Horizon):
  Fix Released

Bug description:
  == Steps to Reproduce ==

  1. Project > Volumes > Create Volume
  2. In Description box, put in '$24.00 ^ ?" and fill out required fields.
  3. Press 'Create Volume' button
  4. Do a search for '$', '^', '?' (other special characters seem to work fine)
  ==> The table remains the same when it should filter out the other rows.

  See attachment.

  
  We are using the jQuery quicksearch plugin to handle filtering.

  In example here:
  http://deuxhuithuit.github.io/quicksearch/r/examples/, at least '$'
  seems to work properly with the basic plugin, so maybe our testQuery()
  function in horizon.tables.js is missing something?

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


References