* adds explanantory sr-only text
* also use ' instead of " for facet name for other translations

co-authored by: Robert Lange <robert.lange@uni-leipzig.de>

* department as required field
* branch issue/19158_a

* added at ticket 19370
* was removed erroneously in ticket 19603

* use getTOC and specific data template for TOC tab

Robert Lange authored 3 years ago and Dorian Merz committed 3 years ago

* beautify bootstrap-accessibility-en.min.js and bootstrap-accessibility-de.min.js
* override bootstrap show / hide / addAriaAndCollapsedClass: don't add aria-expanded to collapsing element
** copy methods from bootstrap.min.js and avoid setting of aria-expanded="false" or aria-expanded="true" for elements to collapse (e.g. ul)
* Recommend SideFacets: add aria-expanded for collapse trigger buttons in templates on page load

Dorian Merz authored 3 years ago and André Lahmann committed 3 years ago

* use static variable for short term caching

* but catching of error still  needs to be optimized

* removes redundant aria-labels

* removes redundant aria-hidden in footer finc link

* avoid loading spinner

* also set input division (fakultät) required in dds-form.php

Gregor Gawol authored 3 years ago and Dorian Merz committed 3 years ago

* invisible comboboxes required false

Gregor Gawol authored 4 years ago and Dorian Merz committed 3 years ago

for details see issue/19158

Robert Lange authored 3 years ago and Dorian Merz committed 3 years ago

* get multiple lines
* show additional data (f) for Marc license information
* combine text a (note f) with url u as link in data-license.phtml
* use external links for licenses
* move template from Default to new SolrMarc directory
* consolidate data-license of Marc and DiCo to avoid confusion with Marc data-license and digital collections introduced with #14583
** rename getLicenceInfo to getLicense in SolrDico record driver
** add Dico data-licence.phtml (moved from de_15)
** use same labels like Dico - possible to use same template under DefaultRecord

* set label sr-only

Robert Lange authored 3 years ago and Dorian Merz committed 3 years ago

* also delete obsolete sample configuration from config.ini
* fix and update test for ExternalCatalogueLink
** use record_id instead of ppn
** fix mocking expactations
** use test by calling: docker exec -it [your_favorite_vufind_folder]_php_1 sh -c "vendor/bin/phpunit --configuration module/finc/tests/phpunit.xml module/finc/tests/unit-tests/src/fincTest/View/Helper/Root/ExternalCatalogueLinkTest.php"

Dorian Merz authored 3 years ago and André Lahmann committed 3 years ago

* streamlines with result list title

* do not show tab for single-element trees

* do not set refreshOnClose to true in lightbox after neccessary catalog login for saving a record as favorite
** set userIsLoggedIn to true
** hide login link
** show logout link
* improve code styles
* also anticipate jshint for 17984 to set back focus on origin after closing
* control overriding of page reload by extra parameter in template link

* usage after changing translations via i18n controller /Admin/I18n: "devops/i18n-merge.sh data/i18n/languages/de.ini local/languages/de.ini"
* add minified local/languages/de.ini
* corrects German typo for errorcode_member_not_found in local

* output:
** remove duplicate token "Email " with obsolete value  "E-Mail" by  "Mailen".
** remove duplicate token "Print " with obsolete value  Druck by  "Drucken".
** remove duplicate token "bookbag_email_selected " with obsolete value  "E-Mail" by  "Links zu ausgewählten Medien per E-Mail versenden".
** remove duplicate token "Skip_navigation_links " with obsolete value  "Sprunglinks zum Inhalt" by  "Sprunglinks zur Suche und zum Inhalt".
** replace token "errorcode_member_not_found " with old value  "Die Benutzernummer exisitiert nicht" by new value  "Der aufgerufene Titel (%%id%%) ist nicht vorhanden."

* minify local en.ini
* usage "devops/i18n-merge.sh data/i18n/languages/en.ini local/languages/en....

* adds delegator for SolrMarcFinc
* can add SolrMarcUrlRules.yaml now
* decide on sub-result sets to be used in actual result of getURLs

* set non-lucene-syntax handling configurable on a per-SearchType basis

co-authored by: André Lahmann <lahmann@ub.uni-leipzig.de>
* added some more comments to searches.ini

Claas Kazzer authored 3 years ago and Dorian Merz committed 3 years ago

* changes overcomplex wording in carousel J

Robert Lange authored 3 years ago and Dorian Merz committed 3 years ago

* allow list of ids and load results as batch from solr
* escape colons centralized in getFincId
* code styles
* move searchRelatedRecords from SolrMarcFincTrait to new finc\Controller\CustomTraits\SearchTrait
* call searchRelatedRecords instead of getFincId within getAlternativeItemId
* merge elements of getFincId into getAlternativeItemId
** handle multiple ids within getAlternativeItemId
** escape colon when existing
** rude exception catching
* replace call of getFincId by getAlternativeItemId within mapPaiaItems

co-authored by "Dorian Merz <merz@ub.uni-leipzig.de>"
* slight refactoring
* add more method comments

co-authored by "André Lahmann <lahmann@ub.uni-leipzig.de>"
* added comment for choice of trait location

* copy from DE-15 code to finc's FincLibero
* use separate queryBoundItem()
* use generic pattern
* make bound item query adaptable

* isTitleHold
* placeTitleHold
* extendPickUpLocation

* now named LiberoWachtl
* less misleading naming
* refactor liberoWachtlUrl INI key
** plus: cache in local property

Claas Kazzer authored 4 years ago and Dorian Merz committed 3 years ago

* creates a German version in finc-accessibility
* translate caroussel terms to German
* use empty standard version to overwrite outdated default
* use language specific versions for German and English in layout template
* add template for similar items to load accessibility plugin as ajax tab
** necessary to register new elements loaded by ajax

co-authored by: Robert Lange <robert.lange@uni-leipzig.de>

Claas Kazzer authored 3 years ago and Dorian Merz committed 3 years ago

* fixes h3 on My Accout Sidebar on XS

Robert Lange authored 3 years ago and Dorian Merz committed 3 years ago

* extend AbstractAdmin to check for access permission for AdminModule
** admin module must be activated in config.ini by admin_enabled = true
** permission access.AdminModule should be set in permissions.ini

Alexander Purr authored 3 years ago and Dorian Merz committed 3 years ago

* own variables are not needed here any more