Duplicate enclosure attributes are created when a client resubmits a post with identical enclosure
|Reported by:||redsweater||Owned by:||josephscott|
The support in 2.6 for accepting enclosures via XMLRPC does not discriminate whether the specified enclosure is already attached to the post in question. This has the effect that if a client redundantly re-specifies the enclosure value, WordPress will redundantly add new enclosure custom field entries, each time the post is edited.
The code that accepts <enclosure> elements should only add a new custom field to the post if the enclosure specified is different from the previous enclosure(s). Arguably, for an XMLRPC context, the specified enclosure should always replace the (presumed singular) enclosure on the post.
Right now, the number of redundant enclosure attributes can spiral out of control depending on the number of times a given post is edited by a remote client that re-specifies all the known attributes of a post.
Consider for example, that when resubmitting a post, it's natural and expected to re-specifiy the <title> even if it hasn't changed. This doesn't cause multiple "title" attributes to appear on a post. The re-specification of the <enclosure> value shouldn't cause attribute bloat in posts.
Change History (12)
- Keywords needs-patch added; has-patch removed
- Milestone changed from 2.8 to Future Release
- Component changed from General to Optimization
- Priority changed from normal to low
- Severity changed from normal to minor
- Milestone changed from Future Release to 2.9
- Type changed from defect (bug) to enhancement