[Koha-bugs] [Bug 33103] Add vendor aliases

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

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

Katrin Fischer <katrin.fischer at bsz-bw.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Signed Off                  |Failed QA

--- Comment #34 from Katrin Fischer <katrin.fischer at bsz-bw.de> ---
1) Please check:

 OK     Koha/Acquisition/Bookseller.pm
 FAIL   Koha/Acquisition/Bookseller/Alias.pm
   FAIL   pod coverage
                POD coverage was greater before, try perl
-MPod::Coverage=PackageName -e666

 OK     Koha/Acquisition/Bookseller/Aliases.pm
 OK     Koha/Schema/Result/Aqbookseller.pm
 FAIL   Koha/Schema/Result/AqbooksellerAlias.pm
   FAIL   pod coverage
                POD is missing for koha_object_class
                POD is missing for koha_objects_class

2) Vendor detail page

* We should add some space between the the alias name and the trash
* More important: it needs at least a tool tip, right now it's a big
accessibility issue. I believe with aria-hidden=true it will actually be
invisible to screen readers?
* Same goes for the Plus icon to add another field: space and accessibility.
* When you click 'enter' in the field, the form submits 'save' and you are back
on the detail page, but the alias was not saved.


3) Other features

* Search and display in the aquisition module work nicely

4) ERM module

I am not persuaded by the display tbh, I believe it will get messy very soon as
often those aliases will be quite long and we might even want to record a
slight name change. 

Could it be an option to show the aliases in the list as "second level"
entries, kind of a hiearchical display, like we have it for funds? 


I know that CORAL uses the aliases also for matching in some form (I believe
when importing data), so it will be really nice to have this feature.

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