[koha-commits] main Koha release repository branch 20.05.x updated. v20.05.04-57-ge3737f5005

Git repo owner gitmaster at git.koha-community.org
Tue Oct 20 17:56:17 CEST 2020


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "main Koha release repository".

The branch, 20.05.x has been updated
       via  e3737f5005860c3fbffd88e0202b2747a2e7933f (commit)
       via  a7fb98694abdedef206215c82be505e8960f6529 (commit)
       via  12e4b053d2c8e80e2035d61467d0d533967778b2 (commit)
       via  3c4eae4a4d983b82633fe312730e6be49b9d4c7c (commit)
      from  787a5df550dff2f9618bd34d087bc5f620a86e9d (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit e3737f5005860c3fbffd88e0202b2747a2e7933f
Author: Owen Leonard <oleonard at myacpl.org>
Date:   Mon Aug 17 13:04:27 2020 +0000

    Bug 26225: Move translatable strings out of biblio_framework.tt and into biblio_framework.js
    
    This patch removes the definition of translatable strings out of
    templates and into the corresponding JavaScript file, using the new JS
    i81n function.
    
    To test:
    
    - Apply the patch, go to Administration -> MARC bibliographic framework.
    - Click Actions -> Export and save the file.
    - Click Actions -> Import.
    - Select a file which isn't CSV or ODS. You should get an error message,
      "Please select a CSV (.csv) or ODS (.ods) spreadsheet file."
    - Select the file you exported previously and click "Import." You should
      see an error message, "Are you sure you want to replace the fields and
      subfields for the default framework structure? ..."
    - Click "OK." You should see a message in the modal window, "Importing
      to framework:..."
    - Edit your exported framework file in such a way that it isn't a valid
      framework export.
    - Repeat the process of importing the file. You should get an error
      message, "Error importing the framework..."
    
    TESTING TRANSLATABILITY
    
    - Update a translation, e.g. fr-FR:
    
      > cd misc/translator
      > perl translate update fr-FR
    
    - Open the corresponding .po file for JavaScript strings, e.g.
      misc/translator/po/fr-FR-messages-js.po
    - Locate strings pulled from
      koha-tmpl/intranet-tmpl/prog/js/biblio_framework.js for translation,
      e.g.:
    
      msgid "Please select a CSV (.csv) or ODS (.ods) spreadsheet file"
      msgstr ""
    
    - Edit the "msgstr" string however you want (it's just for testing).
    - Install the updated translation:
    
      > perl translate install fr-FR
    
    - Switch to your newly translated language in the staff client
      and repeat the test plan above. The translated strings should
      appear.
    
    https://bugs.koha-community.org/show_bug.cgi?id=26226
    
    Signed-off-by: David Nind <david at davidnind.com>
    
    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
    
    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    (cherry picked from commit 1eb0fe4e5ea87423d4c3c936ad7320634a42c54e)
    
    Signed-off-by: Lucas Gass <lucas at bywatersolutions.com>

commit a7fb98694abdedef206215c82be505e8960f6529
Author: Owen Leonard <oleonard at myacpl.org>
Date:   Mon Aug 17 12:26:39 2020 +0000

    Bug 26225: Move translatable strings out of audio_alerts.tt and into audio_alerts.js
    
    This patch removes the definition of translatable strings out of
    templates and into the corresponding JavaScript file, using the new JS
    i81n function.
    
    To test:
    
    - Apply the patch, go to Administration -> Audio alerts.
    - Click the "Delete selected alerts" button without checking any
      checkboxes. You should see an error: "Check the box next to the alert
      you want to delete."
    - Check the checkbox for an existing sound and click "Delete selected
      alerts." You should get a confirmation message, "Are you sure you want
      to delete the selected audio alerts?"
    - Click "New alert."
    - Without filling any details, click the "Play sound" button. You should
      see an error message, "Please select or enter a sound."
    
    TESTING TRANSLATABILITY
    
    - Update a translation, e.g. fr-FR:
    
    > cd misc/translator
    > perl translate update fr-FR
    
    - Open the corresponding .po file for JavaScript strings, e.g.
      misc/translator/po/fr-FR-messages-js.po
    - Locate strings pulled from
      koha-tmpl/intranet-tmpl/prog/js/audio_alerts.js for translation, e.g.:
    
      msgid "Are you sure you want to delete the selected audio alerts?"
      msgstr ""
    
    - Edit the "msgstr" string however you want (it's just for testing).
    - Install the updated translation:
    
      > perl translate install fr-FR
    
    - Switch to your newly translated language in the staff client
      and repeat the test plan above. The translated strings should
      appear.
    
    Signed-off-by: David Nind <david at davidnind.com>
    
    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    (cherry picked from commit ac12b9c6485332f6ca34bb3d4a98c0c341bfe943)
    
    Signed-off-by: Lucas Gass <lucas at bywatersolutions.com>

commit 12e4b053d2c8e80e2035d61467d0d533967778b2
Author: Owen Leonard <oleonard at myacpl.org>
Date:   Tue Sep 1 13:46:35 2020 +0000

    Bug 26339: Move translatable strings out of addorderiso2709.tt into addorderiso2709.js
    
    This patch moves strings defined for translation in addorderiso2709.tt
    into addorderiso2709.js for translation using the new double-underscore
    i81n function.
    
    To test, apply the patch and go to Acquisitions -> Vendor -> Basket ->
    Add orders from MARC file.
    
     - Click "Add orders" next to a staged file.
     - Without making any selections, click "Save." You should get an error,
       "There is no record selected."
     - Select a record and click "Save." You should get an error, "Some
       budgets are not defined in item records."
     - Enter a non-numeric value in the "Quantity" field and click "Save."
       You should get an error, "1 quantity values are not filled in or are
        not numbers."
    
    TESTING TRANSLATABILITY
    
    - Update a translation, e.g. fr-FR:
    
      > cd misc/translator
      > perl translate update fr-FR
    
    - Open the corresponding .po file for JavaScript strings, e.g.
      misc/translator/po/fr-FR-messages-js.po
    - Locate strings pulled from
      koha-tmpl/intranet-tmpl/prog/js/addorderiso2709.js for translation,
      e.g.:
    
      msgid "Some budgets are not defined in item records"
      msgstr ""
    
    - Edit the "msgstr" string however you want (it's just for testing).
    - Install the updated translation:
    
      > perl translate install fr-FR
    
    - Switch to your newly translated language in the staff client
      and repeat the test plan above. The translated strings should
      appear.
    
    Signed-off-by: David Nind <david at davidnind.com>
    
    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    (cherry picked from commit 2ac76aee66c5114a944bc0b00d952b3df5dad85a)
    
    Signed-off-by: Lucas Gass <lucas at bywatersolutions.com>

commit 3c4eae4a4d983b82633fe312730e6be49b9d4c7c
Author: Owen Leonard <oleonard at myacpl.org>
Date:   Thu Aug 20 15:53:28 2020 +0000

    Bug 25317: Move translatable strings out of additem.js.inc
    
    This patch moves the definition of translatable strings out of
    additem.js.inc and into additem.js using the new JS i81n function.
    additem.js.inc is removed, being obsolete.
    
    To test:
    
    When creating an order:
    
    - Go to Administration -> System preferences and set "AcqCreateItem" to
      "when placing an order."
    - Apply the patch and go to Acquisitions -> Vendor -> Add to basket ->
      From a new (empty) record.
    - In the "Item" section, confirm that the buttons at the bottom of the
      form are correct: "Add item," "Clear," and "Add multiple items."
    - Click "Add multiple items." The placeholder in the revealed form field
      should read "Number of items to add." The corresponding button should
      be labeled "Add."
    - You should see a note, "NOTE: Fields listed in the 'UniqueItemsFields'
      system preference will not be copied."
    - Fill out the item entry form and add a number to the "multiple items"
      field. Click "Add."
    - A table of items should be displayed with "Edit" and "Delete" buttons
      for each new item.
    - Click one of the "Edit" buttons. The item form should be populated
      with the item's data, with an "Update item" button at the bottom.
    
    When receiving an order:
    
    - Go to Administration -> System preferences and set "AcqCreateItem" to
      "when receiving an order."
    - Go to Acquisitions -> Vendor -> Receive shipments.
    - Select or create an invoice.
    - Click "Receive" on an order which has a quantity greater than 1.
    - Add two items, duplicating in each at least one value which is marked
      as unique by the "UniqueItemFields" system preference (e.g. fill in
      the same barcode number for each item).
    - Click "Save." You should get an alert about duplicated values, and
      there should be an alert message on the page, "barcode 'XXX' is
      duplicated."
    - Edit one of the two items and change the barcode to one which already
      exists in your database.
    - Click "Save." An alert message should be shown on the page, "barcode
      'XXX' already exists in the database."
    - Note: I was unable to find out how to trigger this error, "You can't
      receive any more items." It may be obsolete.
    
    TESTING TRANSLATABILITY
    
    - Update a translation, e.g. fr-FR:
    
      > cd misc/translator
      > perl translate update fr-FR
    
    - Open the corresponding .po file for JavaScript strings, e.g.
      misc/translator/po/fr-FR-messages-js.po
    - Locate strings pulled from
      koha-tmpl/intranet-tmpl/prog/js/additem.js for translation,
      e.g.:
    
      msgid "Add multiple items"
      msgstr ""
    
    - Edit the "msgstr" string however you want (it's just for testing).
    - Install the updated translation:
    
      > perl translate install fr-FR
    
    - Switch to your newly translated language in the staff client
      and repeat the test plan above. The translated strings should
      appear.
    
    Signed-off-by: David Nind <david at davidnind.com>
    
    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    (cherry picked from commit 9d056d0364cfdc96e5c8fc6a84f933db970ffbd0)
    
    Signed-off-by: Lucas Gass <lucas at bywatersolutions.com>

-----------------------------------------------------------------------

Summary of changes:
 .../intranet-tmpl/prog/en/includes/additem.js.inc  | 16 ------------
 .../prog/en/modules/acqui/addorderiso2709.tt       |  6 -----
 .../prog/en/modules/acqui/neworderempty.tt         |  1 -
 .../prog/en/modules/acqui/orderreceive.tt          |  1 -
 .../prog/en/modules/admin/audio_alerts.tt          |  5 ----
 .../prog/en/modules/admin/biblio_framework.tt      |  4 ---
 koha-tmpl/intranet-tmpl/prog/js/additem.js         | 29 +++++++++++-----------
 koha-tmpl/intranet-tmpl/prog/js/addorderiso2709.js | 10 ++++----
 koha-tmpl/intranet-tmpl/prog/js/audio_alerts.js    |  8 +++---
 .../intranet-tmpl/prog/js/biblio_framework.js      | 11 ++++----
 10 files changed, 30 insertions(+), 61 deletions(-)
 delete mode 100644 koha-tmpl/intranet-tmpl/prog/en/includes/additem.js.inc


hooks/post-receive
-- 
main Koha release repository


More information about the koha-commits mailing list