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

--- Comment #12 from Nick Clemens <n...@bywatersolutions.com> ---
(In reply to Jonathan Druart from comment #10)
> Comment on attachment 55093 [details] [review]
> /youtu\.?be/
> seems easier to read, don't you think?

Yeah, I can get behind that

> @@ +109,5 @@
> > +                                'fmt' => '43' #webm is the only format 
> > compatible to all modern browsers. maybe check for available qualities
> > +                            }
> > +                        );
> > +                    };
> > +                    if ($@) { warn $@; }
> 
> What kind of errors do you handle here?
if an URL contains 'youtube' but isn't a www.youtube.com link the downloader
will fail - this way it fails nicer - something like this:
http://www.nytimes.com/topic/company/youtube

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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