neither the fullscreen or autostart options take any effect. Attached patch seems to fix this.

diff -Naur ../flvmediaplayer/players/jw_player_4.x.inc ./players/jw_player_4.x.inc
--- ../flvmediaplayer/players/jw_player_4.x.inc 2009-05-01 17:55:59.000000000 +0100
+++ ./players/jw_player_4.x.inc 2009-05-01 17:55:55.000000000 +0100
@@ -307,15 +307,15 @@
     '#collapsed' => true,
   );

-  $form['playback']['param_fullscreen'] = array(
+  $form['playback']['param_allowfullscreen'] = array(
     '#title' => t('Allow fullscreen'),
     '#type' => 'checkbox',
     '#return_value' => 'true',
-    '#default_value' => $config['param_fullscreen'],
+    '#default_value' => $config['param_allowfullscreen'],
     '#description' => t('Enable fullscreen display'),
   );
-
-  $form['playback']['flvmp_autostart'] = array(
+  
+  $form['playback']['flashvar_autostart'] = array(
     '#title' => t('Autostart'),
     '#type' => 'checkbox',
     '#default_value' => $config['flashvar_autostart'],
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mrfelton’s picture

UPDATE:
The paramater 'allowscriptaccess' must be set to 'always' not 'true' in order to work in all browsers. When set to 'true', fullscreen does not work in IE. This updated patch also changes the paramater value for allowscriptaccess to 'always'

mrfelton’s picture

Status: Active » Needs review
arthurf’s picture

Thanks, these are committed to the dev branch

arthurf’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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

mrfelton’s picture

Status: Closed (fixed) » Active

You did not apply the updated patch, and so allowscriptaccess is still set to 'true', which is incorrect - it should be 'always' as per my patch at #1

mrfelton’s picture

Version: 6.x-1.0-beta3 » 6.x-1.x-dev
FileSize
974 bytes

Actually, it seems that you have muddled the values for fullscreen and autostart. Please apply this patch to correct.