Closed (fixed)
Project:
RC4 email
Version:
7.x-1.x-dev
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
23 Jan 2015 at 13:02 UTC
Updated:
28 Dec 2015 at 09:04 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
mpp commentedWe prefer to have the dependency to base64.js added to the module.
Also see https://www.drupal.org/node/1181070
Comment #2
mpp commentedComment #3
Anonymous (not verified) commentedDoes IE9 not work with
<!--[if IE]>tag? I am not sure if we want to include a whole Javascript library just for IE... and if we are going to do it, it should be done through the library API.Comment #4
mpp commentedThe problem here is that rc4_email_preprocess_html() is not always excecuted (e.g. for the imce popup). This in turn results in a JavaScript error on IE browsers as it doesn't find the base64 dependency.
So I would either do a check on Drupal.settings.rc4_email if it is set before performing any actions:
Or implement hook_libraries_info to provide base64 as a library as some projects may not want to have a dependency to an external resource.
Comment #5
mpp commentedNew patch with test if Drupal.settings.rc4_email is set.
Comment #6
damienmckennaThis uses the more correct
typeOf varname !== 'undefined'structure.Comment #7
damienmckennaWhoops, I misspelled "typeof".
Comment #8
Anonymous (not verified) commentedI haven't got around to test it yet but may I ask this: is that all it needed was to patch the js with typeof rc4_email !== undefined? If so please rtbc this and I'll commit the patch straight away. Thanks.
Comment #9
mpp commentedIndeed
Comment #11
Anonymous (not verified) commentedPatch committed, thanks @mmp and @DamienMcKenna!