Jump to:

1042 Posts in 379 Topics by 372 members

Upgrading SilverStripe

SilverStripe Forums » Upgrading SilverStripe » Upgrading to 2.4.1 results in 
 in HTML code

Ask questions about upgrading SilverStripe to the latest version.

Moderators: martimiz, Howard, Sean, Ryan M., biapar, Willr, Ingo, swaiba, simon_w

Page: 1 2
Go to End
Author Topic: 2224 Views
  • Ingo
    Avatar
    Forum Moderator
    787 Posts

    Re: Upgrading to 2.4.1 results in 
 in HTML code Link to this post

    12 October 2010 at 7:55am

    Thanks for tracking this down Marcus - it seems to be fairly isolated to the HTMLValue class, so at least we don't have to start debugging TinyMCE. TinyMCE has a remove_linebreaks option by the way, but we shouldn't make assumptions about the editing frontend from within HTMLValue. The DOMDocument API doesn't seem to have any options to control this behaviour ("preserveWhiteSpace" doesn't seem to have an effect on this).

    I've committed your fix in http://open.silverstripe.org/changeset/111878, including a basic unit test to validate the behaviour.

  • Sean
    Avatar
    Forum Moderator
    889 Posts

    Re: Upgrading to 2.4.1 results in 
 in HTML code Link to this post

    13 October 2010 at 12:43am

    Subsequent fix applied here which is more portable: http://open.silverstripe.org/changeset/111949/modules
    Also added a brief comment describing this change, and why it's necessary in HTMLValue.

    Previously this broke on Windows (ironically), now this works on UNIX based operating systems as well as Windows through passing unit tests.

    Cheers,
    Sean

    2224 Views
Page: 1 2
Go to Top

Want to know more about the company that brought you SilverStripe? Then check out SilverStripe.com

Comments on this website? Please give feedback.