Move seek js file [#2719851]

We're going to remove a couple of items in the refactor theming patch into their own patches.

This one is regarding the seek functionality. The patch moves jw_player_seek.js into the new js subdirectory, and renames the file as "jw_player.seek.js" to be consistent with similar files.

I will post the patch in a few minutes.

Comment	File	Size	Author
#2	jw_player-move_seek_js_file-2719851-2.patch	2.13 KB	ron_s
#2

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

6 May 2016 at 03:20

ron_s created an issue. See original summary.

Comment #2

ron_s CreditAttribution: ron_s commented 6 May 2016 at 03:22

Status:

Active

» Needs review

File	Size
jw_player-move_seek_js_file-2719851-2.patch	2.13 KB

See attached, thanks.

Comment #3

Berdir

German

Switzerland

CreditAttribution: Berdir at MD Systems GmbH commented 10 May 2016 at 06:20

Version:	7.x-2.x-dev	» 8.x-1.x-dev
Status:	Needs review	» Patch (to be ported)

Patch is small enough, but note that your git client is still not properly set up for renames. This is what I see when I do a git diff:

diff --git a/jw_player_seek.js b/js/jw_player.seek.js
similarity index 100%
rename from jw_player_seek.js
rename to js/jw_player.seek.js
diff --git a/jw_player.module b/jw_player.module
index 946d345..2ade65b 100644
--- a/jw_player.module
+++ b/jw_player.module
@@ -820,7 +820,7 @@ function jw_player_libraries_info() {
       ),
       'integration files' => array(
         'jw_player' => array(
-          'js' => array('jw_player_seek.js')
+          'js' => array('js/jw_player.seek.js')
         ),
       ),
       'callbacks' => array(