Changeset 4964 for sites/trunk/common/includes/wporg-sso/class-wporg-sso.php

Timestamp:

02/20/2017 11:49:01 PM (9 years ago)

Author:

ocean90

Message:

WordPress.org SSO: Remove & operator for objects, fix whitespace usage, and update DocBlocks.

File:

• sites/trunk/common/includes/wporg-sso/class-wporg-sso.php (modified) (5 diffs)

sites/trunk/common/includes/wporg-sso/class-wporg-sso.php

@@ -2 +2 @@
 if ( ! class_exists( 'WPOrg_SSO' ) ) {
         /**
-         * Single Sign-On (SSO) handling for WordPress/bbPress/GlotPress instances under *.wordpress.org.
+         * Single Sign-On (SSO) handling for WordPress/bbPress instances under *.wordpress.org.
          *
          * @author stephdau
@@ -51 +51 @@
 
                         // If we got a host by now, it's a safe wordpress.org-based one, add it to the list of allowed redirects
-                        if ( ! empty( $host ) && ! in_array( $host, $hosts ) ){
+                        if ( ! empty( $host ) && ! in_array( $host, $hosts ) ) {
                                 $hosts[] = $host;
                         }
@@ -81 +81 @@
 
                 }
-
 
                 /**
@@ -114 +113 @@
                                         $redirect_to = $redirect_to_referrer;
                                 }
-                        } else{
+                        } else {
                                 // Otherwise, attempt to guess the parent dir of where they came from and validate that.
                                 $redirect_to_source_parent = preg_replace( '/\/[^\/]+\.php\??.*$/', '/', "https://{$this->host}{$_SERVER['REQUEST_URI']}" );
@@ -153 +152 @@
                  * Validates if target URL is within our bounds, then redirects to it if so, or to WP.org homepage (returns if headers already sent).
                  *
-                 * @param string $to Destination URL
-                 * @param number $status HTTP redirect status, defaults to 302
+                 * @note: using our own over wp_safe_redirect(), etc, because not all targeted platforms (WP/BB/GP/etc) implement an equivalent, we run early, etc.
                  *
-                 * @note: using our own over wp_safe_redirect(), etc, because not all targeted platforms (WP/BB/GP/etc) implement an equivalent, we run early, etc.
+                 * @param string $to     Destination URL
+                 * @param int    $status HTTP redirect status, defaults to 302
                  */
                 protected function _safe_redirect( $to, $status = 302 ) {