Implemented custom asset versioning to make creating releases easier

diff --git a/app/helpers.php b/app/helpers.php
new file mode 100644 (file)
index 0000000..f25a8f7
--- /dev/null
+++ b/app/helpers.php
@@ -0,0 +1,30 @@
+<?php
+
+if (! function_exists('versioned_asset')) {
+    /**
+     * Get the path to a versioned file.
+     *
+     * @param  string  $file
+     * @return string
+     *
+     * @throws \InvalidArgumentException
+     */
+    function versioned_asset($file)
+    {
+        static $manifest = null;
+
+        if (is_null($manifest)) {
+            $manifest = json_decode(file_get_contents(public_path('build/manifest.json')), true);
+        }
+
+        if (isset($manifest[$file])) {
+            return '/' . $manifest[$file];
+        }
+
+        if (file_exists(public_path($file))) {
+            return '/' . $file;
+        }
+
+        throw new InvalidArgumentException("File {$file} not defined in asset manifest.");
+    }
+}
\ No newline at end of file

diff --git a/composer.json b/composer.json
index 7865be636d416dae02c45ba740015faa671e12cc..748ff68f786fa749c2e824d2f73a8e2a8230e0f2 100644 (file)
--- a/composer.json
+++ b/composer.json
@@ -25,7 +25,10 @@
         ],
         "psr-4": {
             "BookStack\\": "app/"
         ],
         "psr-4": {
             "BookStack\\": "app/"

-        }
+        },
+        "files": [
+            "app/helpers.php"
+        ]

     },
     "autoload-dev": {
         "classmap": [
     },
     "autoload-dev": {
         "classmap": [

diff --git a/gulpfile.js b/gulpfile.js
index 621e665bede0b129c1777e0954c09cf5d369141f..4396187395b41d01f72206bd534c9b7935b4f2e8 100644 (file)
--- a/gulpfile.js
+++ b/gulpfile.js
@@ -1,8 +1,26 @@
 var elixir = require('laravel-elixir');
 
 var elixir = require('laravel-elixir');
 

+// Custom extensions
+var gulp = require('gulp');
+var Task = elixir.Task;
+var fs = require('fs');
+
+elixir.extend('queryVersion', function(inputFiles) {
+     new Task('queryVersion', function() {
+         var manifestObject = {};
+         var uidString = Date.now().toString(16).slice(4);
+         for (var i = 0; i < inputFiles.length; i++) {
+             var file = inputFiles[i];
+             manifestObject[file] = file + '?version=' + uidString;
+         }
+         var fileContents = JSON.stringify(manifestObject, null, 1);
+         fs.writeFileSync('public/build/manifest.json', fileContents);
+     }).watch(['./public/css/*.css', './public/js/*.js']);
+});
+

 elixir(function(mix) {
     mix.sass('styles.scss')
         .sass('print-styles.scss')
         .browserify(['jquery-extensions.js', 'global.js'], 'public/js/common.js')
 elixir(function(mix) {
     mix.sass('styles.scss')
         .sass('print-styles.scss')
         .browserify(['jquery-extensions.js', 'global.js'], 'public/js/common.js')

-        .version(['css/styles.css', 'css/print-styles.css', 'js/common.js']);
+        .queryVersion(['css/styles.css', 'css/print-styles.css', 'js/common.js']);

 });
 });

diff --git a/package.json b/package.json
index af2cbae588ea01b4a65669c406f23a3401764ea3..e33ad170e62065f784e1007b79934301b9020ac5 100644 (file)
--- a/package.json
+++ b/package.json
@@ -1,7 +1,7 @@
 {
   "private": true,
   "devDependencies": {
 {
   "private": true,
   "devDependencies": {

-    "gulp": "^3.8.8",
+    "gulp": "^3.9.0",

     "insert-css": "^0.2.0"
   },
   "dependencies": {
     "insert-css": "^0.2.0"
   },
   "dependencies": {

diff --git a/resources/views/base.blade.php b/resources/views/base.blade.php
index 9c3ddbf91a2c0cdf06c5efb54c4cf45de99dcafd..553c634cbeee4ee6097f403f918a59d1161a1d6e 100644 (file)
--- a/resources/views/base.blade.php
+++ b/resources/views/base.blade.php
@@ -9,8 +9,8 @@
     <meta charset="utf-8">
 
     <!-- Styles and Fonts -->
     <meta charset="utf-8">
 
     <!-- Styles and Fonts -->

-    <link rel="stylesheet" href="{{ elixir('css/styles.css') }}">
-    <link rel="stylesheet" media="print" href="{{ elixir('css/print-styles.css') }}">
+    <link rel="stylesheet" href="{{ versioned_asset('css/styles.css') }}">
+    <link rel="stylesheet" media="print" href="{{ versioned_asset('css/print-styles.css') }}">

     <link href='//fonts.googleapis.com/css?family=Roboto:400,400italic,500,500italic,700,700italic,300italic,100,300' rel='stylesheet' type='text/css'>
     <link rel="stylesheet" href="/libs/material-design-iconic-font/css/material-design-iconic-font.min.css">
 
     <link href='//fonts.googleapis.com/css?family=Roboto:400,400italic,500,500italic,700,700italic,300italic,100,300' rel='stylesheet' type='text/css'>
     <link rel="stylesheet" href="/libs/material-design-iconic-font/css/material-design-iconic-font.min.css">
 


@@ -79,6 +79,6 @@
     </section>
 
 @yield('bottom')
     </section>
 
 @yield('bottom')

-<script src="{{ elixir('js/common.js') }}"></script>
+<script src="{{ versioned_asset('js/common.js') }}"></script>

 </body>
 </html>
 </body>
 </html>