[pLog-svn] r3850 - plog/trunk/class/dao

Oscar Renalias oscar at renalias.net
Thu Aug 10 09:10:59 GMT 2006


I can't sincerely remember why I gave that answer a while back... it
definitely isn't right and I think our current fix is better.

On 8/10/06, Reto Hugi <plog at hugi.to> wrote:
> On 10.08.2006 08:23, Oscar Renalias wrote:
> > Wasn't the issue about the same problem?
> >
> > I've only modified the default parameter of getText(), so that it now
> > is '' (empty) instead of '<br/>'. I suppose that if people are using
> > the normal non-html editor and forgot to surround their text with
> > '<p>...</p>' then it's their problem, not ours.
> >
>
> sure, no worries. But it's a small compliance problem. if somebody
> designed his template with a workaround it will brake again. (maybe)
>
> "I think it's too late to change this anyway... We could probably extend
> Article::getText() to include an extra parameter in the method so that
> users can choose the code that is used to replace "[@more@]". It should
> obviously default to "<br>" for compatibilty reasons, but it would allow
> users who don't like this behaviour to set it to somethinge else."
>
> Just to point this out... honestly I like it better the way it is now,
> we should leave it with the latest correction.
>


More information about the pLog-svn mailing list