Add config option for timestamps, fixes #18

Author: MGatner

examples/Assets.php

@@ -28,6 +28,9 @@ class Assets extends \Tatter\Assets\Config\Assets
 	// Starting directory for manifest publication
 	public $publishBase = ROOTPATH . 'vendor/';
 
+	// Whether to append file modification timestamps on asset tags
+	public $useTimestamps = true;
+	
 	// Additional paths to load per route
 	// Relative to fileBase, no leading/trailing slashes
 	public $routes = [

src/Config/Assets.php

@@ -19,6 +19,9 @@ class Assets extends BaseConfig
 	// Starting directory for manifest publication
 	public $publishBase = ROOTPATH . 'vendor/';
 
+	// Whether to append file modification timestamps on asset tags
+	public $useTimestamps = true;
+	
 	// Additional paths to load per route
 	// Relative to fileBase, no leading/trailing slashes
 	public $routes = [];

src/Libraries/Assets.php

@@ -186,7 +186,7 @@ public function tag(string $path)
 
 		// Check for the actual file for version control
 		$file = $this->config->fileBase . $path;
-		if (is_file($file))
+		if (is_file($file) && $this->config->useTimestamps)
 		{
 			$url .=  '?v=' . filemtime($file);
 		}

tests/assets/LibraryTest.php

@@ -0,0 +1,27 @@
+<?php
+
+use CodeIgniter\Config\Services;
+use Tatter\Assets\Libraries\Assets;
+use Tests\Support\AssetsTestCase;
+
+class LibraryTest extends AssetsTestCase
+{
+	public function testTagAddsTimestamps()
+	{
+		$file   = 'alert.js';
+		$result = $this->assets->tag($file);
+
+		$this->assertEquals('<script src="http://example.com/assets/alert.js?v=1594481279" type="text/javascript"></script>', $result);
+	}
+
+	public function testTagRespectsConfigUseTimestamps()
+	{
+		$this->config->useTimestamps = false;
+
+		$file   = 'alert.js';
+		$assets = new Assets($this->config);
+		$result = $assets->tag($file);
+
+		$this->assertEquals('<script src="http://example.com/assets/alert.js" type="text/javascript"></script>', $result);
+	}
+}