[wp-trac] [WordPress Trac] #14691: Allow commas in tag names

WordPress Trac noreply at wordpress.org
Mon Oct 7 13:46:27 UTC 2013


#14691: Allow commas in tag names
-------------------------------------------------+-------------------------
 Reporter:  mdawaffe                             |       Owner:
     Type:  enhancement                          |      Status:  new
 Priority:  normal                               |   Milestone:  Future
Component:  Taxonomy                             |  Release
 Severity:  normal                               |     Version:  3.0.1
 Keywords:  has-patch needs-testing needs-unit-  |  Resolution:
  tests                                          |
-------------------------------------------------+-------------------------

Comment (by joehoyle):

 I am not a fan of this implementation. In my opinion there should be
 better separation of concerns so the front end is responsible for passing
 an array of terms to the backend. The current implementation passes the
 terms as a CSV string (in a textarea) which seems like a total hack - we
 can pass arrays in POST so let's use that.

 This would require changing the tax meta box's javascript to store each
 term in a separate input with a `name` like `tax_input[post_tag][]` rather
 than parsing a CSV string on the backend.

 I am inclined to think including the library is more than necessary, also
 the patch is so old it doesn't apply as `post.js` has changed
 significantly since it was written.

--
Ticket URL: <http://core.trac.wordpress.org/ticket/14691#comment:11>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list