This project is read-only.

A small bug ?

Nov 12, 2010 at 12:22 PM

Hi there,

I was going though the plugin and at line 304 , I saw this :

if (data[o.resultsProperty][i].name.toLowerCase().indexOf(params.q.toLowerCase()) === 0) {

now, why do you use .name , instead of  data[o.resultProperty][i][o.displayValue] ?



Nov 12, 2010 at 10:37 PM

You’re absolutely right, good catch!

I’ve updated the code, and the new version is here: http://flexbox.codeplex.com/SourceControl/changeset/changes/3f412607df7b

Thanks,

Noah

From: roxin [mailto:notifications@codeplex.com]
Sent: Friday, November 12, 2010 4:22 AM
To: Noah Heldman
Subject: A small bug ? [flexbox:234444]

From: roxin

Hi there,

I was going though the plugin and at line 304 , I saw this :

if (data[o.resultsProperty][i].name.toLowerCase().indexOf(params.q.toLowerCase()) === 0) {
 

now, why do you use .name , instead of data[o.resultProperty][i][o.displayValue] ?

 

Read the full discussion online.

To add a post to this discussion, reply to this email (flexbox@discussions.codeplex.com)

To start a new discussion for this project, email flexbox@discussions.codeplex.com

You are receiving this email because you subscribed to this discussion on CodePlex. You can unsubscribe or change your settings on codePlex.com.

Please note: Images and attachments will be removed from emails. Any posts to this discussion will also be available online at codeplex.com

Nov 13, 2010 at 3:47 AM

Great, thank you for the quick reply.