Changeset 947

Timestamp:

10/29/2014 12:37:52 AM (10 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();
     }
+
 }