Allow passing multiple pattern databases as an array to the new `patt…

…ern_paths` argument

- Support `patterns_path` argument but deprecate `pattern_path` attribute accessor
- Add new `patterns_paths` array argument to enable loading multiple patterns files

Fixes #11

README.md

@@ -98,6 +98,11 @@ or when instantiating a `UserAgentParser::Parser`:
 UserAgentParser::Parser.new(patterns_path: '/some/path/to/regexes.yaml').parse(ua_string)
 ```
 
+It is also possible to specify multiple patterns paths if you want to add some custom patterns without forking the main database:
+```ruby
+UserAgentParser::Parser.new(patterns_paths: [UserAgentParser::DefaultPatternsPath, '/some/path/to/regexes.yaml'])
+```
+
 ## Command line tool
 
 The gem incldes a `user_agent_parser` bin command which will read from

lib/user_agent_parser/parser.rb

@@ -4,6 +4,8 @@
 
 module UserAgentParser
   class Parser
+    extend Gem::Deprecate
+
     FAMILY_REPLACEMENT_KEYS = %w[
       family_replacement
       v1_replacement
@@ -22,11 +24,15 @@ class Parser
 
     private_constant :FAMILY_REPLACEMENT_KEYS, :OS_REPLACEMENT_KEYS
 
-    attr_reader :patterns_path
+    attr_reader :patterns_paths
+
+    def initialize(patterns_path: nil, patterns_paths: [])
+      @patterns_paths = [patterns_path, *patterns_paths].compact
+      if @patterns_paths.empty?
+        @patterns_paths = [UserAgentParser::DefaultPatternsPath]
+      end
 
-    def initialize(patterns_path: nil)
-      @patterns_path = patterns_path || UserAgentParser::DefaultPatternsPath
-      @ua_patterns, @os_patterns, @device_patterns = load_patterns(@patterns_path)
+      @ua_patterns, @os_patterns, @device_patterns = load_patterns(@patterns_paths)
     end
 
     def parse(user_agent)
@@ -35,9 +41,24 @@ def parse(user_agent)
       parse_ua(user_agent, os, device)
     end
 
+    def patterns_path
+      patterns_paths.first
+    end
+    deprecate :patterns_path, :patterns_paths, 2020, 12
+
     private
 
-    def load_patterns(path)
+    def load_patterns(patterns_paths)
+      patterns_paths.inject([[], [], []]) do |patterns, path|
+        ua_patterns, os_patterns, device_patterns = load_patterns_file(path)
+        patterns[0] += ua_patterns
+        patterns[1] += os_patterns
+        patterns[2] += device_patterns
+        patterns
+      end
+    end
+
+    def load_patterns_file(path)
       yml = YAML.load_file(path)
 
       # Parse all the regexs

spec/custom_regexes.yaml

@@ -1,17 +1,17 @@
 user_agent_parsers:
-  - regex: '.*'
+  - regex: 'Any.*'
     family_replacement: 'Custom browser'
     v1_replacement: '1'
     v2_replacement: '2'
     v3_replacement: '3'
     v4_replacement: '4'
 
 os_parsers:
-  - regex: '.*'
+  - regex: 'Any.*'
     os_replacement: 'Custom OS'
     os_v1_replacement: '1'
     os_v2_replacement: '2'
 
 device_parsers:
-  - regex: '.*'
+  - regex: 'Any.*'
     device_replacement: 'Custom device'

spec/other_regexes.yaml

@@ -0,0 +1,17 @@
+user_agent_parsers:
+  - regex: 'Other.*'
+    family_replacement: 'Other browser'
+    v1_replacement: '1'
+    v2_replacement: '2'
+    v3_replacement: '3'
+    v4_replacement: '4'
+
+os_parsers:
+  - regex: 'Other.*'
+    os_replacement: 'Other OS'
+    os_v1_replacement: '1'
+    os_v2_replacement: '2'
+
+device_parsers:
+  - regex: 'Other.*'
+    device_replacement: 'Other device'

spec/parser_spec.rb

@@ -57,6 +57,10 @@ def custom_patterns_path
     File.join(File.dirname(__FILE__), 'custom_regexes.yaml')
   end
 
+  def other_patterns_path
+    File.join(File.dirname(__FILE__), 'other_regexes.yaml')
+  end
+
   describe '::parse' do
     it 'parses a UA' do
       ua = UserAgentParser.parse('Mozilla/5.0 (Macintosh; U; PPC Mac OS X; en-us) AppleWebKit/418.8 (KHTML, like Gecko) Safari/419.3')
@@ -69,7 +73,7 @@ def custom_patterns_path
   end
 
   describe '#initialize with a custom patterns path' do
-    it 'uses the custom patterns' do
+    it 'accepts a single pattern_path' do
       parser = UserAgentParser::Parser.new(patterns_path: custom_patterns_path)
       ua = parser.parse('Any user agent string')
 
@@ -85,6 +89,16 @@ def custom_patterns_path
 
       _(ua.device.family).must_equal('Custom device')
     end
+
+    it 'accepts pattern_paths array' do
+      parser = UserAgentParser::Parser.new(patterns_paths: [custom_patterns_path, other_patterns_path])
+
+      ua = parser.parse('Any user agent string')
+      oua = parser.parse('Other user agent string')
+
+      _(ua.family).must_equal('Custom browser')
+      _(oua.family).must_equal('Other browser')
+    end
   end
 
   describe '#parse' do