bug #10192
closedExternal Links do not work in E+M
60%
Description
NaK:
die Externen Such-Links aus E+M funktionieren teilweise nicht, hinter dem Taxon-Namen bleibt immer ein Anführungszeichen und eine Klammer stehen.
Dianthus barbatus')
Related issues
Updated by Andreas Müller over 1 year ago
- Related to task #9689: External Links Module: Tests implemented added
Updated by Andreas Müller over 1 year ago
- Related to task #9659: External Links Module reimplemented for flexible configuration added
Updated by Katja Luther over 1 year ago
- Status changed from New to In Progress
The problem comes from the call of "drupal_strip_dangerous_protocols("JavaScript:popupExternalLinks('url'))" inside l(...), this call results in removing everything in front of the colon, but the "')" still at the end of the link. This additional security issue came with the drupal update (https://www.drupal.org/node/3307791).
I could remove "JavaScript:popupExternalLinks..." because it is removed anyway by the function, mentioned above. Otherwise we could add javascript to the allowed protocols or find another way to create the link without calling l().
Updated by Katja Luther over 1 year ago
- Status changed from In Progress to Resolved
- Assignee changed from Katja Luther to Andreas Müller
I removed the javascript call and add the attribute to open the link in a new tab. Please review.
Updated by Andreas Müller over 1 year ago
- Status changed from Resolved to Feedback
- Assignee changed from Andreas Müller to Katja Luther
- % Done changed from 0 to 60
How can I test this? It is not yet pushed to production, or? And on test the updates are not yet installed as far as I remember. Is there an example on integration?
Updated by Katja Luther over 1 year ago
Andreas Müller wrote in #note-5:
How can I test this? It is not yet pushed to production, or? And on test the updates are not yet installed as far as I remember. Is there an example on integration?
For example here:
Updated by Katja Luther over 1 year ago
- Assignee changed from Katja Luther to Andreas Müller
Updated by Andreas Müller over 1 year ago
- Assignee changed from Andreas Müller to Katja Luther
It works now, but it changed from using popups to opening a new tab which is maybe not wanted (there might be a reason why we were using popups). We should ask users if this is accepted behavior before closing this ticket.
Updated by Katja Luther over 1 year ago
- Status changed from Feedback to Closed
The answers of the users were all that the new solution is ok or even better, so we can close this ticket.