remove fragments of old localization and adjust remaining inline js strings, fixes #992

Closes #992

app/controllers/localizations.php

-<?php
-# Lifter010: TODO
-/*
- * Copyright (C) 2010 - Marcus Lunzenauer <mlunzena@uos.de>
- *
- * This program is free software; you can redistribute it and/or
- * modify it under the terms of the GNU General Public License as
- * published by the Free Software Foundation; either version 2 of
- * the License, or (at your option) any later version.
- */
-
-
-/**
- * TODO
- */
-class LocalizationsController extends Trails_Controller {
-
-    /**
-     * Callback function being called before an action is executed. If this
-     * function does not return FALSE, the action will be called, otherwise
-     * an error will be generated and processing will be aborted. If this function
-     * already #rendered or #redirected, further processing of the action is
-     * withheld.
-     *
-     * @param string  Name of the action to perform.
-     * @param array   An array of arguments to the action.
-     *
-     * @return bool
-     */
-    public function before_filter(&$action, &$args)
-    {
-        require_once 'lib/language.inc.php';
-
-        // substitute dashes with underscores, remove things like .UTF-8
-        $args = (array) strtr(current(explode('.', $action)), "-", "_");
-
-        // invalid language?
-        if (!isset($GLOBALS['INSTALLED_LANGUAGES'][$action])) {
-            $action = "not_acceptable";
-        } else {
-            $action = "show";
-        }
-    }
-
-
-    function show_action($language = NULL)
-    {
-        $this->set_content_type('application/javascript; charset=UTF-8');
-
-        $modified = filemtime(dirname(__FILE__) . '/../views/localizations/show.php');
-        $this->response->add_header('Last-Modified', date("r", $modified));
-
-        if (isset($_SERVER['HTTP_IF_MODIFIED_SINCE'])) {
-            if (strtotime($_SERVER['HTTP_IF_MODIFIED_SINCE']) === $modified) {
-                $this->set_status(304, "Not modified.");
-                $this->render_nothing();
-                return;
-            }
-        }
-
-        $this->language = $language;
-        setLocaleEnv($language, "studip");
-
-        // make this instance available to the view to use
-        // the helper methods
-        $this->plugin = $this;
-    }
-
-
-    function not_acceptable_action($language = NULL)
-    {
-        $this->set_status(406);
-        $this->set_content_type('application/json; charset=UTF-8');
-        $languages = array_keys($GLOBALS['INSTALLED_LANGUAGES']);
-        $this->render_text(json_encode($languages));
-    }
-}

app/controllers/messages.php

@@ -936,8 +936,10 @@ class MessagesController extends AuthenticatedController {
             '#',
             Icon::create('trash'),
             [
-                'onclick' => "return STUDIP.Dialog.confirm('Wirklich %s Nachricht(en) löschen?'.toLocaleString().replace('%s',
-        jQuery('#bulk tbody :checked').length), function() { jQuery('#bulk').submit(); }) "
+                'onclick' => sprintf(
+                    'return STUDIP.Dialog.confirm("%s".replace("%%s", $("#bulk tbody :checked").length), function() { $("#bulk").submit(); })',
+                    _('Wirklich %s Nachricht(en) löschen?')
+                )
             ]
         );
 

app/views/admin/autoinsert/manual.php

@@ -148,12 +148,10 @@
                     function (json) {
                         var result = "";
                         if (!json || json.error) {
-                            result = "Fehler".toLocaleString() + ": ";
-                            result += json.error
-                                ? json.error.toLocaleString()
-                                : "Fehler bei der Übertragung".toLocaleString();
+                            result = '<?= _('Fehler') ?>: ';
+                            result += json.error || '<?= _('Fehler bei der Übertragung') ?>';
                         } else {
-                            result = "Gefundene Nutzer".toLocaleString() + ": ";
+                            result = '<?= _('Gefundene Nutzer') ?>: ';
                             result += "<strong>" + json.users + "</strong>";
                         }
                         $('#autoinsert_count').html(result);
@@ -162,7 +160,7 @@
                 event.preventDefault();
             });
             $('input[name^=remove_filter]').click(function (event) {
-                return confirm("Wollen Sie diesen Filter wirklich entfernen?".toLocaleString());
+                return confirm('<?= _('Wollen Sie diesen Filter wirklich entfernen?') ?>');
             });
         });
     </script>

app/views/localizations/show.php

