After creating #999524: Centralize syncing of data pulled from Ooyala, I found that it would be valuable to separate our Taxonomy integration into a separate module so that it can implement its own hook_nodeapi() and hook_ooyala_sync_pull_videos() functions. Right now our code is littered with if module_exists('taxonomy') all over the place.

CommentFileSizeAuthor
#1 ooyala_separate_taxonomy.patch74.91 KBquicksketch

Comments

quicksketch’s picture

quicksketch
he/him
commented
Status: Active » Fixed
StatusFileSize
new ooyala_separate_taxonomy.patch74.91 KB

This was actually very easy to accomplish. I'm glad to see all of our Taxonomy code centralized with this new approach, the new way we form_alter() in the taxonomy settings to the Ooyala form also make a lot more sense than the previous approach. I've tested this out and it works great. Because we've only had taxonomy syncing for 2 days but this renames a table, I didn't bother with an upgrade path to rename the table. At the same time though, this module will install a new table ("ooyala_taxonomy" now instead of "ooyala_term") and build the new list of terms as needed, so it's not going to cause anyone any trouble anyway.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.