Add basic facility for extra options in GUI and API.

scriptshifter/hooks/korean/romanizer.py

@@ -162,7 +162,7 @@ def _romanize_name(src):
 
     if re.search("[a-zA-Z0-9]", src):
         warnings.append(f"{src} is not a recognized personal name.")
-        return None, warnings
+        return "", warnings
 
     # `parsed` can either be a modified Korean string with markers, or in case
     # of a foreign name, the final romanized name.
@@ -180,7 +180,7 @@ def _romanize_name(src):
 
             if not any(lname_rom_ls):
                 warnings.append(f"{parsed} is not a recognized Korean name.")
-                return None, warnings
+                return "", warnings
 
             lname_rom = " ".join(lname_rom_ls)
 
@@ -198,7 +198,7 @@ def _romanize_name(src):
             return parsed, warnings
 
     warnings.append(f"{src} is not a recognized Korean name.")
-    return None, warnings
+    return "", warnings
 
 
 def _parse_kor_name(src):

scriptshifter/rest_api.py

@@ -4,7 +4,7 @@ from base64 import b64encode
 from copy import deepcopy
 from os import environ, urandom
 
-from flask import Flask, Response, jsonify, render_template, request
+from flask import Flask, jsonify, render_template, request
 
 from scriptshifter.tables import list_tables, load_table
 from scriptshifter.trans import transliterate
@@ -60,23 +60,32 @@ def dump_table(lang):
     return jsonify(tbl)
 
 
-@app.route("/transliterate", methods=["POST"])
-def transliterate_form():
-    """ UI version of the `trans` endpoint. Passes everything via form. """
-    return transliterate_req(
-            request.form["lang"], request.form.get("r2s", False))
+@app.route("/options/<lang>", methods=["GET"])
+def get_options(lang):
+    """
+    Get extra options for a table.
+    """
+    tbl = load_table(lang)
 
+    return jsonify(tbl.get("options", []))
 
-@app.route("/trans/<lang>/r2s", methods=["POST"], defaults={"r2s": True})
-@app.route("/trans/<lang>", methods=["POST"])
-def transliterate_req(lang, r2s=False):
+
+@app.route("/trans", methods=["POST"])
+def transliterate_req():
+    lang = request.form["lang"]
     in_txt = request.form["text"]
     capitalize = request.form.get("capitalize", False)
+    t_dir = request.form.get("t_dir", "s2r")
+    if t_dir not in ("s2r", "r2s"):
+        return f"Invalid direction: {t_dir}", 400
+
     if not len(in_txt):
         return ("No input text provided! ", 400)
+    options = request.form.get("options", {})
+    logger.debug(f"Extra options: {options}")
 
     try:
-        out, warnings = transliterate(in_txt, lang, r2s, capitalize)
+        out, warnings = transliterate(in_txt, lang, t_dir, options, capitalize)
     except (NotImplementedError, ValueError) as e:
         return (str(e), 400)
 

scriptshifter/tables/data/korean_names.yml

@@ -2,6 +2,13 @@ general:
   name: Korean (Names)
   description: Korean names S2R.
 
+options:
+  - id: marc_field
+    label: MARC field
+    description: enter 0 if not applicable.
+    type: int
+    default: 0
+
 script_to_roman:
   hooks:
     post_config:

scriptshifter/tables/data/korean_nonames.yml

@@ -1,5 +1,5 @@
 general:
-  name: Korean (Nnon-names)
+  name: Korean (Non-names)
   description: Korean S2R.
 
 script_to_roman:

scriptshifter/templates/index.html

@@ -32,7 +32,7 @@
 
 
 
-    <form id="transliterate" action="/transliterate" method="POST">
+    <form id="transliterate" action="/trans" method="POST">
         <fieldset>
             <label for="text">Input text</label>
             <textarea id="text" name="text"></textarea>
@@ -42,9 +42,19 @@
                     <option value="{{ k }}">{{ v["name"] }}</option>
                 {% endfor %}
             </select>
+        </fieldset>
+        <fieldset>
+            <legend>Direction</legend>
             <div>
-                <label class="label-inline" for="r2s">Roman to Script</label>
-                <input type="checkbox" id="r2s" name="r2s">
+                <label class="label-inline" for="s2r">Script to Roman</label>
+                <input
+                        type="radio" id="opt_s2r" name="t_dir" value="s2r"
+                        checked>
+            </div>
+            <div>
+                <label class="label-inline" for="r2s">Roman to script</label>
+                <input
+                        type="radio" id="opt_r2s" name="t_dir" value="r2s">
             </div>
         </fieldset>
         <fieldset>
@@ -64,6 +74,7 @@
                 <input type="radio" id="all" name="capitalize" value="all">
             </div>
         </fieldset>
