From 268da1986e106fd8763beea61bfe0fff75ee97ee Mon Sep 17 00:00:00 2001
From: Filip Komar <filip@mageia.org>
Date: Sun, 5 Jan 2014 16:01:51 +0000
Subject: some usability improvements

---
 langs/diff.php | 48 ++++++++++++++++++++++++++++++++++++------------
 1 file changed, 36 insertions(+), 12 deletions(-)

(limited to 'langs/diff.php')

diff --git a/langs/diff.php b/langs/diff.php
index 948243cbf..9e5484a2f 100644
--- a/langs/diff.php
+++ b/langs/diff.php
@@ -26,15 +26,25 @@ if (!file_exists($target_file)) {
 }
 
 if($source_file =='en/about/constitution.en.lang') {
-	$constitution = true;
+    $constitution = true;
 } else {
-	$constitution = false;
+    $constitution = false;
 }
 
 if($source_file =='en/about/license.en.lang') {
-	$license = true;
+    $license = true;
 } else {
-	$license = false;
+    $license = false;
+}
+
+if ($source_file == '../_nav/langs/en.lang') {
+    $file   = substr(_lang_file_switch($source_file, $target_lang), 14);
+    $note   = '<span style="color: red; font-weight: bold"> NOTE: THIS IS TRANSLATION OF NAVIGATION SO LOCATION FOR COMMIT IS DIFFERENT!</span>';
+    $commit = '<a href="//svnweb.mageia.org/web/www/trunk/_nav/langs/' . $file . '?view=markup">web/www/trunk/_nav/langs/' . $file . '</a> and for now to the <a href="//svnweb.mageia.org/web/nav/langs/' . $file . '?view=markup">web/nav/langs/' . $file . '</a> too';
+} else {
+    $file   = _lang_file_switch($source_file, $target_lang);
+    $note   = '';
+    $commit = '<a href="//svnweb.mageia.org/web/www/trunk/langs/' . $file . '?view=markup">web/www/trunk/langs/' . $file . '</a>';
 }
 
 $source_file = realpath($source_file);
@@ -45,7 +55,6 @@ if (false === strstr($source_file, '/langs/') ||
     die('no sorry');
 }
 
-
 $diff = _lang_diff($source_file, $target_file);
 
 $s = '<a href="report.php">&laquo; back to langs report</a>';
@@ -53,25 +62,38 @@ $s .= sprintf('<h1>Differences between %s source and %s target</h1>',
     substr($source_file, strpos($source_file, '/langs/')+7), $target_lang);
 
 if($constitution) {
-	$s .= '<h2 style="color: red;">Please translate constitution first</h2>';
-	$s .= '<p>You can find it in <a href="//svnweb.mageia.org/org/constitution/">svn</a>. ';
+    $s .= '<h2 style="color: red;">Please translate constitution first</h2>';
+    $s .= '<p>You can find it in <a href="//svnweb.mageia.org/org/constitution/">svn</a>. ';
 }
 
+$num_of_untranslated_strings = 0;
 if($license) {
-	$s .= '<h2 style="color: red;">Please translate license first</h2>';
-	$s .= sprintf('<p>You can find it in <a href="http://gitweb.mageia.org/software/drakx/tree/perl-install/share/po/">git</a> (file %s.po). ', $target_lang); // git doesn't work yet on https
+    require_once('../en/about/license/license.php');
+    $license_numbers = array_pop(read_license_from_vcs($target_lang));
+    $num_of_untranslated_strings = $license_numbers["untran"]; // number of all license sentences
+    $diff['untranslated_sentences_in_license'] = $license_numbers['untranslated_sentences']; // add untranslated license sentences
+    $s .= '<h2 style="color: red;">Please translate license into ' . $langs[$target_lang] . ' first</h2>';
+    $s .= sprintf('<p>You can find it in <a href="http://gitweb.mageia.org/software/drakx/tree/perl-install/share/po/%s.po">git</a>. ', $target_lang); // git doesn't work yet on https
 }
 
 if($constitution || $license) {
-	$s .= 'Please read <a href="https://wiki.mageia.org/en/Internationalisation_Team_%28i18n%29#Special_cases_of_web_pages">page on wiki for more information</a> about that.</p>';
+    $s .= 'Please read <a href="https://wiki.mageia.org/en/Internationalisation_Team_%28i18n%29#Special_cases_of_web_pages">page on wiki for more information</a> about that.</p>';
 }
-foreach (array('missing' => 'missing strings', 'notrans' => 'untranslated strings', 'extra' => 'unused (old) strings', 'dup_str' => 'duplicate strings') as $type => $name) {
+
+$s .= '<p>After translation:</p>
+        <ul>
+            <li>if you have a SVN account, you may commit updated file directly in <code>' . $commit . '</code>. ' . $note . '</li>
+            <li>or attach it to a mail to Mageia localization team (<code>mageia-i18n</code>, <a href="irc://irc.freenode.net/#mageia-i18n">#mageia-i18n</a> on Freenode).</li>
+        </ul>';
+
+foreach (array('missing' => 'missing strings', 'notrans' => 'untranslated strings', 'extra' => 'unused (old) strings', 'dup_str' => 'duplicate strings', 'untranslated_sentences_in_license' => 'untranslated msgid strings in ' . $num_of_untranslated_strings . ' sentences in separate <a href="http://gitweb.mageia.org/software/drakx/tree/perl-install/share/po/' . $target_lang . '.po">license file</a> (in git)') as $type => $name) {
 
     if (count($diff[$type]) > 0) {
         $s .= sprintf('<h2>%d %s:</h2>', count($diff[$type]), $name);
         $s .= '<pre>';
+        $prefix = (($type == 'untranslated_sentences_in_license') ? 'msgid: ' : ';');
         foreach ($diff[$type] as $l)
-            $s .= sprintf(";%s\n\n", htmlspecialchars($l));
+            $s .= sprintf("%s%s\n\n", $prefix, htmlspecialchars($l));
         $s .= '</pre>';
     }
 }
@@ -83,6 +105,8 @@ header('Content-Type: text/html;charset=utf-8');
 <head>
     <meta charset="utf-8">
     <meta name="robot" content="noindex,nofollow,nosnippet">
+    <?php echo sprintf('<title>Differences between %s source and %s target</title>',
+    substr($source_file, strpos($source_file, '/langs/')+7), $target_lang); ?>
     <style>
     pre { background: #eee; padding: 0.6em; }
     </style>
-- 
cgit v1.2.1