![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
Opened 12 years ago
Closed 12 years ago
#27821 closed defect (bug) (fixed)
wpviews: Empty video/audio playlist is not visible in TinyMCE and throws javascript errors when selected
| Reported by: | gcorne | Owned by: | nacin |
|---|---|---|---|
| Priority: | normal | Milestone: | 3.9 |
| Component: | Editor | Version: | 3.9 |
| Severity: | normal | Keywords: | has-patch commit |
| Cc: | Focuses: | javascript |
Description
If no valid attachments are found, when a video or audio playlist is rendered as a wpview, the empty div is not visible in TinyMCE and throws javascript errors when selected.
Uncaught TypeError: Cannot read property 'set' of undefined wp-playlist.js?ver=3.9-RC2-src:81
We should handle this in the same way that we handle the gallery wpview.
Attachments (2)
Change History (8)
#2
@
12 years ago
- Keywords commit added
The changes gcorne makes in 27821-01.patch are good for protecting errors in the wild. Our features were folded in out of sync, so this gives playlists some protective parity with galleries.
This ticket was mentioned in IRC in #wordpress-dev by nacin. View the logs.
12 years ago
This ticket was mentioned in IRC in #wordpress-dev by azaozz. View the logs.
12 years ago
Note:
See TracTickets
for help on using tickets.
27821-01.patch renders an error when no tracks are found that when selected shows the toolbar so that the user can add tracks or delete the playlist.