[wp-trac] [WordPress Trac] #6481: Fancy permalinks should be enabled on new sites

WordPress Trac noreply at wordpress.org
Mon Dec 29 15:54:14 UTC 2014


#6481: Fancy permalinks should be enabled on new sites
------------------------------------+------------------
 Reporter:  Denis-de-Bernardy       |       Owner:
     Type:  enhancement             |      Status:  new
 Priority:  normal                  |   Milestone:  4.2
Component:  Permalinks              |     Version:  2.7
 Severity:  minor                   |  Resolution:
 Keywords:  has-patch dev-feedback  |     Focuses:
------------------------------------+------------------

Comment (by ericlewis):

 Replying to [comment:51 valendesigns]:
 > My patch iterates on 6481.3.diff and removes the unnecessary
 `wp_install_verify_permalink_structure()` function and generally cleans
 things up a bit.

 Hi valendesigns, thanks for your work here. However, cramming more lines
 into an already quite long function increases our code's complexity.
 Creating single responsibility functions keeps our code legible and less
 complex. We should probably have developer documentation for practices
 like these, we probably should.

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


More information about the wp-trac mailing list