This afternoon I was helping someone profile a slow page in #drupal-support during which I found out the following:
During the execution of media_wysiwyg_filter() a file_load() is issued for each individual media tag. I spent some time rewriting portions of that functionality to first extract all tag information, then load all relevant files at once via file_load_multiple() and then replace the tags.
Out of time now, but uploading patch for testbot to take a look.
| Comment | File | Size | Author |
|---|---|---|---|
| batch-replace-media-wysiwyg-tokens.patch | 9.41 KB | legolasbo |
Comments
Comment #2
legolasboComment #3
dshields commentedThis drastically reduces the load time of the node edit page when there are many media tags included in the node!
Thanks for the patch!!
Comment #4
chris matthews commentedClosing this issue as outdated. However, if you think this issue is still important, please let us know and we will gladly re-open it for review.
sincerely,
- the Drupal Media Team
Comment #5
legolasboI'd say, take a look if this still applies, because it drastically improves performance
Comment #7
legolasboI guess this is outdated indeed