Changeset 10470

Timestamp:

01/27/2016 04:43:44 PM (9 years ago)

Author:

djpaul

Message:

Introduce base classes for new Email implementation.

• Class BP_Email represents an email; the bits you see in your email client like the subject, message, recipients. It also encapsulates custom email headers and tokens.
• Class BP_Email_Recipient represents an email recipient. It holds an email address and recipient name. If the class is instantiated with a WordPress user ID, then it also has a WP_User reference.
• Interface BP_Email_Delivery is what mail delivery services will use to implement support for emails sent by BuddyPress.
• Class BP_PHPMailer implements BP_Email_Delivery for PHPMailer.

See #6592. Props timersys, mercime, boonebgorges, hnla, DJPaul.

Location:

trunk

Files:

• src/bp-core/bp-core-classes.php (modified) (1 diff)
• src/bp-core/classes/class-bp-email-delivery.php (added)
• src/bp-core/classes/class-bp-email-recipient.php (added)
• src/bp-core/classes/class-bp-email.php (added)
• src/bp-core/classes/class-bp-phpmailer.php (added)
• tests/phpunit/includes/install.php (modified) (2 diffs)
• tests/phpunit/includes/loader.php (modified) (1 diff)
• tests/phpunit/includes/mock-mailer.php (added)
• tests/phpunit/testcases/core/class-bp-email-recipient.php (added)
• tests/phpunit/testcases/core/class-bp-email.php (added)
• tests/phpunit/testcases/core/functions.php (modified) (1 diff)

trunk/src/bp-core/bp-core-classes.php

@@ -26 +26 @@
 require dirname( __FILE__ ) . '/classes/class-bp-attachment-avatar.php';
 require dirname( __FILE__ ) . '/classes/class-bp-attachment-cover-image.php';
+require dirname( __FILE__ ) . '/classes/class-bp-email-recipient.php';
+require dirname( __FILE__ ) . '/classes/class-bp-email.php';
+require dirname( __FILE__ ) . '/classes/class-bp-email-delivery.php';
+require dirname( __FILE__ ) . '/classes/class-bp-phpmailer.php';

trunk/tests/phpunit/includes/install.php

@@ -14 +14 @@
 require_once $config_file_path;
 require_once $tests_dir_path . '/includes/functions.php';
+require_once $tests_dir_path . '/includes/mock-mailer.php';
 
 function _load_buddypress() {
@@ -49 +50 @@
 }
 
+function _bp_mock_mailer( $class ) {
+    return 'BP_UnitTest_Mailer';
+}
+tests_add_filter( 'bp_send_email_delivery_class', '_bp_mock_mailer' );
+
 // Install BuddyPress
 bp_version_updater();

trunk/tests/phpunit/includes/loader.php

@@ -8 +8 @@
 // Bootstrap BP
 require dirname( __FILE__ ) . '/../../../src/bp-loader.php';
+
+require_once( dirname( __FILE__ ) . '/mock-mailer.php' );
+function _bp_mock_mailer( $class ) {
+    return 'BP_UnitTest_Mailer';
+}
+tests_add_filter( 'bp_send_email_delivery_class', '_bp_mock_mailer' );

trunk/tests/phpunit/testcases/core/functions.php

@@ -708 +708 @@
         $this->assertTrue( empty( $not_image ) );
     }
+
+    public function test_emails_should_have_correct_link_color() {
+        $appearance = bp_email_get_appearance_settings();
+
+        $content    = '<a href="http://example.com">example</a>';
+        $link_color = 'style="color: ' . esc_attr( $appearance['highlight_color'] ) . ';';
+        $result     = bp_email_add_link_color_to_template( $content, 'template', 'add-content' );
+        $this->assertContains( $link_color, $result );
+
+        $content     = '<a href="http://example.com" style="display: block">example</a>';
+        $link_color .= 'display: block';
+        $result      = bp_email_add_link_color_to_template( $content, 'template', 'add-content' );
+        $this->assertContains( $link_color, $result );
+    }
 }