Changes between Initial Version and Version 1 of Ticket #21760, comment 44
- Timestamp:
- 07/05/2014 08:03:46 AM (7 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #21760, comment 44
initial v1 3 3 A couple thoughts on the patch: 4 4 5 * Line 980 of **src/wp-includes/taxonomy.php** should use `$term->term_id` as the key, not the `$term` object.5 * ~~Line 980 of **src/wp-includes/taxonomy.php** should use `$term->term_id` as the key, not the `$term` object.~~ 6 6 * `$bucket` is not a term used in core (other than fleetingly in a few comments). I think for clarity and consistency, we should stick with the `$group` nomenclature. 7 7 * The keys for the term names will prove problematic. Whitespace is not allowed in Memcached keys. Additionally, these keys can only be a grand total of 250 characters in Memcached (and I'm assuming there are other limits in other object caches). These should be hashed or sanitized in someway before being used as keys.