Remove first_columns parameter

simonhammes · simonhammes · commit 1824597a8815 · 2024-11-15T15:05:31.000+01:00
This is redundant since we're now using Airtable's metadata API
diff --git a/seatable_api/convert_airtable.py b/seatable_api/convert_airtable.py
@@ -495,7 +495,7 @@ def get_schema(self):
 
 class AirtableConvertor(object):
 
-    def __init__(self, airtable_api_key, airtable_base_id, base, table_names, first_columns=[], links=[], excluded_column_types=[], excluded_columns=[]):
+    def __init__(self, airtable_api_key, airtable_base_id, base, table_names, links=[], excluded_column_types=[], excluded_columns=[]):
         """
         airtable_api_key: str
         airtable_base_id: str
@@ -508,7 +508,6 @@ def __init__(self, airtable_api_key, airtable_base_id, base, table_names, first_
         self.airtable_api = AirtableAPI(airtable_api_key, airtable_base_id)
         self.base = base
         self.table_names = table_names
-        self.first_columns = first_columns
         self.links = links
         self.excluded_column_types = excluded_column_types
         self.excluded_columns = excluded_columns
@@ -517,7 +516,6 @@ def __init__(self, airtable_api_key, airtable_base_id, base, table_names, first_
         self.files_convertor = FilesConvertor(airtable_api_key, base)
         self.rows_convertor = RowsConvertor(self.files_convertor)
         self.links_convertor = LinksConvertor()
-        self.get_first_column_map()
         self.get_link_map()
 
     def convert_metadata(self):
@@ -679,8 +677,7 @@ def convert_tables(self):
             table = self.table_map.get(table_name)
             if not table:
                 airtable_columns = self.airtable_column_map[table_name]
-                first_column_name = self.first_column_map.get(table_name) or \
-                    airtable_columns[0]['name']
+                first_column_name = airtable_columns[0]['name']
                 columns = []
                 for column in airtable_columns:
                     if column['type'] == ColumnTypes.LINK:
@@ -788,14 +785,6 @@ def delete_demo_rows(self):
         logger.info('Demo rows deleted from SeaTable base')
         time.sleep(1)
 
-    def get_first_column_map(self):
-        self.first_column_map = {}
-        for column in self.first_columns:
-            table_name = column[0]
-            column_name = column[1]
-            self.first_column_map[table_name] = column_name
-        return self.first_column_map
-
     def get_link_map(self):
         self.link_map = {}
         for link in self.links:
