Make WordPress Core

Opened 18 years ago

Closed 18 years ago

#7483 closed defect (bug) (fixed)

redirect_canonical not send type_of_url to user_trailingslashit

Reported by: xorax Owned by: markjaquith
Priority: high Milestone: 2.6.1
Component: General Version: 2.7
Severity: minor Keywords: has-patch canonical url redirect
Cc: Focuses:

Description

the function redirect_canonical not send correct type_of_url to user_trailingslashit when path is not home.

an accolade missing and it's in accordance with the indentation.

canonical.php :
***************
*** 165,175 ****
  		} else {
  			foreach ( array('single', 'category', 'page', 'day', 'month', 'year') as $type ) {
  				$func = 'is_' . $type;
! 				if ( call_user_func($func) )
  					$user_ts_type = $type;
  					break;
  				}
  			}
  		$redirect['path'] = user_trailingslashit($redirect['path'], $user_ts_type);
  	} elseif ( is_home() ) {
  		$redirect['path'] = trailingslashit($redirect['path']);
--- 165,176 ----
  		} else {
  			foreach ( array('single', 'category', 'page', 'day', 'month', 'year') as $type ) {
  				$func = 'is_' . $type;
! 				if ( call_user_func($func) ) {
  					$user_ts_type = $type;
  					break;
  				}
  			}
+ 		}
  		$redirect['path'] = user_trailingslashit($redirect['path'], $user_ts_type);
  	} elseif ( is_home() ) {
  		$redirect['path'] = trailingslashit($redirect['path']);

Attachments (1)

7483.diff​ (641 bytes ) - added by DD32 18 years ago.
Reporters code in .diff format.

Download all attachments as: .zip

Change History (6)

@DD32
18 years ago

Reporters code in .diff format.

#1 @DD32
18 years ago

  • Keywords has-patch added
  • Version → 2.7

#2 @ryan
18 years ago

  • Owner changed from anonymous to markjaquith

#3 @markjaquith
18 years ago

  • Resolution → fixed
  • Status new → closed

(In [8593]) Fix braces so user_trailingslashit gets the correct information passed in canonical redirect code. props xorax. fixes #7483 for trunk

#4 @markjaquith
18 years ago

  • Milestone 2.7 → 2.6.1
  • Resolution fixed
  • Status closed → reopened

Reopening for 2.6.1

#5 @markjaquith
18 years ago

  • Resolution → fixed
  • Status reopened → closed

(In [8594]) Fix braces so user_trailingslashit gets the correct information passed in canonical redirect code. props xorax. fixes #7483 for 2.6.1

Note: See TracTickets for help on using tickets.