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.
Please change the DEFINE to
define('MAIL_LINE_ENDINGS', isset($_SERVER['WINDIR']) ||
(array_key_exists('SERVER_SOFTWARE',$_SERVER) &&
strpos($_SERVER['SERVER_SOFTWARE'], 'Win32')) !== FALSE ? "\r\n" : "\n");
strpos returns correct but as the index does not exist it give error message.
Comment | File | Size | Author |
---|---|---|---|
#6 | mail.inc_.patch | 578 bytes | adixon |
Comments
Comment #1
arne_hortell CreditAttribution: arne_hortell commentedThis bug is still there.
Comment #2
ziobudda CreditAttribution: ziobudda commentedThis bug is still here and it creates a FATAL ERROR if I install nodejs integration module watchdog submodule.
Comment #3
arne_hortell CreditAttribution: arne_hortell commentedThanks all you people keeping drupal.org working!!!
Comment #4
arne_hortell CreditAttribution: arne_hortell commentedStill same issue, my fix works though...
Comment #5
arne_hortell CreditAttribution: arne_hortell commentedStill same error, see my fix above.
Comment #6
adixon CreditAttribution: adixon commentedYes, I have this also. A version of the arne's patch attached.
I'm curious why this hasn't had more attention. My guess is that it only shows up when a cron task tries to send mail (and I'm seeing it when it's being triggered via a CiviCRM cron).
In any case, seems straightforward, though it'd be nice if all this 'windows-specific overrides' ended up in it's own file somewhere.
Comment #7
arne_hortell CreditAttribution: arne_hortell commentedStill not fixed in core, patch works
Comment #8
Manuel Garcia CreditAttribution: Manuel Garcia as a volunteer and at Appnovation for Pfizer, Inc. commentedThis was always meant for Drupal core, not mailsystem module.
Closing as duplicate because this was already fixed on #2356055: Notice in includes/mail.inc define