Change some error wording.

Signed-off-by: Chin Yeung Li <[email protected]>

Author: chinyeungli

src/attributecode/cmd.py

@@ -368,6 +368,7 @@ def attrib(input, output, api_url, api_key, scancode, min_license_score, referen
 
     rendered = ''
     license_dict = {}
+    errors = []
 
     if not quiet:
         print_version()
@@ -392,7 +393,7 @@ def attrib(input, output, api_url, api_key, scancode, min_license_score, referen
             click.echo(msg)
             sys.exit(1)
 
-    if input.endswith('.json') or input.endswith('.csv'):
+    if input.endswith('.json') or input.endswith('.csv') or input.endswith('.xlsx'):
         is_about_input = False
         from_attrib = True
         if not reference:
@@ -409,8 +410,12 @@ def attrib(input, output, api_url, api_key, scancode, min_license_score, referen
         errors, abouts = collect_inventory(input)
 
     if not abouts:
-        msg = 'No ABOUT file or reference is found from the input. Attribution generation halted.'
-        click.echo(msg)
+        if errors:
+            errors = unique(errors)
+            errors_count = report_errors(errors, quiet, verbose, log_file_loc=output + '-error.log')
+        else:
+            msg = 'No ABOUT file or reference is found from the input. Attribution generation halted.'
+            click.echo(msg)
         sys.exit(1)
 
     if not is_about_input:

src/attributecode/model.py

@@ -717,7 +717,7 @@ def validate_fields(fields, about_file_path, running_inventory, base_dir,
 def validate_field_name(name):
     if not is_valid_name(name):
         msg = ('Field name: %(name)r contains illegal name characters: '
-               '0 to 9, a to z, A to Z and _. (or empty spaces)')
+               '0 to 9, a to z, A to Z and _. (or empty spaces) and is ignored.')
         return Error(CRITICAL, msg % locals())
 
 

tests/test_model.py

@@ -581,7 +581,7 @@ def test_About_rejects_non_ascii_names_and_accepts_unicode_values(self):
         test_file = get_test_loc('test_model/parse/non_ascii_field_name_value.about')
         a = model.About(test_file)
         expected = [
-            Error(CRITICAL, "Field name: 'mat\xedas' contains illegal name characters: 0 to 9, a to z, A to Z and _. (or empty spaces)")
+            Error(CRITICAL, "Field name: 'mat\xedas' contains illegal name characters: 0 to 9, a to z, A to Z and _. (or empty spaces) and is ignored.")
         ]
         assert expected == a.errors