Skip to content

[5.0] Make sure the editor content is a string, not null#41705

Merged
HLeithner merged 2 commits intojoomla:5.0-devfrom
Fedik:fix-editor-null-content
Sep 12, 2023
Merged

[5.0] Make sure the editor content is a string, not null#41705
HLeithner merged 2 commits intojoomla:5.0-devfrom
Fedik:fix-editor-null-content

Conversation

@Fedik
Copy link
Member

@Fedik Fedik commented Sep 11, 2023

Summary of Changes

Make sure the editor content is a string.

Testing Instructions

Run following, somewhere in layout or in template:

$editor = Joomla\CMS\Editor\Editor::getInstance('tinymce');
echo $editor->display('text', null);

Actual result BEFORE applying this Pull Request

An error

Expected result AFTER applying this Pull Request

You get rendered editor

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:
  • No documentation changes for docs.joomla.org needed
  • Pull Request link for manual.joomla.org:
  • No documentation changes for manual.joomla.org needed

@HLeithner HLeithner merged commit 3940041 into joomla:5.0-dev Sep 12, 2023
@HLeithner
Copy link
Member

thanks

@Fedik Fedik deleted the fix-editor-null-content branch September 12, 2023 08:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments