The current javascript tries to replace every instances of "[[*]]". The current tag has a type attribute that can be used to only apply this where it is needed.
The way media module handle attributes in tags is really clean and could be a best practice. That's why I followed the same principle and created a custom filter for [[{"type":"node", "nid":"5"}]]. This is where I faced this issue.
Maybe this approach could be documented in the "examples" modules after this is fixed.
Patch following
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | wysiwyg_media_replace-1314266-1.patch | 523 bytes | idflood |
Comments
Comment #1
idflood commentedComment #2
dave reidComment #3
dave reidShould this also be fixed in the PHP versions in media.filter.inc?
Comment #4
idflood commented@Dave Reid: Oh you are right. There are a couple of regex that should probably be corrected there. This didn't caused issues on the specific setup where this bug has been found but it will be cleaner if they are fixed. The problem is that I don't like regex, so I'm not sure how to properly fix them.
Comment #5
dave reidMarked #1742856: Wysiwyg inserted media showing [Bad Link: Plugin Not Found] due to Freelinking module and #1828580: media filter removes double-bracketed expressions not intended as media module file refs as duplicates of this issue. Adding to alpha blockers tag.
Comment #6
aaron commentedApparently there has been a lot of work in the modules since this issue was reported. The patch no longer applies.
Comment #7
mengi commentedEDIT:
Please ignore, sorry.
Comment #9
dave reidI cannot find where this code lives anymore. I'm going to just close it out.