From d607508135a7b87a6010d40e90fe56f8086fda1f Mon Sep 17 00:00:00 2001
From: JcDenis <contact@jcdenis.fr>
Date: Thu, 19 Aug 2021 22:29:52 +0200
Subject: [PATCH] move settings to _config file

---
 _config.php |  26 ++++++++-
 index.php   | 155 ----------------------------------------------------
 2 files changed, 25 insertions(+), 156 deletions(-)

diff --git a/_config.php b/_config.php
index 9a086f8..359365f 100644
--- a/_config.php
+++ b/_config.php
@@ -42,12 +42,36 @@ $combo_backup_folder = [
 ];
 
 $combo_start_page = [
-    'setting' => __('Settings'),
     'modules_plugin' => __('Plugins'),
     'modules_theme' => __('Themes'),
     'pack' => __('Import/Export')
 ];
 
+# -- Set settings --
+if (!empty($_POST['save'])) {
+        try {
+            if (empty($_POST['translater_write_po'])
+             && empty($_POST['translater_write_langphp'])) {
+                throw new Exception('You must choose one file format at least');
+            }
+            foreach($translater->getDefaultSettings() as $k => $v) {
+                $translater->set($k, (isset($_POST['translater_' . $k]) ? $_POST['translater_' . $k] : ''));
+            }
+            foreach($translater->proposal->getTools() AS $k => $v) {
+                $v->save();
+            }
+            dcPage::addSuccessNotice(
+                __('Configuration has been successfully updated.')
+            );
+            http::redirect(
+                $list->getURL('module=translater&conf=1&redir=' .
+                $list->getRedir())
+            );
+        } catch (Exception $e) {
+            $core->error->add(sprintf($errors[$action], $e->getMessage()));
+        }
+}
+
 # -- Display form --
 echo '
 <div class="fieldset">
diff --git a/index.php b/index.php
index ecda5f3..1f9adda 100644
--- a/index.php
+++ b/index.php
@@ -23,30 +23,6 @@ dcPage::checkSuper();
 
 $translater = new dcTranslater($core);
 
