Fixed menu item title
Added autoplay feature
Fixed incorrect values on settings page

---

On a side note I'd highly recommend using your own variable storage for these settings rather than flooding the variable table. But that's for another day.

CommentFileSizeAuthor
patch_commit_cabd6ce18571.patch3.87 KBjdelaune
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jec006’s picture

Status: Needs review » Fixed

Committed here: http://drupalcode.org/project/video_embed_field.git/commit/2e5c09c

In re the variables thing:

My understanding is that the standard way to store settings in the variable table to allow features (via strongarm) to be able to store and override them. Drupal also will handle all the cacheing of them at this point, saving some development time. Check out other modules (an example I have right now is colorbox: http://drupalcode.org/project/colorbox.git/blob/HEAD:/colorbox.admin.inc).

Status: Fixed » Closed (fixed)

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