Changeset 947 for sites/trunk/translate.wordpress.org/includes/gp-plugins/wporg-help/wporg-help.php

Timestamp:

10/29/2014 12:37:52 AM (12 years ago)

Author:

coffee2code

Message:

translate.wordpress.org: change 'getting started' and 'need help?' links on projects page to point/redirect to the GlotPress part of the polyglots handbook

fixes #688.
props markoheijnen.

File:

• sites/trunk/translate.wordpress.org/includes/gp-plugins/wporg-help/wporg-help.php (modified) (5 diffs)

sites/trunk/translate.wordpress.org/includes/gp-plugins/wporg-help/wporg-help.php

@@ -2 +2 @@
 
 class GP_Help_Page_Plugin extends GP_Plugin {
-        var $id = 'wporg-help';
-        var $hide_notice = 'wporg-help-hide-notice'; 
-        
+        public $id = 'wporg-help';
+        private $hide_notice = 'wporg-help-hide-notice';
+        const handbook_link = 'https://make.wordpress.org/polyglots/handbook/tools/glotpress-translate-wordpress-org/';
+
         function __construct() {
                 parent::__construct();
@@ -12 +13 @@
                 $this->add_action( 'after_notices' );
         }
-        
+
         function init() {
                 GP::$router->add( '/getting-started', array('GP_Help_Page_Plugin_Route', 'getting_started') );
+                GP::$router->add( '/getting-started/', array('GP_Help_Page_Plugin_Route', 'getting_started') );
                 GP::$router->add( '/getting-started/hide-notice', array('GP_Help_Page_Plugin_Route', 'hide_notice') );
         }
-        
+
         function after_hello() {
                 if ( GP::$user->logged_in() || $this->is_notice_hidden() ) {
-                        echo '<em><a class="secondary" href="'.gp_url('/getting-started').'">Need help?</a></em>';
+                        echo '<em><a class="secondary" href="' . self::handbook_link . '">Need help?</a></em>';
                 }
         }
-        
+
         function is_notice_hidden() {
                 return ( gp_array_get( $_COOKIE, $this->hide_notice ) || ( GP::$user->logged_in() && GP::$user->get_meta( $this->hide_notice ) ) );
         }
-        
+
         function hide_notice() {
                 if ( GP::$user->logged_in() ) {
@@ -34 +36 @@
                 setcookie( $this->hide_notice, '1', time() + 3600*24*30, gp_url( '/' ) ); // a month
         }
-        
+
         function after_notices() {
                 if ( $this->is_notice_hidden() ) return;
@@ -41 +43 @@
                 <div class="notice" id="help-notice">
                         New to <em>translate.wordpress.org</em>?
-                        Have a look at the <a href="<?php echo esc_url( gp_url( '/getting-started' ) ); ?>">Getting Started guide.</a>
+                        Have a look at the <a href="<?php echo self::handbook_link; ?>" target="_blank">Getting Started guide.</a>
                         <a id="hide-help-notice" class="secondary" style="float: right;" href="<?php echo esc_url( $hide_url ); ?>">Hide</a>
                 </div>
@@ -53 +55 @@
 <?php
         }
-        
+
 }
 
 class GP_Help_Page_Plugin_Route extends GP_Route {
+
         function __construct() {
                 parent::__construct();
                 $this->template_path = GP::$plugins->wporg_help->template_path;
         }
-        
+
         function getting_started() {
-                GP::$plugins->wporg_help->remove_action( 'after_notices' );             
-                $this->tmpl( 'getting-started' );
+                GP::$plugins->wporg_help->remove_action( 'after_notices' );
+
+                gp_redirect( GP_Help_Page_Plugin::handbook_link, 301 );
+                exit;
         }
-        
+
         function hide_notice() {
                 GP::$plugins->wporg_help->hide_notice();
         }
+
 }