[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