Opened 23 months ago

Closed 16 months ago

Last modified 15 months ago

#18126 closed task (blessed) (fixed)

Expose image options on XML-RPC

Reported by: koke Owned by: josephscott
Priority: normal Milestone: 3.4
Component: XML-RPC Version: 3.2.1
Severity: normal Keywords: mobile has-patch commit dev-reviewed
Cc: isaackeyet, ercoli@…

Description

To pick good defaults when inserting images from the mobile apps, it'd be nice to have the following options exposed:

  • image_default_link_type
  • image_default_size
  • image_default_align

Attachments (1)

path-core-18126.diff (899 bytes) - added by daniloercoli 21 months ago.

Download all attachments as: .zip

Change History (11)

  • Cc isaackeyet added
  • Cc ercoli@… added
  • Keywords has-patch added; needs-patch removed

This looks fine to me, just one question. What are the legitimate values for these options? My copy of WP -trunk has empty values for all three.

Defaults look like this to me:

mysql> select * from wp_options where option_name LIKE 'image%';
+-----------+---------+-------------------------+--------------+----------+
| option_id | blog_id | option_name             | option_value | autoload |
+-----------+---------+-------------------------+--------------+----------+
|        76 |       0 | image_default_align     |              | yes      | 
|        74 |       0 | image_default_link_type | file         | yes      | 
|        75 |       0 | image_default_size      |              | yes      | 
+-----------+---------+-------------------------+--------------+----------+

For image_default_size, trunk/wp-admin/includes/media.php#L730:

// if $check is not enabled, default to the first available size that's bigger than a thumbnail

For image_default_align, defaults to 'none' in trunk/wp-admin/includes/media.php#L682

  • Milestone changed from Awaiting Review to 3.4

Looks fine to me.

  • Type changed from enhancement to task (blessed)
  • Keywords commit dev-reviewed added
  • Owner set to josephscott
  • Status changed from new to assigned

Patch looks good to me.

  • Resolution set to fixed
  • Status changed from assigned to closed

In [19884]:

Expose image_default_link_type, image_default_size, and image_default_align
options via XML-RPC

Props koke
Fixes #18126

Related: #20228

Note: See TracTickets for help on using tickets.