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

--- Comment #8 from Janusz Kaczmarek <[email protected]> ---
(In reply to Marcel de Rooy from comment #7)
> +                    if ( end - start > 1 ) {
> 
> What is the purpose of this test?
> Should it be >0 btw ?
> end index is not inclusive in the substring call
> 
> What about other places in the script where we do a similar call?

This is to not delete spaces from one-character-long positions in 008. Without
this change activating select drop-down looses the focus, because the value
becomes an empty string instead of remaining ' '.  But for longer input fields,
where end - start > 1 (e.g. like 008/7-10, 008/11-14 for dates) we want not to
have spaces, but have the cursor placed at the beginning of the input field.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to