Hey all,

is there any way i can get the option loop to work??
if not, is there any way i can help with getting it to work

thanks in advance

CommentFileSizeAuthor
#4 add-repeat-option-issue-#1330634.patch1.28 KBrreiss

Comments

rickvug’s picture

rickvug commented
Title: Loop » Support loop option in presets
Category: support » feature

This is a good feature request. Certainly something I'd like to see in the module but not at the top of my list. Patches welcome. :)

Anonymous’s picture

Anonymous (not verified) commented

I would be interested in trying this, but I can't seem to find the source code. I followed the instructions to use git to make a clone of the master, and everything looks like it worked, but I cannot find any source code in the directory structure I cloned. I must be missing something pretty basic, but would sure like to know what it is. Any help is appreciated.

korzh-nick’s picture

korzh-nick commented

Here's a way to make a movie defltny repetitive. But how to make it in the settings I do not know

diff --git a/sites/all/modules/jw_player/jw_player.module b/sites/all/modules/jw_player/jw_player.module
index 0e15952..89a9bfa 100755
--- a/sites/all/modules/jw_player/jw_player.module
+++ b/sites/all/modules/jw_player/jw_player.module
@@ -253,6 +253,7 @@ function jw_player_default_settings() {
       'height' => '480',
       'autoplay' => FALSE,
       'controlbar' => 'bottom',
+      'repeat' => 'always',
     );
   }
 
@@ -316,7 +317,7 @@ function jw_player_preprocess_jw_player(&$variables) {
   // Copy player variables into their own array to be set as JavaScript
   // configuration.
   // @todo Bad smell here. Refactoring needed.
-  $player_variables = array('width', 'height', 'controlbar', 'playlist.position', 'playlist.size', 'skin', 'autoplay');
+  $player_variables = array('width', 'height', 'controlbar', 'playlist.position', 'playlist.size', 'skin', 'repeat', 'autoplay');
   foreach ($player_variables as $key) {
     if (!empty($variables[$key])) {
       $variables['config'][$key] = $variables[$key];
rreiss’s picture

rreiss commented
Version: 7.x-1.x-dev » 7.x-2.x-dev
Issue summary: View changes
Status: Active » Needs review
StatusFileSize
new add-repeat-option-issue-#1330634.patch1.28 KB

I've added the feature to V2.x-dev .
Please review and commit.

Thanks.

Status: Needs review » Needs work

The last submitted patch, 4: add-repeat-option-issue-#1330634.patch, failed testing.

sgdev’s picture

sgdev commented
Status: Needs work » Closed (duplicate)
Related issues: +#2713725: Refactor theming

This is going to be resolved once the refactor theming patch is committed.

https://www.drupal.org/node/2713725