Skip to content

Commit

Permalink
Fixed an issue where the wrong schema is displayed for a foreign key …
Browse files Browse the repository at this point in the history
…in the schema diff tool. pgadmin-org#5488
  • Loading branch information
akshay-joshi committed Dec 1, 2022
1 parent 475726d commit 4b32657
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 4 deletions.
2 changes: 2 additions & 0 deletions docs/en_US/release_notes_6_17.rst
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,9 @@ Housekeeping
Bug fixes
*********

| `Issue #5488 <https://github.com/pgadmin-org/pgadmin4/issues/5488>`_ - Fixed an issue where the wrong schema is displayed for a foreign key in the schema diff tool.
| `Issue #5495 <https://github.com/pgadmin-org/pgadmin4/issues/5495>`_ - Ensure that the query history date format in Desktop mode matches the format of the locale of the pgadmin server.
| `Issue #5505 <https://github.com/pgadmin-org/pgadmin4/issues/5505>`_ - Fixed an issue where the CSV file would not download if the CSV quote character length exceeded 1.
| `Issue #5513 <https://github.com/pgadmin-org/pgadmin4/issues/5513>`_ - Ensure that DATA_DIR dependent folders/files are automatically created inside the specified DATA_DIR if they are not specified separately in the configuration file.
| `Issue #5548 <https://github.com/pgadmin-org/pgadmin4/issues/5548>`_ - Fixed an issue where editor position was wrong when editing data from result grid.
| `Issue #5575 <https://github.com/pgadmin-org/pgadmin4/issues/5575>`_ - Ensure the query tool is launched successfully for the servers registered with the PostgreSQL service.
Original file line number Diff line number Diff line change
Expand Up @@ -355,8 +355,8 @@ def _checks_for_schema_diff(table, schema, data):

if 'remote_schema' not in data:
data['remote_schema'] = None
elif 'schema' in data and (schema is None or schema == ''):
data['remote_schema'] = data['schema']
if 'schema' in data and (schema is None or schema == ''):
data['remote_schema'] = data['schema']

if 'remote_table' not in data:
data['remote_table'] = None
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,8 @@ class SchemaDiffTableCompare(SchemaDiffObjectCompare):
constraint_keys_to_ignore = ['relname', 'nspname', 'parent_tbl',
'attrelid', 'adrelid', 'fknsp', 'confrelid',
'references', 'refnsp', 'remote_schema',
'conkey', 'indkey', 'references_table_name']
'conkey', 'indkey', 'references_table_name',
'refnspoid']

trigger_keys_to_ignore = ['xmin', 'tgrelid', 'tgfoid', 'tfunction',
'tgqual', 'tgconstraint']
Expand Down
1 change: 0 additions & 1 deletion web/pgadmin/tools/schema_diff/directory_compare.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,6 @@
import string
from pgadmin.tools.schema_diff.model import SchemaDiffModel
from flask import current_app
from pgadmin.utils.preferences import Preferences
from pgadmin.utils.constants import PGADMIN_STRING_SEPARATOR

count = 1
Expand Down

0 comments on commit 4b32657

Please sign in to comment.