-$combo_start_page = [
-    'setting' => __('Settings'),
-    'modules_plugin' => __('Plugins'),
-    'modules_theme' => __('Themes'),
-    'pack' => __('Import/Export')
-];
-
-$combo_backup_limit = [
-    5 => 5,
-    10 => 10,
-    15 => 15,
-    20 => 20,
-    40 => 40,
-    60 => 60
-];
-
-$combo_backup_folder = [
-    'module' => __('locales folders of each module'),
-    'plugin' => __('plugins folder root'),
-    'public' => __('public folder root'),
-    'cache' => __('cache folder of Dotclear'),
-    'translater' =>__('locales folder of translater')
-];
-
 $succes = [
     'save_setting' => __('Configuration successfully updated'),
     'update_lang' => __('Translation successfully updated'),
@@ -306,30 +282,6 @@ switch ($action) {
         }
     break;
 
-    /**
-     * Save settings
-     */
-    case 'save_setting':
-        try {
-            if (empty($_POST['translater_write_po'])
-             && empty($_POST['translater_write_langphp'])) {
-                throw new Exception('You must choose one file format at least');
-            }
-
-            foreach($translater->getDefaultSettings() as $k => $v) {
-                $translater->set($k,(isset($_POST['translater_' . $k]) ? $_POST['translater_' . $k] : ''));
-            }
-
-            foreach($translater->proposal->getTools() AS $k => $v) {
-                $v->save();
-            }
-
-            http::redirect($p_url . '&part=setting&msg=' . $action);
-        } catch (Exception $e) {
-            $core->error->add(sprintf($errors[$action], $e->getMessage()));
-        }
-    break;
-
     /**
      * Modules / Unknow / None
      */
@@ -423,14 +375,6 @@ switch ($part) {
     case 'pack':
         $title = '<span class="page-title">' . __('Import/Export') . '</span>';
     break;
-
-    /**
-     * Settings
-     */
-    case 'setting': 
-    default:
-        $title = '<span class="page-title">' . __('Settings') . '</span>';
-    break;
 }
 
 #
@@ -1118,105 +1062,6 @@ switch ($part) {
         </form>
         </div>';
     break;
-
-    /**
-     * Settings
-     */
-    case 'setting':
-    default:
-        echo '
-        <form id="setting-form" method="post" action="' . $p_url . '">
-
-        <div class="multi-part" id="setting-translation" title="' . __('Translation') . '">
-        <p><label class="classic">' . 
-        form::checkbox('translater_write_po', '1' ,$translater->write_po) . ' 
-        ' . __('Write .po files') . '</label></p>
-        <p><label class="classic">' . 
-        form::checkbox('translater_write_langphp', '1', $translater->write_langphp) . ' 
-        ' . __('Write .lang.php files') . '</label></p>
-        <p><label class="classic">' . 
-        form::checkbox('translater_scan_tpl', '1', $translater->scan_tpl) . ' 
-        ' . __('Translate also strings of template files') . '</label></p>
-        <p><label class="classic">' . 
-        form::checkbox('translater_parse_nodc', '1', $translater->parse_nodc) . ' 
-        ' . __('Translate only unknow strings') . '</label></p>
-        <p><label class="classic">' . 
-        form::checkbox('translater_hide_default', '1', $translater->hide_default) . ' 
-        ' . __('Hide default modules of Dotclear') . '</label></p>
-        <p><label class="classic">' . 
-        form::checkbox('translater_parse_comment', '1', $translater->parse_comment) . ' 
-        ' . __('Write comments in files') . '</label></p>
-        <p><label class="classic">' . 
-        form::checkbox('translater_parse_user', '1', $translater->parse_user) . ' 
-        ' . __('Write informations about author in files') . '</label><br />
-        ' . form::field('translater_parse_userinfo', 65, 255, $translater->parse_userinfo) . '</p>
-        </div>
-
-        <div class="multi-part" id="setting-tool" title="' . __('Tools') . '">
-        <p><label class="classic">' . __('Default language of l10n source:') . '<br />' . 
-        form::combo('translater_proposal_lang',
-            array_flip($translater->getIsoCodes()), $translater->proposal_lang) . '</label></p>
-
-        <h4>' . __('Select and configure the tool to use to translate strings:') . '</h4>';
-
-        foreach($translater->proposal->getTools() AS $k => $v) {
-            $form = $v->form();
-
-            echo '
-            <dd>
-            <dt><label class="classic">' . 
-            form::radio('translater_proposal_tool', $k, $k == $translater->proposal_tool) . ' 
-            ' . $v->getDesc() . '</label></dt><dd>' . 
-            (empty($form) ?
-                '<p>' . sprintf(__('Nothing to configure for %s tool . '), $v->getName()) . '</p>' :
-                $form
-            ) . '</dd></dl>';
-        }
-
-        echo '
-        </div>
-
-        <div class="multi-part" id="setting-pack" title="' . __('Import/Export') . '">
-        <p><label class="classic">' . 
-        form::checkbox('translater_import_overwrite', '1', $translater->import_overwrite) . ' 
-        ' . __('Overwrite existing languages') . '</label></p>
-        <p><label class="classic">' . __('Name of exported package') . '<br />
-        ' . form::field('translater_export_filename', 65, 255, $translater->export_filename) . '</label></p>
-        </div>
-
-        <div class="multi-part" id="setting-backup" title="' . __('Backups') . '">
-        <p><label class="classic">' . 
-        form::checkbox('translater_backup_auto', '1', $translater->backup_auto) . ' 
-        ' . __('Make backups when changes are made') . '</label></p>
-        <p><label class="classic">' . sprintf(__('Limit backups to %s files per module'),
-        form::combo('translater_backup_limit',
-            array_flip($combo_backup_limit), $translater->backup_limit)) . '</label></p>
-        <p><label class="classic">' . sprintf(__('Store backups in %s'),
-        form::combo('translater_backup_folder',
-            array_flip($combo_backup_folder), $translater->backup_folder)) . '</label></p>
-        </div>
-
-        <div class="multi-part" id="setting-plugin" title="' . __('Behaviors') . '">
-        <p><label class="classic">' . __('Default start menu:') . '<br />' . 
-        form::combo('translater_start_page',
-            array_flip($combo_start_page), $translater->start_page) . '</label></p>
-        <p><label class="classic">' . 
-        form::checkbox('translater_plugin_menu', '1', $translater->plugin_menu) . ' 
-        ' . __('Enable menu on extensions page') . '</label></p>
-        <p><label class="classic">' . 
-        form::checkbox('translater_theme_menu', '1', $translater->theme_menu) . ' 
-        ' . __('Enable menu on themes page') . '</label></p>
-        </div>
-
-        <div class="clear">
-        <p><input type="submit" name="save" value="' . __('Save') . '" />' . 
-        $core->formNonce() . 
-        form::hidden(array('p'), 'translater') . 
-        form::hidden(array('part'), 'setting') . 
-        form::hidden(array('action'), 'save_setting') . '
-        </p></div>
-        </form>';
-    break;
 }
 
 dcPage::helpBlock('translater');