+        <div id="options"></div>
         <fieldset>
             <input class="button-primary" type="submit" value="Transliterate!">
         </fieldset>
@@ -77,26 +88,59 @@
     </div>
 
     <script type="text/javascript">
-        document.getElementById('transliterate').addEventListener('submit',(event)=>{
+        document.getElementById('lang').addEventListener('change',(event)=>{
+            let lang = document.getElementById("lang").value;
+
+            fetch('/options/' + lang)
+              .then(response=>response.json())
+                .then((data) => {
+                    document.getElementById("options").replaceChildren();
+                    data.forEach((opt)=>{
+                        let fset = document.createElement("fieldset");
+                        let label = document.createElement("label");
+                        label.setAttribute("for", opt.id);
+                        label.append(opt.label);
+
+                        let input = document.createElement("input");
+                        input.setAttribute("id", opt.id);
+                        input.setAttribute("name", opt.id);
+                        input.classList.add("option_i");
+                        input.value = opt.default;
+
+                        fset.append(label, input);
+                        document.getElementById("options").append(fset);
+                    });
+                });
+
+            event.preventDefault();
+            return false;
+        })
+        document.getElementById('lang').dispatchEvent(new Event('change'));
 
 
+        document.getElementById('transliterate').addEventListener('submit',(event)=>{
 
             const data = new URLSearchParams();
 
+            let t_dir = Array.from(document.getElementsByName("t_dir")).find(r => r.checked).value;
+
             let capitalize = Array.from(document.getElementsByName("capitalize")).find(r => r.checked).value;
 
 
             data.append('text',document.getElementById('text').value)
             data.append('lang',document.getElementById('lang').value)
-
-            if (document.getElementById('r2s').checked){
-                data.append('r2s','on')
-            }
-
+            data.append('t_dir',t_dir)
             data.append('capitalize',capitalize)
 
+            let options = {};
+            let option_inputs = document.getElementsByClassName("option_i");
+            for (i = 0; i < option_inputs.length; i++) {
+                let el = option_inputs[i];
+                options[el.getAttribute('id')] = el.value;
+            };
+            data.append('options', JSON.stringify(options));
 
-            fetch('/transliterate', {
+            fetch('/trans', {
                 method: 'post',
                 body: data,
             })
@@ -117,12 +161,6 @@
               alert("Error:\n" + error)
             });
 
-
-
-
-
-
-
             event.preventDefault()
             return false
 

scriptshifter/trans.py

@@ -36,7 +36,7 @@ class Context:
         self.dest_ls = []
 
 
-def transliterate(src, lang, r2s=False, capitalize=False):
+def transliterate(src, lang, t_dir="s2r", options={}, capitalize=False):
     """
     Transliterate a single string.
 
@@ -45,6 +45,13 @@ def transliterate(src, lang, r2s=False, capitalize=False):
 
         lang (str): Language name.
 
+        t_dir (str): Transliteration direction. Either `s2r` for
+            script-to-Roman (default) or `r2s`  for Roman-to-script.
+
+        capitalize: capitalize words: one of `False` (no change - default),
+            `"first"` (only first letter), or `"all"` (first letter of each
+            word).
+
     Keyword args:
         r2s (bool): If False (the default), the source is considered to be a
         non-latin script in the language and script specified, and the output
@@ -54,8 +61,8 @@ def transliterate(src, lang, r2s=False, capitalize=False):
     Return:
         str: The transliterated string.
     """
-    source_str = "Latin" if r2s else lang
-    target_str = lang if r2s else "Latin"
+    source_str = "Latin" if t_dir == "r2s" else lang
+    target_str = lang if t_dir == "r2s" else "Latin"
     logger.info(f"Transliteration is from {source_str} to {target_str}.")
 
     cfg = load_table(lang)
@@ -64,16 +71,18 @@ def transliterate(src, lang, r2s=False, capitalize=False):
     # General directives.
     general = cfg.get("general", {})
 
-    if not r2s and "script_to_roman" not in cfg:
+    if t_dir == "s2r" and "script_to_roman" not in cfg:
         raise NotImplementedError(
             f"Script-to-Roman transliteration not yet supported for {lang}."
         )
-    elif r2s and "roman_to_script" not in cfg:
+    elif t_dir == "r2s" and "roman_to_script" not in cfg:
         raise NotImplementedError(
             f"Roman-to-script transliteration not yet supported for {lang}."
         )
 
-    langsec = cfg["script_to_roman"] if not r2s else cfg["roman_to_script"]
+    langsec = (
+            cfg["script_to_roman"] if t_dir == "s2r"
+            else cfg["roman_to_script"])
     # langsec_dir = langsec.get("directives", {})
     langsec_hooks = langsec.get("hooks", {})