[wp-trac] [WordPress Trac] #30995: Pass meta_id when retrieving multiple metas

[WordPress Trac]

#30995: Pass meta_id when retrieving multiple metas
------------------------------------------+-----------------------------
 Reporter:  GregLone                      |       Owner:
     Type:  enhancement                   |      Status:  new
 Priority:  normal                        |   Milestone:  Future Release
Component:  Options, Meta APIs            |     Version:  2.9
 Severity:  normal                        |  Resolution:
 Keywords:  needs-patch needs-unit-tests  |     Focuses:
------------------------------------------+-----------------------------
Changes (by boonebgorges):

 * keywords:   => needs-patch needs-unit-tests
 * milestone:  Awaiting Review => Future Release


Comment:

 Replying to [comment:2 GregLone]:
 > In the same time, all the `*_metadata_by_mid()` would benefit of the
 cache.

 IMO, this makes the proposal worth pursuing. It'd be nice to see improved
 unit tests around cache invalidation for `update_metadata()`,
 `update_metadata_by_id()`, and their `delete_` equivalents as part of this
 improvement.

 I'm somewhat concerned about backward compatibility. In the same way that
 core sometimes references `$metas[0]`, it's possible that there are
 plugins doing the same thing. Does anyone have a good idea of how to gauge
 this impact? My google-fu is failing me :)

--
Ticket URL: <https://core.trac.wordpress.org/ticket/30995#comment:3>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform