[wp-trac] [WordPress Trac] #11456: excerpt_more tweak
WordPress Trac
wp-trac at lists.automattic.com
Wed Dec 16 19:11:37 UTC 2009
#11456: excerpt_more tweak
--------------------------+-------------------------------------------------
Reporter: demetris | Owner:
Type: defect (bug) | Status: closed
Priority: high | Milestone: 2.9
Component: Formatting | Version: 2.9
Severity: normal | Resolution: fixed
Keywords: has-patch |
--------------------------+-------------------------------------------------
Comment(by demetris):
@hakre:
LOL. My concatenation is rather unpopular. And I got rboren into trouble
with it. :-D
From the wp-dev channel:
> (6:43:37 PM) filosofo: uh, why ' ' . '[...]' in r12414 ?
> (6:44:01 PM) filosofo: how is that better than ' [...]'?
> SNIP
> (6:52:03 PM) rboren: filosofo: Thought about changing it but then
thought it made the rather extraneous nature of the space more apparent.
> (6:52:11 PM) rboren: Would like to just get rid of it though.
> SNIP
> (7:01:24 PM) demetris: rboren: filosofo: re r12414 -- My patch split the
string into two for the reason rboren said. To make it obvious where the
space was added, and also as an indication to people hacking the code in
the future, that this space is there on purpose; it is not a typo, for
example. But I am not a coder myself, so I dont know if this is good
practice.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/11456#comment:4>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list