Ticket #15928: 15928.diff

src/wp-includes/post.php

@@ -4944 +4944 @@
                 $url = get_the_guid( $post->ID );
         }
 
+        // Match the current scheme.
+        $url = set_url_scheme( $url );
+
         /**
          * Filter the attachment URL.
          *

tests/phpunit/tests/post/attachments.php

@@ -278 +278 @@
                 $this->assertEquals( $attachment->post_parent, $post_id );
         }
 
+        /**
+         * @ticket 15928
+         */
+        public function test_wp_get_attachment_with_https_off() {
+                $filename = ( DIR_TESTDATA . '/images/test-image.jpg' );
+                $contents = file_get_contents( $filename );
+
+                $upload = wp_upload_bits( basename( $filename ), null, $contents );
+                $this->assertTrue( empty( $upload['error'] ) );
+
+                // Set attachment ID
+                $attachment_id = $this->_make_attachment( $upload );
+
+                $is_ssl = is_ssl();
+                $_SERVER['HTTPS'] = 'off';
+
+                // Test that wp_get_attachemt_url returns with http scheme
+                $url = wp_get_attachment_url( $attachment_id );
+                $this->assertSame( $url, set_url_scheme( $url, 'http' ) );
+
+                // Cleanup
+                $_SERVER['HTTPS'] = $is_ssl ? 'on' : 'off';
+        }
+
+        /**
+         * @ticket 15928
+         */
+        public function test_wp_get_attachment_url_with_https_on() {
+                $filename = ( DIR_TESTDATA . '/images/test-image.jpg' );
+                $contents = file_get_contents( $filename );
+
+                $upload = wp_upload_bits( basename( $filename ), null, $contents );
+                $this->assertTrue( empty( $upload['error'] ) );
+
+                // Set attachment ID
+                $attachment_id = $this->_make_attachment( $upload );
+
+                $is_ssl = is_ssl();
+                $_SERVER['HTTPS'] = 'on';
+
+                // Test that wp_get_attachemt_url returns with https scheme
+                $url = wp_get_attachment_url( $attachment_id );
+                $this->assertSame( $url, set_url_scheme( $url, 'https' ) );
+
+                // Cleanup
+                $_SERVER['HTTPS'] = $is_ssl ? 'on' : 'off';
+        }
+
 }
+