Changes between Initial Version and Version 1 of Ticket #7009, comment 3
- Timestamp:
- 04/11/2016 07:43:05 PM (9 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #7009, comment 3
initial v1 6 6 after doing more deep debug. i found the victim. 7 7 8 multisite code is fine it has nothing wrong in it which is.8 multisite code is fine it has nothing wrong in it, which is. 9 9 {{{#!php 10 10 <?php … … 14 14 }}} 15 15 16 So what it doing it setting the sub site subdomain to the host. and mail uses host for right hand part of message id header info. 16 So what it doing it setting the sub site subdomain to the host. and phpmailer uses it for message id header info right hand part. 17 eg. message-id@host 17 18 18 and what we are doing in buddypress for mail functionality is.19 and what we are doing in BuddyPress for mail functionality is. 19 20 20 21 {{{#!php … … 30 31 31 32 So from above code. 33 32 34 return get_current_site()->domain; // From fix_phpmailer_messageid() <-- this line is correct which is taken from multisite setup. 33 35 34 But this line is wrong.36 But problem is with this line. 35 37 return preg_replace( '#^https?://#i', '', bp_get_option( 'home' ) ); 36 38 37 which is returning sub site url with sub paths instead of host.39 which is returning sub site url with sub paths (eg. domain.com/subsite1) instead of host (eg, domain.com). 38 40 as host name cannot contain the subsite directory and http://. and in most cases its subsite. 39 41 … … 48 50 subsite1.domain.com 49 51 50 So in BuddyPress that function code should be changed to something..52 So in BuddyPress mail api that function code should be changed to something.. 51 53 52 54 {{{#!php