Do not add comment on table nor column

Closes #14614

trytond-gis/trytond_gis/postgis/table.py

@@ -12,15 +12,16 @@ logger = logging.getLogger(__name__)
 
 class TableHandler(PGTableHandler):
 
-    def add_column(self, column_name, abstract_type, default=None, comment=''):
+    def add_column(self, column_name, abstract_type, default=None):
         if abstract_type.startswith('GIS_'):
             column_adder = self.add_geometry_column
         else:
             column_adder = super().add_column
-        column_adder(column_name, abstract_type, default, comment)
+        column_adder(column_name, abstract_type, default)
 
-    def add_geometry_column(self, column_name, abstract_type, default_fun=None,
-            fill_default=True, comment=''):
+    def add_geometry_column(
+            self, column_name, abstract_type, default_fun=None,
+            fill_default=True):
         cursor = Transaction().connection.cursor()
 
         match = GIS_SQL_TYPE_RE.match(abstract_type)

trytond/trytond/backend/postgresql/table.py

@@ -56,11 +56,6 @@ class TableHandler(TableHandlerInterface):
         if view_exists:
             return
 
-        if model.__doc__ and self.is_owner:
-            cursor.execute(SQL('COMMENT ON TABLE {} IS %s').format(
-                        Identifier(self.table_name)),
-                (model.__doc__,))
-
         def migrate_to_identity(table, column):
             previous_seq_name = f"{table}_{column}_seq"
             cursor.execute(
@@ -324,7 +319,7 @@ class TableHandler(TableHandlerInterface):
                 .format(Identifier(self.table_name), Identifier(column_name)),
                 (value,))
 
-    def add_column(self, column_name, sql_type, default=None, comment=''):
+    def add_column(self, column_name, sql_type, default=None):
         cursor = ClientCursor(Transaction().connection)
         database = Transaction().database
 
@@ -332,13 +327,6 @@ class TableHandler(TableHandlerInterface):
         match = VARCHAR_SIZE_RE.match(sql_type)
         field_size = int(match.group(1)) if match else None
 
-        def add_comment():
-            if comment and self.is_owner:
-                cursor.execute(
-                    SQL('COMMENT ON COLUMN {}.{} IS %s').format(
-                        Identifier(self.table_name),
-                        Identifier(column_name)),
-                    (comment,))
         if self.column_exist(column_name):
             if (column_name in ('create_date', 'write_date')
                     and column_type[1].lower() != 'timestamp(6)'):
@@ -350,7 +338,6 @@ class TableHandler(TableHandlerInterface):
                         Identifier(self.table_name),
                         Identifier(column_name)))
 
-            add_comment()
             base_type = column_type[0].lower()
             typname = self._columns[column_name]['typname']
             if base_type != typname:
@@ -411,7 +398,6 @@ class TableHandler(TableHandlerInterface):
                 Identifier(self.table_name),
                 Identifier(column_name),
                 SQL(column_type)))
-        add_comment()
 
         if default:
             # check if table is non-empty:

trytond/trytond/backend/sqlite/table.py

@@ -190,17 +190,16 @@ class TableHandler(TableHandlerInterface):
     def db_default(self, column_name, value):
         pass
 
-    def add_column(self, column_name, sql_type, default=None, comment=''):
+    def add_column(self, column_name, sql_type, default=None):
         database = Transaction().database
         column_type = database.sql_type(sql_type)
         match = VARCHAR_SIZE_RE.match(sql_type)
         field_size = int(match.group(1)) if match else None
 
-        self._add_raw_column(column_name, column_type, default, field_size,
-            comment)
+        self._add_raw_column(column_name, column_type, default, field_size)
 
-    def _add_raw_column(self, column_name, column_type, default=None,
-            field_size=None, string=''):
+    def _add_raw_column(
+            self, column_name, column_type, default=None, field_size=None):
         if self.column_exist(column_name):
             base_type = column_type[0].upper()
             if base_type != self._columns[column_name]['typname']:

trytond/trytond/backend/table.py

@@ -71,7 +71,7 @@ class TableHandlerInterface(object):
     def db_default(self, column_name, value):
         raise NotImplementedError
 
-    def add_column(self, column_name, abstract_type, default=None, comment=''):
+    def add_column(self, column_name, abstract_type, default=None):
         raise NotImplementedError
 
     def add_fk(self, columns, reference, ref_columns=None, on_delete=None):