Closed (outdated)
Project:
oEmbed
Version:
7.x-1.0-rc2
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
23 Apr 2012 at 20:55 UTC
Updated:
20 Oct 2025 at 09:14 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
Rob_Feature commentedA correction...features does output a file but it's just called featurename_..inc (nothing between the periods)
Comment #2
Anonymous (not verified) commentedRob, I suspect that the problem is the same as #1330140: CTools export fails with Features. Can you give that a shot?
When I worked on the module last year in D6, I ported it to use ctools export_ui. I also added the api version info to the schema, but this was rejected because it was going to break existing data. See http://drupal.org/node/892988#comment-3493144
Can you prepare a patch that adds the necessary info to the schema? It's another one of those things that keeps this module from an official release.
Comment #3
Rob_Feature commentedSure I'll give it a try..not sure I understand the schema well enough to actually 'fix it', but I'll take a look.
Comment #4
Anonymous (not verified) commentedCool. So the "schema" is SQL database tables are understood by Drupal. ctools exportables add some extra values in the same schema to tell ctools how to export that table. You probably just need to add the 'api' details that were in the other issue you linked to above.
If you get stuck, I can make a patch but then you need to test it. :-)
Comment #5
Rob_Feature commentedTesting I can do :) But I'll also try and patch if you don't get around to it soon.
Comment #6
Rob_Feature commentedTook a look at this today and it's over my head...sorry. I'd be happy to test it immediately if you post a patch.
Comment #7
Cap'taine Crochet commentedI've just done what you said and i'm now able to export a provider preset into a feature, and the hook_ctools_plugin_api looks right. I've attached the update, for you to reproduce faster. Thanks!
Comment #8
SilviaT commentedThe patch worked for me on d7.15!
Comment #9
stuart.crouch commentedWhats the current status on this bug? Is the patch what was asked for and it needs testing now, or the work still needs doing? I just tried to use features to apply our oEmbed providers across several drupal sites, but came across the same issue (features.inc file has the code above and the oembed file is just featurename..inc
Happy to pick this up if more work is needed, but cant understand *what* is needed at the moment.
Comment #10
Anonymous (not verified) commentedIf you're using the latest development version of oEmbed project, you should be able to export your provider configuration as ctools exportables in Features.
Comment #11
stuart.crouch commentedOK. I'll try it out. What else do you need to get a stable version of this module out of the door? Sorry, I'm not very good at reading drupal's bug tracker in order to understand each modules roadmap.
Comment #12
stuart.crouch commentedOK. I grabbed the latest ctools and the media module (btw we started using oembed to get away from the media module, but never mind)
I deleted all my features and went to create a new one, and recieved this
Fatal error: Call to undefined function file_type_get_enabled_types() in /opt/drupal/sites/open.ac.uk.d7test2/modules/oembed/media_oembed/media_oembed.file_default_displays.inc on line 14
Call Stack:
0.0001 643064 1. {main}() /opt/drupal/index.php:0
0.1587 36751368 2. menu_execute_active_handler() /opt/drupal/index.php:21
0.1604 37166392 3. call_user_func_array() /opt/drupal/includes/menu.inc:516
0.1604 37166728 4. drupal_get_form() /opt/drupal/includes/menu.inc:516
0.1604 37167576 5. drupal_build_form() /opt/drupal/includes/form.inc:131
0.1604 37170320 6. drupal_retrieve_form() /opt/drupal/includes/form.inc:339
0.1604 37171792 7. call_user_func_array() /opt/drupal/includes/form.inc:795
0.1604 37172216 8. features_export_form() /opt/drupal/includes/form.inc:795
0.7891 52120896 9. features_invoke() /opt/drupal/sites/open.ac.uk.d7test2/modules/features/features.admin.inc:93
0.7891 52121352 10. call_user_func_array() /opt/drupal/sites/open.ac.uk.d7test2/modules/features/features.module:516
0.7891 52121688 11. file_display_features_export_options() /opt/drupal/sites/open.ac.uk.d7test2/modules/features/features.module:516
0.7891 52121768 12. ctools_component_features_export_options() /opt/drupal/sites/open.ac.uk.d7test2/modules/features/includes/features.ctools.inc(26) : eval()'d code:1
0.7891 52121984 13. _ctools_features_export_default_list() /opt/drupal/sites/open.ac.uk.d7test2/modules/features/includes/features.ctools.inc:133
0.7892 52121984 14. ctools_export_default_list() /opt/drupal/sites/open.ac.uk.d7test2/modules/features/includes/features.ctools.inc:306
0.7892 52122200 15. ctools_export_crud_load_all() /opt/drupal/sites/open.ac.uk.d7test2/modules/ctools/includes/export.inc:1232
0.7892 52122328 16. ctools_export_load_object() /opt/drupal/sites/open.ac.uk.d7test2/modules/ctools/includes/export.inc:148
0.7900 52133144 17. _ctools_export_get_defaults() /opt/drupal/sites/open.ac.uk.d7test2/modules/ctools/includes/export.inc:496
0.7904 52151872 18. media_oembed_file_default_displays() /opt/drupal/sites/open.ac.uk.d7test2/modules/ctools/includes/export.inc:680
Comment #13
Anonymous (not verified) commentedOne of the issues to resolve is the dependencies, but you need to upgrade File Entity to the latest unstable release. 7.x-2.x-unstable7 I think. Media should be updated, but it's actually not used, so you can probably hack around disabling it in the meantime.
Comment #14
humansky commentedTested patch #7 works with features 2.x.
Comment #15
humansky commentedSwitching back to active, I found another bug. After I added the patch in #7, I noticed that the other oEmbed providers disappear. Is this happening to anyone else?
Comment #16
sheldonkreger commentedMost of my providers don't export via
drush fe my_feature oembed_provider:provider_name -y
It will say the feature has been updated but no components will actually be added. They remain overridden on admin/config/media/oembed/provider/default
Sad panda.
Comment #17
thedavidmeister commentedAs to #15, oembedcore simply needs to update its default export hook, and implement hook_ctools_plugin_api(), for the defaults to reappear, like:
Patch in #7 totally works for exporting new, custom providers.
Comment #18
seantwalshApplied #7 and #15 and they resolved both issues.
Comment #19
thedavidmeister commentedThis appears to have been resolved in the latest dev snapshot, but the developers did not leave a note here?
Comment #20
sheldonkreger commentedIf it's fixed, it's fixed. :-)
On the other hand . . . if you didn't fix it, it isn't fixed. :-)
Can somebody confirm the fix and link to a commit?
Comment #21
thedavidmeister commentedIt's resolved in the latest development version because the architecture of the relevant part of the code is totally different, so the changes I was suggesting earlier don't even make sense in the new setup.
There's no exact commit I could point to and say "that's where it was fixed" because the system is just different in general, which is not that surprising when there's a 2 year gap between "latest stable" and "development snapshot".
I'm guessing a new stable version should be rolled, but that's not in-scope for this issue.
Comment #22
humansky commentedI'm uploading comment #17 in patch form. However, like @thedavidmeister said....do not use the 0.x version of this module. The latest version solves all this nonsense. I just need the patch file for a drush make file that requires the older version of this module.
Comment #23
astonvictor commentedD7 reached its EOL back in January 2025, and there is no active release for D7 for this module anymore.
Development or support is not planned for D7. All D7-related issues are marked as outdated in a bunch.