Ran PEP8 and use os.path.splitext to detect language

Ran PEP8 to make everything look pretty.

Changed from explicitly replacing the .tmLanguage extension to using os.path.splitext to get the filename without the extension so that it works with the new sublime-syntax files.

Replaced `from xml.dom.minidom import *` with `from xml.dom.minidom import parseString`, since that's the only function used from that module.

Author: bheklilr

indentxml.py

@@ -2,30 +2,32 @@
 import sublime_plugin
 import re
 import json
-from xml.dom.minidom import *
-from os.path import basename
+from xml.dom.minidom import parseString
+from os.path import basename, splitext
 
 
 class BaseIndentCommand(sublime_plugin.TextCommand):
+
     def __init__(self, view):
         self.view = view
         self.language = self.get_language()
 
-    def get_language(self):        
+    def get_language(self):
         syntax = self.view.settings().get('syntax')
-        language = basename(syntax).replace('.tmLanguage', '').lower() if syntax != None else "plain text"
+        language = splitext(basename(syntax))[0].lower() if syntax is not None else "plain text"
         return language
 
     def check_enabled(self, lang):
         return True
 
     def is_enabled(self):
         """
-        Enables or disables the 'indent' command.
-        Command will be disabled if there are currently no text selections and current file is not 'XML' or 'Plain Text'.
-        This helps clarify to the user about when the command can be executed, especially useful for UI controls.
+        Enables or disables the 'indent' command. Command will be disabled if
+        there are currently no text selections and current file is not 'XML' or
+        'Plain Text'. This helps clarify to the user about when the command can
+        be executed, especially useful for UI controls.
         """
-        if self.view == None:
+        if self.view is None:
             return False
 
         return self.check_enabled(self.get_language())
@@ -38,21 +40,22 @@ def run(self, edit):
         regions = view.sel()
         # if there are more than 1 region or region one and it's not empty
         if len(regions) > 1 or not regions[0].empty():
-                for region in view.sel():
-                    if not region.empty():
-                        s = view.substr(region).strip()
-                        s = self.indent(s)
-                        view.replace(edit, region, s)
-        else:   #format all text
-                alltextreg = sublime.Region(0, view.size())
-                s = view.substr(alltextreg).strip()
-                s = self.indent(s)
-                view.replace(edit, alltextreg, s)
+            for region in view.sel():
+                if not region.empty():
+                    s = view.substr(region).strip()
+                    s = self.indent(s)
+                    view.replace(edit, region, s)
+        else:  # format all text
+            alltextreg = sublime.Region(0, view.size())
+            s = view.substr(alltextreg).strip()
+            s = self.indent(s)
+            view.replace(edit, alltextreg, s)
 
 
 class AutoIndentCommand(BaseIndentCommand):
+
     def get_text_type(self, s):
-        language =  self.language 
+        language = self.language
         if language == 'xml':
             return 'xml'
         if language == 'json':
@@ -73,22 +76,23 @@ def indent(self, s):
             command = IndentJsonCommand(self.view)
         if text_type == 'notsupported':
             return s
-        
+
         return command.indent(s)
 
     def check_enabled(self, lang):
         return True
 
 
 class IndentXmlCommand(BaseIndentCommand):
-    def indent(self, s):                
+
+    def indent(self, s):
         # convert to utf
-        s = s.encode("utf-8") 
+        s = s.encode("utf-8")
         xmlheader = re.compile(b"<\?.*\?>").match(s)
         # convert to plain string without indents and spaces
         s = re.compile(b'>\s+([^\s])', re.DOTALL).sub(b'>\g<1>', s)
         # replace tags to convince minidom process cdata as text
-        s = s.replace(b'<![CDATA[', b'%CDATAESTART%').replace(b']]>', b'%CDATAEEND%') 
+        s = s.replace(b'<![CDATA[', b'%CDATAESTART%').replace(b']]>', b'%CDATAEEND%')
         try:
             s = parseString(s).toprettyxml()
         except Exception as e:
@@ -100,18 +104,19 @@ def indent(self, s):
         s = s.replace('%CDATAESTART%', '<![CDATA[').replace('%CDATAEEND%', ']]>')
         # remove xml header
         s = s.replace("<?xml version=\"1.0\" ?>", "").strip()
-        if xmlheader: 
-                s = xmlheader.group().decode("utf-8") + "\n" + s 
+        if xmlheader:
+            s = xmlheader.group().decode("utf-8") + "\n" + s
         return s
 
     def check_enabled(self, language):
         return ((language == "xml") or (language == "plain text"))
 
 
 class IndentJsonCommand(BaseIndentCommand):
+
     def check_enabled(self, language):
         return ((language == "json") or (language == "plain text"))
 
     def indent(self, s):
         parsed = json.loads(s)
-        return json.dumps(parsed, sort_keys=True, indent=4, separators=(',', ': '))
+        return json.dumps(parsed, sort_keys=True, indent=4, separators=(',', ': '))