-<?php
-
-$translations = array(
-    // app/controllers/messages.php
-    'Wirklich %s Nachrichten löschen?' => _('Wirklich %s Nachrichten löschen?'),
-    ', [\'id\' => $this->user]))
-                        . \' Nachrichten endgültig gelöscht.' => _(', [\'id\' => $this->user]))
-                        . \' Nachrichten endgültig gelöscht.'),
-    ', [\'id\' => $this->user]))
-                        . \' Nachrichten endgütlig gelöscht.' => _(', [\'id\' => $this->user]))
-                        . \' Nachrichten endgütlig gelöscht.'),
-
-    // app/views/admin/autoinsert/manual.php
-    'Fehler' => _('Fehler'),
-    'Fehler bei der Übertragung' => _('Fehler bei der Übertragung'),
-    'Gefundene Nutzer' => _('Gefundene Nutzer'),
-    'Wollen Sie diesen Filter wirklich entfernen?' => _('Wollen Sie diesen Filter wirklich entfernen?'),
-
-    // public/assets/javascripts/ckeditor/plugins/studip-quote/plugin.js
-    'Zitat einfügen' => _('Zitat einfügen'),
-    'Zitat teilen' => _('Zitat teilen'),
-    'Zitat löschen' => _('Zitat löschen'),
-    '%s hat geschrieben:' => _('%s hat geschrieben:'),
-
-    // public/plugins_packages/core/Forum/javascript/forum.js
-    'Wenn Sie die Seite verlassen, gehen ihre Änderungen verloren!' => _('Wenn Sie die Seite verlassen, gehen ihre Änderungen verloren!'),
-    'Sind sie sicher, dass Sie diese Kategorie entfernen möchten? ' => _('Sind sie sicher, dass Sie diese Kategorie entfernen möchten? '),
-    'Alle Bereiche werden dann nach "Allgemein" verschoben!' => _('Alle Bereiche werden dann nach "Allgemein" verschoben!'),
-    'Sind sie sicher, dass Sie diesen Bereich löschen möchten? ' => _('Sind sie sicher, dass Sie diesen Bereich löschen möchten? '),
-    'Es werden auch alle Beiträge in diesem Bereich gelöscht!' => _('Es werden auch alle Beiträge in diesem Bereich gelöscht!'),
-    'Sind sie sicher, dass Sie ihren bisherigen Beitrag verwerfen wollen?' => _('Sind sie sicher, dass Sie ihren bisherigen Beitrag verwerfen wollen?'),
-    'Anonym' => _('Anonym'),
-    'Die Senderin/der Sender dieser Nachricht möchte Sie auf den folgenden Beitrag aufmerksam machen. ' => _('Die Senderin/der Sender dieser Nachricht möchte Sie auf den folgenden Beitrag aufmerksam machen. '),
-    'Link zum Beitrag: ' => _('Link zum Beitrag: '),
-    'Beitrag verschieben' => _('Beitrag verschieben'),
-    'Thema schließen' => _('Thema schließen'),
-    'Thema öffnen' => _('Thema öffnen'),
-    'Hervorhebung aufheben' => _('Hervorhebung aufheben'),
-    'Thema hervorheben' => _('Thema hervorheben'),
-
-);
-
-?>
-<?= json_encode($translations) ?>
\ No newline at end of file

lib/include/html_head.inc.php

@@ -57,10 +57,6 @@ $getInstalledLanguages = function () {
             wysiwyg_enabled: <?= json_encode((bool) Config::get()->WYSIWYG) ?>
         }
     </script>
-<? if ($_SESSION['_language'] !== 'de_DE'): ?>
-    <link rel="localization" hreflang="<?= htmlReady(strtr($_SESSION['_language'], '_', '-')) ?>"
-          href="<?= URLHelper::getScriptLink('dispatch.php/localizations/' . $_SESSION['_language']) ?>" type="application/vnd.oftn.l10n+json">
-<? endif ?>
 
     <?= PageLayout::getHeadElements() ?>
 

templates/layouts/base.php

@@ -86,10 +86,6 @@ $getInstalledLanguages = function () {
             server_timestamp: <?= time() ?>
         }
     </script>
-<? if ($_SESSION['_language'] !== 'de_DE'): ?>
-    <link rel="localization" hreflang="<?= htmlReady(strtr($_SESSION['_language'], '_', '-')) ?>"
-          href="<?= URLHelper::getScriptLink('dispatch.php/localizations/' . $_SESSION['_language']) ?>" type="application/vnd.oftn.l10n+json">
-<? endif ?>
 
     <?= PageLayout::getHeadElements() ?>