Issue 5729 - remove_redundant_translations fails due to the presence of system file

translations.py

 #!/usr/bin/env python
 
 import filecmp
 import os
 import re
 import shutil
 import sys
 import urllib2
 from lxml import etree
 from StringIO import StringIO
@@ skipping 63 matching lines @@
             language.findtext('code'): language.findtext('translated_progress')
         })
     return languages
 
 def remove_redundant_translations():
     default_locale_path = os.path.join(LOCALES_PATH, DEFAULT_LOCALE)
     for locale in os.listdir(LOCALES_PATH):
         if locale == DEFAULT_LOCALE:
             continue
         locale_path = os.path.join(LOCALES_PATH, locale)
+        # ignore files
+        if os.path.isfile(locale_path):
+            continue;

[Vasily Kuznetsov, 2017/09/22 16:27:10]

You don't need the semicolon at the end of the line.

[jens, 2017/09/26 12:09:17]

Ah, you're right. I know it's not necessary in python but it's just a habit from
doing Java all the time.

+        # ignore hidden folders
+        if locale_path.startswith('.'):
+            continue;
         if not filecmp.dircmp(locale_path, default_locale_path).diff_files:
             shutil.rmtree(locale_path)
 
 if __name__ == "__main__":
     if len(sys.argv) < 3:
         print_usage()
         sys.exit(1)
 
     command, api_key = sys.argv[1:]
     if command != "get":
         print_usage()
         sys.exit(2)
 
     get_translations(api_key)
     remove_redundant_translations()