Changeset 639

Timestamp:

05/23/2014 05:58:19 AM (10 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 );
     }