Ticket #7483 (closed defect (bug): fixed)

Opened 4 years ago

Last modified 4 years ago

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:

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

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

Change History

DD324 years ago

Reporters code in .diff format.

comment:1   DD324 years ago

  • Keywords has-patch added
  • Version set to 2.7

comment:2   ryan4 years ago

  • Owner changed from anonymous to markjaquith
  • Status changed from new to closed
  • Resolution set to fixed

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

  • Status changed from closed to reopened
  • Resolution fixed deleted
  • Milestone changed from 2.7 to 2.6.1

Reopening for 2.6.1

  • Status changed from reopened to closed
  • Resolution set to fixed

(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.