remove unused _parse_text

Author: bsipocz

astroquery/xmatch/core.py

@@ -2,12 +2,11 @@
 
 from io import StringIO, BytesIO
 
-from astropy.io import ascii, votable
+from astropy.io import votable
 import astropy.units as u
 from astropy.table import Table
 from requests import HTTPError
 
-
 from astroquery.query import BaseQuery
 from astroquery.exceptions import InvalidQueryError
 from astroquery.utils import url_helpers, prepend_docstr_nosections, async_to_sync
@@ -78,7 +77,7 @@ def query(self, cat1, cat2, max_distance,
         if get_query_payload:
             return response
 
-        return Table.read(BytesIO(response.content), format='votable')
+        return Table.read(BytesIO(response.content), format='votable', use_names_over_ids=True)
 
     @prepend_docstr_nosections("\n" + query.__doc__)
     def query_async(self, cat1, cat2, max_distance, colRA1=None, colDec1=None,
@@ -190,22 +189,5 @@ def get_available_tables(self, cache=True):
         content = response.text
         return content.splitlines()
 
-    def _parse_text(self, text):
-        """
-        Parse a CSV text file that has potentially duplicated header names
-        """
-        header = text.split("\n")[0]
-        colnames = header.split(",")
-        for column in colnames:
-            if colnames.count(column) > 1:
-                counter = 1
-                while colnames.count(column) > 0:
-                    colnames[colnames.index(column)] = column + "_{counter}".format(counter=counter)
-                    counter += 1
-        new_text = ",".join(colnames) + "\n" + "\n".join(text.split("\n")[1:])
-        result = ascii.read(new_text, format='csv', fast_reader=False)
-
-        return result
-
 
 XMatch = XMatchClass()

astroquery/xmatch/tests/test_xmatch.py

@@ -104,9 +104,3 @@ def test_xmatch_query_cat1_table_local(monkeypatch):
         'errHalfMaj', 'errHalfMin', 'errPosAng', 'Jmag', 'Hmag', 'Kmag',
         'e_Jmag', 'e_Hmag', 'e_Kmag', 'Qfl', 'Rfl', 'X', 'MeasureJD']
     assert len(table) == 11
-
-
-@pytest.mark.parametrize('datafile', DATA_FILES.values())
-def test_parse_text(datafile):
-    xm = XMatch()
-    xm._parse_text(datafile)