Work at SourceForge, help us to make it a better place! We have an immediate need for a Support Technician in our San Francisco or Denver office.

Close

Regarding Bug #3310209

Developers
croaker___
2011-11-11
2013-05-15
  • croaker___
    croaker___
    2011-11-11

    Balliano

    https://sourceforge.net/tracker/?func=detail&atid=620566&aid=3310209&group_id=98294

    It seems churlish to just close the bug and blame the problem on ckeditor - even though it is their problem.

    By the simple replacement of a couple of lines of code in "./themes/default/widgets/rich_textarea/rich_textarea.php" we can error trap ckeditor's bug and feed a sensible result back to P4A.

    Of course, as you say in the bug, P4A can't know if a theme exists or not - but it doesn't need to. My small change will feed back a sensible result to P4A and stop it crashing.

    If we want P4A to work properly and help the developer build applications quickly, surely we should do all we can to make life as easy as possible?

    I assume that because rich_textarea.php is part of ckeditor, you don't want to interfere with it to ease future upgrades? You still need to edit line 89 in any case ("filebrowserBrowseUrl:").

    Regs
    Martin

     
  • rich_textarea.php is not a part of ckeditor but that's not the point, report the bug to the right place, which is ckeditor