[Koha-bugs] [Bug 11431] Add additional sound options for warnings
bugzilla-daemon at bugs.koha-community.org
bugzilla-daemon at bugs.koha-community.org
Thu Feb 19 15:04:26 CET 2015
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11431
Jonathan Druart <jonathan.druart at biblibre.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Signed Off |Failed QA
--- Comment #69 from Jonathan Druart <jonathan.druart at biblibre.com> ---
(In reply to paxed from comment #68)
> (In reply to Kyle M Hall from comment #67)
> > Created attachment 35846 [details] [review] [review]
>
> Just wondering if http:// vs https:// matters for this:
>
> + // This is way faster than substring
> + if ( ! ( sound.charAt(4) == ':' && sound.charAt(5) == '/' &&
> sound.charAt(6) == '/' ) ) {
Good catch, maybe a regex is much more appropriate here.
--
You are receiving this mail because:
You are watching all bug changes.
More information about the Koha-bugs
mailing list