Rename parameter to a more appropriate one

Signed-off-by: Ravi <jravi248@gmail.com>

Author: JRavi2

src/extractcode/archive.py

@@ -104,14 +104,14 @@ def can_extract(location):
         return True
 
 
-def should_extract(location, kinds, ignored_extensions=()):
+def should_extract(location, kinds, ignore_pattern=()):
     """
     Return True if this location should be extracted based on the provided
     kinds
     """
     location = os.path.abspath(os.path.expanduser(location))
-    ignored_extensions = {extension : 'User ignore: Supplied by --ignore' for extension in ignored_extensions}
-    should_ignore = is_ignored(location, ignored_extensions)
+    ignore_pattern = {extension : 'User ignore: Supplied by --ignore' for extension in ignore_pattern}
+    should_ignore = is_ignored(location, ignore_pattern)
     if get_extractor(location, kinds) and not should_ignore:
         return True
 

src/extractcode/extract.py

@@ -97,7 +97,7 @@
 ExtractEvent = namedtuple('ExtractEvent', 'source target done warnings errors')
 
 
-def extract(location, kinds=extractcode.default_kinds, recurse=False, replace_originals=False, ignored_extensions=()):
+def extract(location, kinds=extractcode.default_kinds, recurse=False, replace_originals=False, ignore_pattern=()):
     """
     Walk and extract any archives found at `location` (either a file or
     directory). Extract only archives of a kind listed in the `kinds` kind tuple.
@@ -125,7 +125,7 @@ def extract(location, kinds=extractcode.default_kinds, recurse=False, replace_or
     """
     processed_events = []
     processed_events_append = processed_events.append
-    for event in extract_files(location, kinds, recurse, ignored_extensions):
+    for event in extract_files(location, kinds, recurse, ignore_pattern):
         yield event
         if replace_originals:
             processed_events_append(event)
@@ -143,7 +143,7 @@ def extract(location, kinds=extractcode.default_kinds, recurse=False, replace_or
                 fileutils.delete(target)
 
 
-def extract_files(location, kinds=extractcode.default_kinds, recurse=False, ignored_extensions=()):
+def extract_files(location, kinds=extractcode.default_kinds, recurse=False, ignore_pattern=()):
     """
     Extract the files found at `location`.
 
@@ -177,7 +177,7 @@ def extract_files(location, kinds=extractcode.default_kinds, recurse=False, igno
                     logger.debug('extract:walk not recurse: skipped  file: %(loc)r' % locals())
                 continue
 
-            if not archive.should_extract(loc, kinds, ignored_extensions):
+            if not archive.should_extract(loc, kinds, ignore_pattern):
                 if TRACE:
                     logger.debug('extract:walk: skipped file: not should_extract: %(loc)r' % locals())
                 continue
@@ -193,7 +193,7 @@ def extract_files(location, kinds=extractcode.default_kinds, recurse=False, igno
             if recurse:
                 if TRACE:
                     logger.debug('extract:walk: recursing on target: %(target)r' % locals())
-                for xevent in extract(location=target, kinds=kinds, recurse=recurse, ignored_extensions=ignored_extensions):
+                for xevent in extract(location=target, kinds=kinds, recurse=recurse, ignore_pattern=ignore_pattern):
                     if TRACE:
                         logger.debug('extract:walk:recurse:extraction event: %(xevent)r' % locals())
                     yield xevent

tests/extractcode/test_extract.py

@@ -1131,7 +1131,7 @@ def test_extract_ignore_file(self):
             'gamma/gamma.zip-extract/c.txt'
         ]
         from extractcode import default_kinds
-        result = list(extract.extract(test_dir, recurse=True, ignored_extensions=('alpha.zip',)))
+        result = list(extract.extract(test_dir, recurse=True, ignore_pattern=('alpha.zip',)))
         check_no_error(result)
         check_files(test_dir, expected)
 
@@ -1151,7 +1151,7 @@ def test_extract_ignore_directory(self):
             'gamma/gamma.zip',
         ]
         from extractcode import default_kinds
-        result = list(extract.extract(test_dir, recurse=True, ignored_extensions=('gamma',)))
+        result = list(extract.extract(test_dir, recurse=True, ignore_pattern=('gamma',)))
         check_no_error(result)
         check_files(test_dir, expected)
 
@@ -1171,6 +1171,6 @@ def test_extract_ignore_pattern(self):
             'gamma/gamma.zip-extract/c.txt'
         ]
         from extractcode import default_kinds
-        result = list(extract.extract(test_dir, recurse=True, ignored_extensions=('b*.zip',)))
+        result = list(extract.extract(test_dir, recurse=True, ignore_pattern=('b*.zip',)))
         check_no_error(result)
         check_files(test_dir, expected)