X-Git-Url: https://scripts.mit.edu/gitweb/autoinstalls/wordpress.git/blobdiff_plain/6c8f14c09105d0afa4c1574215c59b5021040e76..7f1521bf193b382565eb753043c161f4cb3fcda7:/wp-includes/wp-diff.php diff --git a/wp-includes/wp-diff.php b/wp-includes/wp-diff.php index 65dd0074..1e8fc0e9 100644 --- a/wp-includes/wp-diff.php +++ b/wp-includes/wp-diff.php @@ -28,27 +28,27 @@ class WP_Text_Diff_Renderer_Table extends Text_Diff_Renderer { /** * @see Text_Diff_Renderer::_leading_context_lines * @var int - * @access protected + * @access public * @since 2.6.0 */ - var $_leading_context_lines = 10000; + public $_leading_context_lines = 10000; /** * @see Text_Diff_Renderer::_trailing_context_lines * @var int - * @access protected + * @access public * @since 2.6.0 */ - var $_trailing_context_lines = 10000; + public $_trailing_context_lines = 10000; /** - * {@internal Missing Description}} + * Threshold for when a diff should be saved or omitted. * * @var float * @access protected * @since 2.6.0 */ - var $_diff_threshold = 0.6; + protected $_diff_threshold = 0.6; /** * Inline display helper object name. @@ -57,7 +57,18 @@ class WP_Text_Diff_Renderer_Table extends Text_Diff_Renderer { * @access protected * @since 2.6.0 */ - var $inline_diff_renderer = 'WP_Text_Diff_Renderer_inline'; + protected $inline_diff_renderer = 'WP_Text_Diff_Renderer_inline'; + + /** + * Should we show the split view or not + * + * @var string + * @access protected + * @since 3.6.0 + */ + protected $_show_split_view = true; + + protected $compat_fields = array( '_show_split_view', 'inline_diff_renderer', '_diff_threshold' ); /** * Constructor - Call parent constructor with params array. @@ -68,8 +79,10 @@ class WP_Text_Diff_Renderer_Table extends Text_Diff_Renderer { * * @param array $params */ - function __construct( $params = array() ) { + public function __construct( $params = array() ) { parent::__construct( $params ); + if ( isset( $params[ 'show_split_view' ] ) ) + $this->_show_split_view = $params[ 'show_split_view' ]; } /** @@ -78,7 +91,7 @@ class WP_Text_Diff_Renderer_Table extends Text_Diff_Renderer { * @param string $header * @return string */ - function _startBlock( $header ) { + public function _startBlock( $header ) { return ''; } @@ -88,7 +101,7 @@ class WP_Text_Diff_Renderer_Table extends Text_Diff_Renderer { * @param array $lines * @param string $prefix */ - function _lines( $lines, $prefix=' ' ) { + public function _lines( $lines, $prefix=' ' ) { } /** @@ -97,8 +110,9 @@ class WP_Text_Diff_Renderer_Table extends Text_Diff_Renderer { * @param string $line HTML-escape the value. * @return string */ - function addedLine( $line ) { - return "+{$line}"; + public function addedLine( $line ) { + return "{$line}"; + } /** @@ -107,8 +121,8 @@ class WP_Text_Diff_Renderer_Table extends Text_Diff_Renderer { * @param string $line HTML-escape the value. * @return string */ - function deletedLine( $line ) { - return "-{$line}"; + public function deletedLine( $line ) { + return "{$line}"; } /** @@ -117,8 +131,8 @@ class WP_Text_Diff_Renderer_Table extends Text_Diff_Renderer { * @param string $line HTML-escape the value. * @return string */ - function contextLine( $line ) { - return " {$line}"; + public function contextLine( $line ) { + return "{$line}"; } /** @@ -126,61 +140,98 @@ class WP_Text_Diff_Renderer_Table extends Text_Diff_Renderer { * * @return string */ - function emptyLine() { - return ' '; + public function emptyLine() { + return ' '; } /** * @ignore - * @access private + * @access public * * @param array $lines * @param bool $encode * @return string */ - function _added( $lines, $encode = true ) { + public function _added( $lines, $encode = true ) { $r = ''; foreach ($lines as $line) { - if ( $encode ) - $line = htmlspecialchars( $line ); - $r .= '' . $this->emptyLine() . $this->addedLine( $line ) . "\n"; + if ( $encode ) { + $processed_line = htmlspecialchars( $line ); + + /** + * Contextually filter a diffed line. + * + * Filters TextDiff processing of diffed line. By default, diffs are processed with + * htmlspecialchars. Use this filter to remove or change the processing. Passes a context + * indicating if the line is added, deleted or unchanged. + * + * @since 4.1.0 + * + * @param String $processed_line The processed diffed line. + * @param String $line The unprocessed diffed line. + * @param string null The line context. Values are 'added', 'deleted' or 'unchanged'. + */ + $line = apply_filters( 'process_text_diff_html', $processed_line, $line, 'added' ); + } + + if ( $this->_show_split_view ) { + $r .= '' . $this->emptyLine() . $this->emptyLine() . $this->addedLine( $line ) . "\n"; + } else { + $r .= '' . $this->addedLine( $line ) . "\n"; + } } return $r; } /** * @ignore - * @access private + * @access public * * @param array $lines * @param bool $encode * @return string */ - function _deleted( $lines, $encode = true ) { + public function _deleted( $lines, $encode = true ) { $r = ''; foreach ($lines as $line) { - if ( $encode ) - $line = htmlspecialchars( $line ); - $r .= '' . $this->deletedLine( $line ) . $this->emptyLine() . "\n"; + if ( $encode ) { + $processed_line = htmlspecialchars( $line ); + + /** This filter is documented in wp-includes/wp-diff.php */ + $line = apply_filters( 'process_text_diff_html', $processed_line, $line, 'deleted' ); + } + if ( $this->_show_split_view ) { + $r .= '' . $this->deletedLine( $line ) . $this->emptyLine() . $this->emptyLine() . "\n"; + } else { + $r .= '' . $this->deletedLine( $line ) . "\n"; + } + } return $r; } /** * @ignore - * @access private + * @access public * * @param array $lines * @param bool $encode * @return string */ - function _context( $lines, $encode = true ) { + public function _context( $lines, $encode = true ) { $r = ''; foreach ($lines as $line) { - if ( $encode ) - $line = htmlspecialchars( $line ); - $r .= '' . - $this->contextLine( $line ) . $this->contextLine( $line ) . "\n"; + if ( $encode ) { + $processed_line = htmlspecialchars( $line ); + + /** This filter is documented in wp-includes/wp-diff.php */ + $line = apply_filters( 'process_text_diff_html', $processed_line, $line, 'unchanged' ); + } + if ( $this->_show_split_view ) { + $r .= '' . $this->contextLine( $line ) . $this->emptyLine() . $this->contextLine( $line ) . "\n"; + } else { + $r .= '' . $this->contextLine( $line ) . "\n"; + } } return $r; } @@ -191,14 +242,14 @@ class WP_Text_Diff_Renderer_Table extends Text_Diff_Renderer { * (TRAC style) sometimes these lines can actually be deleted or added rows. * We do additional processing to figure that out * - * @access private + * @access public * @since 2.6.0 * * @param array $orig * @param array $final * @return string */ - function _changed( $orig, $final ) { + public function _changed( $orig, $final ) { $r = ''; // Does the aforementioned additional processing @@ -222,7 +273,7 @@ class WP_Text_Diff_Renderer_Table extends Text_Diff_Renderer { $diff = $renderer->render( $text_diff ); // If they're too different, don't include any or - if ( $diff_count = preg_match_all( '!(.*?|.*?)!', $diff, $diff_matches ) ) { + if ( preg_match_all( '!(.*?|.*?)!', $diff, $diff_matches ) ) { // length of all text between or $stripped_matches = strlen(strip_tags( join(' ', $diff_matches[0]) )); // since we count lengith of text between or (instead of picking just one), @@ -264,7 +315,11 @@ class WP_Text_Diff_Renderer_Table extends Text_Diff_Renderer { } elseif ( $final_rows[$row] < 0 ) { // Final is blank. This is really a deleted row. $r .= $this->_deleted( array($orig_line), false ); } else { // A true changed row. - $r .= '' . $this->deletedLine( $orig_line ) . $this->addedLine( $final_line ) . "\n"; + if ( $this->_show_split_view ) { + $r .= '' . $this->deletedLine( $orig_line ) . $this->emptyLine() . $this->addedLine( $final_line ) . "\n"; + } else { + $r .= '' . $this->deletedLine( $orig_line ) . "" . $this->addedLine( $final_line ) . "\n"; + } } } @@ -281,11 +336,11 @@ class WP_Text_Diff_Renderer_Table extends Text_Diff_Renderer { * * @since 2.6.0 * - * @param unknown_type $orig - * @param unknown_type $final - * @return unknown + * @param array $orig + * @param array $final + * @return array */ - function interleave_changed_lines( $orig, $final ) { + public function interleave_changed_lines( $orig, $final ) { // Contains all pairwise string comparisons. Keys are such that this need only be a one dimensional array. $matches = array(); @@ -363,48 +418,6 @@ class WP_Text_Diff_Renderer_Table extends Text_Diff_Renderer { } return array($orig_matches, $final_matches, $orig_rows, $final_rows); - -/* - // Debug - echo "\n\n\n\n\n"; - - echo "-- DEBUG Matches: Orig -> Final --"; - - foreach ( $orig_matches as $o => $f ) { - echo "\n\n\n\n\n"; - echo "ORIG: $o, FINAL: $f\n"; - var_dump($orig[$o],$final[$f]); - } - echo "\n\n\n\n\n"; - - echo "-- DEBUG Matches: Final -> Orig --"; - - foreach ( $final_matches as $f => $o ) { - echo "\n\n\n\n\n"; - echo "FINAL: $f, ORIG: $o\n"; - var_dump($final[$f],$orig[$o]); - } - echo "\n\n\n\n\n"; - - echo "-- DEBUG Rows: Orig -- Final --"; - - echo "\n\n\n\n\n"; - foreach ( $orig_rows as $row => $o ) { - if ( $o < 0 ) - $o = 'X'; - $f = $final_rows[$row]; - if ( $f < 0 ) - $f = 'X'; - echo "$o -- $f\n"; - } - echo "\n\n\n\n\n"; - - echo "-- END DEBUG --"; - - echo "\n\n\n\n\n"; - - return array($orig_matches, $final_matches, $orig_rows, $final_rows); -*/ } /** @@ -416,7 +429,7 @@ class WP_Text_Diff_Renderer_Table extends Text_Diff_Renderer { * @param string $string2 * @return int */ - function compute_string_distance( $string1, $string2 ) { + public function compute_string_distance( $string1, $string2 ) { // Vectors containing character frequency for all chars in each string $chars1 = count_chars($string1); $chars2 = count_chars($string2); @@ -428,7 +441,7 @@ class WP_Text_Diff_Renderer_Table extends Text_Diff_Renderer { if ( !$string1 ) return $difference; - // Return distance per charcter (of string1) + // Return distance per character (of string1). return $difference / strlen($string1); } @@ -440,10 +453,69 @@ class WP_Text_Diff_Renderer_Table extends Text_Diff_Renderer { * @param int $b * @return int */ - function difference( $a, $b ) { + public function difference( $a, $b ) { return abs( $a - $b ); } + /** + * Make private properties readable for backwards compatibility. + * + * @since 4.0.0 + * @access public + * + * @param string $name Property to get. + * @return mixed Property. + */ + public function __get( $name ) { + if ( in_array( $name, $this->compat_fields ) ) { + return $this->$name; + } + } + + /** + * Make private properties settable for backwards compatibility. + * + * @since 4.0.0 + * @access public + * + * @param string $name Property to check if set. + * @param mixed $value Property value. + * @return mixed Newly-set property. + */ + public function __set( $name, $value ) { + if ( in_array( $name, $this->compat_fields ) ) { + return $this->$name = $value; + } + } + + /** + * Make private properties checkable for backwards compatibility. + * + * @since 4.0.0 + * @access public + * + * @param string $name Property to check if set. + * @return bool Whether the property is set. + */ + public function __isset( $name ) { + if ( in_array( $name, $this->compat_fields ) ) { + return isset( $this->$name ); + } + } + + /** + * Make private properties un-settable for backwards compatibility. + * + * @since 4.0.0 + * @access public + * + * @param string $name Property to unset. + */ + public function __unset( $name ) { + if ( in_array( $name, $this->compat_fields ) ) { + unset( $this->$name ); + } + } } /** @@ -462,7 +534,7 @@ class WP_Text_Diff_Renderer_inline extends Text_Diff_Renderer_inline { * @param string $newlineEscape * @return string */ - function _splitOnWords($string, $newlineEscape = "\n") { + public function _splitOnWords($string, $newlineEscape = "\n") { $string = str_replace("\0", '', $string); $words = preg_split( '/([^\w])/u', $string, -1, PREG_SPLIT_DELIM_CAPTURE ); $words = str_replace( "\n", $newlineEscape, $words );