For a lot of good reasons it is better to have flash player or others external libraries in both module separate and all modules common place.
Current usage is to place these kinds of data in sites/all/libraries/.

See module libraries.
Furthermore module SWF Tools has recently implemented this method.

CommentFileSizeAuthor
#4 2_added_support_for_libraries.patch6.22 KBmichaek

Comments

heylookalive’s picture

heylookalive commented
Assigned: Unassigned » heylookalive

Agreed on this, will look into using the API without necessarily requiring the module dependency.

Anonymous’s picture

Anonymous (not verified) commented

OK !

This module could help you in _video_for_everbody_get_available_players() function.

Libraries module have a very small size and can be use by other modules.
I do not want to have a lot of modules on my Drupal sites but as for me this one is very important.

ekes’s picture

ekes commented

+1 for libraries dependency, it's gaining popularity and not repeating the detection code is worth it.

michaek’s picture

michaek commented
Status: Active » Needs review
StatusFileSize
new 2_added_support_for_libraries.patch6.22 KB

I implemented this, requiring the dependency. I don't think this code is perfect yet, but I think it's definitely worth using as a starting point. The patch I've attached follows from the patch I posted to #863636: Refactor html output to template, so don't be surprised if it doesn't work on its own!

heylookalive’s picture

heylookalive commented

Hi,

Thanks for the code :) I'll review it and get use of libraries API into the next release.

Thanks!

klonos’s picture

klonos
he/him
Primary language English
Location 90% Melbourne, Australia - 10% Larissa, Greece
commented
Title: Do not embed flash player or external libraries in module » Do not embed flash player or external libraries in module - Use the Libraries API instead.

...following.