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

Reto Hugi plog at hugi.to
Thu Aug 10 09:06:30 GMT 2006


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