Changeset 2554

Timestamp:

02/23/2016 01:40:44 AM (9 years ago)

Author:

dd32

Message:

Plugins Directory: Create the tab/section links correctly (no double slashes, no multiple endpoints in one url) and add support for the Screenshots & Other Notes tabs/sections.
See #1584

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/class-wporg-plugin-directory-template.php (modified) (4 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/class-wporg-plugin-directory-template.php

@@ -58 +58 @@
         $raw_sections = array_unique( array_merge( $raw_sections, array(
             'description',
+            'screenshots',
             'stats',
             'support',
@@ -64 +65 @@
         ) ) );
 
-        $sections = array();
-        $title    = '';
-        $url      = get_permalink();
+        $sections  = array();
+        $title     = '';
+        $permalink = get_permalink();
 
         foreach ( $raw_sections as $section_slug ) {
@@ -73 +74 @@
                 case 'description':
                     $title = _x( 'Description', 'plugin tab title', 'wporg-plugins' );
+                    $url   = $permalink;
                     break;
 
                 case 'installation':
                     $title = _x( 'Installation', 'plugin tab title', 'wporg-plugins' );
-                    $url   = trailingslashit( $url ) . '/' . $section_slug . '/';
+                    $url   = trailingslashit( $permalink ) . $section_slug . '/';
                     break;
 
                 case 'faq':
                     $title = _x( 'FAQ', 'plugin tab title', 'wporg-plugins' );
-                    $url   = trailingslashit( $url ) . '/' . $section_slug . '/';
+                    $url   = trailingslashit( $permalink ) . $section_slug . '/';
                     break;
 
                 case 'screenshots':
                     $title = _x( 'Screenshots', 'plugin tab title', 'wporg-plugins' );
-                    $url   = trailingslashit( $url ) . '/' . $section_slug . '/';
+                    $url   = trailingslashit( $permalink ) . $section_slug . '/';
                     break;
 
                 case 'changelog':
                     $title = _x( 'Changelog', 'plugin tab title', 'wporg-plugins' );
-                    $url   = trailingslashit( $url ) . '/' . $section_slug . '/';
+                    $url   = trailingslashit( $permalink ) . $section_slug . '/';
                     break;
 
                 case 'stats':
                     $title = _x( 'Stats', 'plugin tab title', 'wporg-plugins' );
-                    $url   = trailingslashit( $url ) . '/' . $section_slug . '/';
+                    $url   = trailingslashit( $permalink ) . $section_slug . '/';
                     break;
 
@@ -112 +114 @@
                 case 'developers':
                     $title = _x( 'Developers', 'plugin tab title', 'wporg-plugins' );
-                    $url   = trailingslashit( $url ) . '/' . $section_slug . '/';
+                    $url   = trailingslashit( $permalink ) . '/' . $section_slug . '/';
                     break;
+
+                case 'other_notes':
+                    $title = _x( 'Other Notes', 'plugin tab title', 'wporg-plugins' );
+                    $url   = trailingslashit( $permalink ) . '/' . $section_slug . '/';
+                    break;
+
+                default:
+                    continue;
             }