Changeset 639

Timestamp:

05/23/2014 05:58:19 AM (12 years ago)

Author:

coffee2code

Message:

Code Reference: Add link to core.trac to view code source for item. Fixes #447. props DrewAPicture

• Change 'Source:' label to 'Source file:'
• Add 'View source...' link that points to relevant line in core.trac when appropriate

Location:

sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-developer

Files:

• content-reference.php (modified) (1 diff)
• content-wp-parser-hook.php (modified) (2 diffs)
• inc/template-tags.php (modified) (3 diffs)

sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-developer/content-reference.php

@@ -26 +26 @@
         <?php
         $source_file = get_source_file();
-        if ( ! empty( $source_file ) ) : ?>
-        <section class="source">
-                <p><strong><?php _e( 'Source:', 'wporg' ); ?></strong> <a href="<?php echo get_source_file_link( $source_file ); ?>"><?php echo esc_html( $source_file ); ?></a></p>
-        </section>
+        if ( ! empty( $source_file ) ) :
+                ?>
+                <section class="source">
+                        <p><strong><?php _e( 'Source file:', 'wporg' ); ?> </strong><a href="<?php echo get_source_file_archive_link( $source_file ); ?>"><?php echo esc_html( $source_file ); ?></a></p>
+                        <p><a href="<?php echo get_source_file_link(); ?>"><?php _e( 'View source&hellip;', 'wporg' ); ?></a></p>
+                </section>
         <?php endif; ?>
 

sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-developer/content-wp-parser-hook.php

@@ -4 +4 @@
 
         <h1><a href="<?php the_permalink() ?>"><?php echo get_signature(); ?></a></h1>
-
         <section class="description">
                 <?php the_excerpt(); ?>
@@ -25 +24 @@
         <?php
         $source_file = get_source_file();
-        if ( ! empty( $source_file ) ) : ?>
-        <section class="source">
-                <p><strong><?php _e( 'Source:', 'wporg' ); ?></strong> <a href="<?php echo get_source_file_link( $source_file ); ?>"><?php echo esc_html( $source_file ); ?></a></p>
-        </section>
+        if ( ! empty( $source_file ) ) :
+                ?>
+                <section class="source">
+                        <p><strong><?php _e( 'Source file:', 'wporg' ); ?> </strong><a href="<?php echo get_source_file_archive_link( $source_file ); ?>"><?php echo esc_html( $source_file ); ?></a></p>
+                        <p><a href="<?php echo get_source_file_link(); ?>"><?php _e( 'View source&hellip;', 'wporg' ); ?></a></p>
+                </section>
         <?php endif; ?>
 

sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-developer/inc/template-tags.php

@@ -558 +558 @@
 
         /**
-         * Retrieve URL to source file archive
+         * Retrieve URL to source file archive.
          *
          * @param string $name
@@ -564 +564 @@
          * @return string
          */
-        function get_source_file_link( $name = null ) {
+        function get_source_file_archive_link( $name = null ) {
 
                 $source_file_object = get_term_by( 'name', empty( $name ) ? get_source_file() : $name, 'wp-parser-source-file' );
@@ -583 +583 @@
 
                 return empty( $source_file_object ) ? '' : esc_html( $source_file_object[0] );
+        }
+
+        /**
+         * Retrieve the URL to the actual source file and line.
+         *
+         * @param null $post_id     Post ID.
+         * @param bool $line_number Whether to append the line number to the URL.
+         *                          Default true.
+         * @return string Source file URL with or without line number.
+         */
+        function get_source_file_link( $post_id = null, $line_number = true ) {
+
+                $post_id = empty( $post_id ) ? get_the_ID() : $post_id;
+                $url     = '';
+
+                // Source file.
+                $source_file = get_source_file( $post_id );
+                if ( ! empty( $source_file ) ) {
+                        $url = 'https://core.trac.wordpress.org/browser/tags/' . get_current_version() . '/src/' . $source_file;
+                        // Line number.
+                        if ( $line_number = get_post_meta( get_the_ID(), '_wp-parser_line_num', true ) ) {
+                                $url .= "#L{$line_number}";
+                        }
+                }
+
+                return esc_url( $url );
         }