Ticket #2969 (closed defect: fixed)

Opened 2 years ago

Last modified 2 years ago

Some Admin AJAX tweaks

Reported by: mdawaffe Assigned to: mdawaffe
Priority: normal Milestone: 2.1
Component: Administration Version: 2.1
Severity: normal Keywords: ajax javascript listman has-patch needs-testing
Cc:

Description

Both a defect and an enhancement.

Attached:

  1. Remove wp_clean_ajax_input() from admin-ajax.php. I can't remember why it's there. It may have been necessary at one point, but now it just urldecodes strings that are already udecoded. To test, try adding the category "no amp here: %26" via ajax cat. Pre-patch, you'll see: "no amp here: &". Post-patch you'll see "no amp here: %26".
  2. Allow plugins to use admin-ajax.php by firing 'wp_ajax_$action' if there's no hardcoded case.
  3. Use full path for AJAX response handler to avoid nastiness.
  4. Allow plugins to use listMan library with their own AJAX response handler.
  5. Make listMan object a little more usable to plugin JS.
  6. Step toward getting rid of nasty eval()s.

Attachments

2969.diff (6.9 kB) - added by mdawaffe on 07/22/06 22:13:52.
tweaks to list-manipulation-js.php and admin-ajax.php

Change History

07/22/06 22:13:52 changed by mdawaffe

  • attachment 2969.diff added.

tweaks to list-manipulation-js.php and admin-ajax.php

07/22/06 22:15:07 changed by mdawaffe

  • owner changed from anonymous to mdawaffe.
  • status changed from new to assigned.

07/22/06 23:49:56 changed by mdawaffe

Oh - forgot: the changeset number for list-manipulation-js.php in script-loader.php should be updated when this goes in.

07/25/06 19:01:54 changed by ryan

  • status changed from assigned to closed.
  • resolution set to fixed.

(In [4042]) Admin AJAX tweaks from mdawaffe. fixes #2969