WordPress.org

Make WordPress Core

Opened 6 years ago

Closed 10 months ago

#18518 closed defect (bug) (wontfix)

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

Reported by: 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 6 years ago.

Download all attachments as: .zip

Change History (8)

@scribu
6 years ago

#1 @scribu
6 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
6 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
6 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
6 years ago

  • Type changed from defect (bug) to enhancement

#5 @chriscct7
3 years ago

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

Needs to be refreshed

#6 @DrewAPicture
2 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
10 months 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.