silverstripe / silverstripe-cms

Silverstripe CMS - this is a module for Silverstripe Framework rather than a standalone app. Use https://github.com/silverstripe/silverstripe-installer/ to set this up.
http://silverstripe.org/
BSD 3-Clause "New" or "Revised" License
515 stars 333 forks source link

Insert from Files button not working in TinyMCE editor #2915

Closed elliot-sawyer closed 9 months ago

elliot-sawyer commented 9 months ago

Module version(s) affected

Silverstripe CMS 5.1

Description

When using a vanilla installation of Silverstripe 5.1, the "Insert From Files" button does not work and causes a console error.

How to reproduce

  1. Login at Silverstripe Demo (demo.silverstripe.org)
  2. Create a new Page
  3. In the Content field, use "Insert from Files" button
  4. Select any existing PDF or upload a new one (Lorem_ipsum.pdf)
  5. Enter something for Link Text and description and press Insert Files
  6. Observe console error messages

Possible Solution

No response

Additional Context

Uncaught TypeError: e.noticeAdd is not a function
    statusMessage https://demo.silverstripe.org/assets/_tinymce/tinymce-cms-f938cf0f6c.js:62`

Validations

PR

elliot-sawyer commented 9 months ago

Have checked against recipe-cms 5.0.0 as well, appears to be broken there

GuySartorelli commented 9 months ago

PR merged. This will be automatically tagged by GitHub actions

elliot-sawyer commented 8 months ago

Is the fix for this a patch release on CMS 5.1, or will we need to wait for 5.2?

GuySartorelli commented 8 months ago

The fix for this was tagged on silverstripe/asset-admin as 2.1.4