Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Ater sending mail I have a warnings (see screenshot).
I use theme based on "Pixture Reloaded".
And 7.x-2.33 version of "Mail System" module.
I found this bug into template_preprocess_mimemail_message() function.
It because this function does not correct $css_files array for drupal_get_css();
Comment | File | Size | Author |
---|---|---|---|
#7 | mimemail-fix_for_warnings-1504782.patch | 1.35 KB | rjkuyvenhoven |
#1 | 0001-Warnings-after-sending-the-letter.patch | 1.11 KB | mrded |
Screen Shot 2012-03-28 at 00.23.14.PNG | 30 KB | mrded |
Comments
Comment #1
mrded CreditAttribution: mrded commentedPlease check my patch.
Comment #2
sgabe CreditAttribution: sgabe commentedThis patch seems to wipe out a feature to get rid of an error message, but it does not seem to be a fix for the issue. Did you test it with a Fusion based theme? Are these lines unnecessary on Drupal 7?
Comment #3
mrded CreditAttribution: mrded commentedNo I did't test it on on Fusion theme. I think it's unnecessary code because it have wrong structure.
Comment #4
sgabe CreditAttribution: sgabe commentedWrong structure does not mean that it is unnecessary, it just means it is wrong.
Comment #5
mrded CreditAttribution: mrded commentedYes, definitely. Sorry :)
Comment #6
sgabe CreditAttribution: sgabe commented#1526754: Multiple Errors after subscription is marked as a duplicate of this.
Comment #7
rjkuyvenhoven CreditAttribution: rjkuyvenhoven commentedthis patch modifies the code to use drupal_add_css() to load local.css file if it exists.
Comment #8
sgabe CreditAttribution: sgabe commentedThanks rjkuyvenhoven, your patch is committed!
Comment #10
sourcesoft CreditAttribution: sourcesoft commentedJust want to add text errors to make it easier goggling since I had a day debugging this :( I got this using zen sub-theme.
Beside the patch working alright, also got fixed after unchecking ' Include site style sheets' setting
Notice: Undefined index: group in drupal_sort_css_js() (line 3025 of common.inc).
Notice: Undefined index: group in drupal_sort_css_js() (line 3025 of common.inc).
Notice: Undefined index: group in drupal_sort_css_js() (line 3028 of common.inc).
Notice: Undefined index: group in drupal_sort_css_js() (line 3025 of common.inc).
Warning: uasort() [function.uasort]: Array was modified by the user comparison function in drupal_get_css() (line 2970 of /common.inc).
Notice: Undefined index: type in drupal_get_css() (line 2975 of /home/common.inc).
Warning: ksort() expects parameter 1 to be array, null given in drupal_group_css() (line 3102 of common.inc).
Notice: Undefined index: type in drupal_group_css() (line 3114 of common.inc).
Notice: Undefined variable: group_keys in drupal_group_css() (line 3134 of common.inc).
Notice: Undefined index: type in drupal_aggregate_css() (line 3175 of common.inc).
Notice: Undefined index: type in drupal_pre_render_styles() (line 3295 of common.inc).