Make WordPress Core

Opened 13 years ago

Closed 7 years ago

#18518 closed defect (bug) (wontfix)

suggest.js: don't store the URL in the options object

Reported by: scribu's profile scribu Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: General Keywords:
Focuses: javascript Cc:

Description

...because it prevents re-using the same instance for different URLs:

var suggest_args = {
	multiple     : true,
	delimiter    : ',',
	resultsClass : 'qc-suggest-results',
	selectClass  : 'qc-suggest-over',
	matchClass   : 'qc-suggest-match',
};

$( '#ticket-assign input' ).suggest( QC_L10N.ajaxurl + '?action=qc-user-search', suggest_args );

$( '#ticket-tags input' ).suggest( QC_L10N.ajaxurl + '?action=ajax-tag-search&tax=post_tag', suggest_args );

With the above code, both inputs will look for items in the same URL: the second one.

Attached patch fixes this.

Attachments (1)

18518.diff (1.4 KB) - added by scribu 13 years ago.

Download all attachments as: .zip

Change History (8)

@scribu
13 years ago

#1 @scribu
13 years ago

  • Summary changed from Don't store the URL in the options object to suggest.js: don't store the URL in the options object

#2 @GaryJ
13 years ago

Will that extra argument be backwards compatible?

Could you add some code such that if source is an object rather than a string (i.e. only two arguments passed in), that source = options.source?

#3 @scribu
13 years ago

That extra argument only affects the internal constructor function, which shouldn't be used directly.

If it's really necessary, sure, you could check if it's an object vs. a string or check the number of arguments passed.

#4 @sorich87
12 years ago

  • Type changed from defect (bug) to enhancement

#5 @chriscct7
9 years ago

  • Keywords needs-refresh added; has-patch removed
  • Type changed from enhancement to defect (bug)

Needs to be refreshed

#6 @DrewAPicture
9 years ago

  • Focuses javascript added

@markjaquith: Could you please take a look at this and give it a yea or nay? It's been out there for four years, not sure if it's still worth pursuing.

#7 @obenland
7 years ago

  • Keywords needs-refresh removed
  • Milestone Awaiting Review deleted
  • Resolution set to wontfix
  • Status changed from new to closed

suggest.js is unused in Core as of [38797].

Note: See TracTickets for help on using tickets.