[Koha-bugs] [Bug 21861] New: The MARC modification template actions editor does not always validate user input

[bugzilla-daemon at bugs.koha-community.org]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21861

            Bug ID: 21861
           Summary: The MARC modification template actions editor does not
                    always validate user input
 Change sponsored?: ---
           Product: Koha
           Version: master
          Hardware: All
                OS: All
            Status: NEW
          Severity: normal
          Priority: P5 - low
         Component: Tools
          Assignee: koha-bugs at lists.koha-community.org
          Reporter: arouss1980 at gmail.com
        QA Contact: testopia at bugs.koha-community.org

When editing a MARC modification template action, you have the option
of adding a conditional clause ("if/unless") to the action you want to
perform. The conditional clause can be further fine-tuned by selecting
one of four comparison operators ("exists/doesn't exist/matches/doesn't
match") from a drop-down list.

There are currently four issues when adding such a clause, namely:

1) When you pick a condition (e.g. "unless") and click on 'Add action'
   or 'Update action', the jQuery function that validates your input
   does not check whether you have actually filled in a value for the
   MARC field (mandatory) or the comparison operator (mandatory).

2) If you pick "matches" or "doesn't match" as a comparison operator,
   the textbox next to the operator can be left blank when saving the
   modification action (the textbox _should_ have a value because the
   operator is meaningless without it).

3) If you check the "RegEx" checkbox and then enter a value in the
   relevant textbox, when you Add/Update the action the "Action"
   column in the actions table will display extra spaces between the
   regex pattern and its delimiters.

4) If you check the "RegEx" checkbox and then enter a value in the
   relevant textbox, when you later edit that modification action
   the "m/ /" characters surrounding the regex textbox are not shown
   anymore.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.