• Resolved boyizmen

    (@boyizmen)


    It may got a wrong logical judgment in file class-wp-statistics.php,
    here’s the source code:

    
    	public function get_referrer_link( $referrer, $length = -1 ) {
    		$html_referrer = $this->html_sanitize_referrer($referrer);
    
    		if ( $length > 0 && strlen($referrer) > $length ) {
    			$html_referrer_limited = $this->html_sanitize_referrer($referrer, $length);
    			$eplises               = '[...]';
    		} else {
    			$html_referrer_limited = $html_referrer;
    			$eplises               = '';
    		}
    
    		if ( substr($html_referrer, 0, 7) !== 'http://' or substr($html_referrer, 0, 8) !== 'https://' ) {
    			// non relative address
    			$html_nr_referrer = 'http://' . $html_referrer;
    		} else {
    			$html_nr_referrer = $html_referrer;
    		}
    
    		return "<a href='{$html_nr_referrer}'><div class='dashicons dashicons-admin-links'></div>{$html_referrer_limited}{$eplises}</a>";
    	}
    

    you may use ‘and’ in if ( substr($html_referrer, 0, 7) !== 'http://' or substr($html_referrer, 0, 8) !== 'https://' ) not or
    And I suggest you use ‘//’ not ‘http://&#8217; to join the url.

Viewing 3 replies - 1 through 3 (of 3 total)
Viewing 3 replies - 1 through 3 (of 3 total)

The topic ‘Bug Report v12.3’ is closed to new replies.