I noticed that the module was not accepting a no_flash value as a parameter. It would always show the baseline value. Minor bug in the code where the base_params was being used for that value rather than the merged array. Simple attached patch corrects the problem.

CommentFileSizeAuthor
swfobjectapi-noflash-patch.txt703 byteslevelos
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

smk-ka’s picture

smk-ka CreditAttribution: smk-ka commented
Status: Needs review » Reviewed & tested by the community

+1 from me. This patch is working fine.

arthurf’s picture

arthurf CreditAttribution: arthurf commented
Status: Reviewed & tested by the community » Fixed

thanks for the patch, it went into the latest release, along with some cache fixes. Also patched the 2.0 release as well

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Fixed » Closed (fixed)

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