pgAdmin 4 commit: Fixed an issue where the user is unable to update col

Fixed an issue where the user is unable to update column-level privileges from the security tab. Fixes #6117

Branch
------
master

Details
-------
https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=2036e9d9a8587374af450b5522932af0ce29f2f0
Author: Rahul Shirsat <rahul(dot)shirsat(at)enterprisedb(dot)com>

Modified Files
--------------
docs/en_US/release_notes_5_0.rst | 1 +
.../templates/functions/pg/sql/11_plus/update.sql | 6 +-
.../templates/functions/pg/sql/12_plus/update.sql | 6 +-
.../templates/functions/pg/sql/9.2_plus/update.sql | 4 +
.../templates/functions/pg/sql/9.5_plus/update.sql | 6 +-
.../templates/functions/pg/sql/9.6_plus/update.sql | 6 +-
.../templates/functions/pg/sql/default/update.sql | 6 +-
.../functions/ppas/sql/12_plus/update.sql | 6 +-
.../functions/ppas/sql/9.2_plus/update.sql | 4 +
.../functions/ppas/sql/9.5_plus/update.sql | 6 +-
.../functions/ppas/sql/9.6_plus/update.sql | 6 +-
.../templates/procedures/pg/sql/11_plus/update.sql | 6 +-
.../procedures/ppas/sql/11_plus/update.sql | 6 +-
.../procedures/ppas/sql/9.5_plus/update.sql | 4 +
.../procedures/ppas/sql/9.6_plus/update.sql | 4 +
.../trigger_functions/pg/sql/11_plus/update.sql | 6 +-
.../trigger_functions/pg/sql/9.2_plus/update.sql | 4 +
.../trigger_functions/pg/sql/9.5_plus/update.sql | 6 +-
.../trigger_functions/pg/sql/default/update.sql | 5 +
.../11_plus/alter_function_change_grantee_acl.msql | 3 +
.../11_plus/alter_function_change_grantee_acl.sql | 21 +
.../pg/11_plus/alter_proc_change_grantee_priv.msql | 13 +
.../pg/11_plus/alter_proc_change_grantee_priv.sql | 18 +
.../functions/tests/pg/11_plus/test_function.json | 53 +-
.../tests/pg/11_plus/test_procedures.json | 86 +++-
.../12_plus/alter_function_change_grantee_acl.msql | 3 +
.../12_plus/alter_function_change_grantee_acl.sql | 21 +
.../pg/12_plus/alter_proc_change_grantee_priv.msql | 13 +
.../pg/12_plus/alter_proc_change_grantee_priv.sql | 18 +
.../functions/tests/pg/12_plus/test_function.json | 53 +-
.../tests/pg/12_plus/test_procedures.json | 86 +++-
.../alter_function_change_grantee_acl.msql | 3 +
.../9.5_plus/alter_function_change_grantee_acl.sql | 21 +
.../functions/tests/pg/9.5_plus/test_function.json | 53 +-
.../alter_function_change_grantee_acl.msql | 3 +
.../9.6_plus/alter_function_change_grantee_acl.sql | 21 +
.../functions/tests/pg/9.6_plus/test_function.json | 53 +-
.../11_plus/alter_function_change_grantee_acl.msql | 3 +
.../11_plus/alter_function_change_grantee_acl.sql | 21 +
.../11_plus/alter_proc_change_grantee_priv.msql | 13 +
.../11_plus/alter_proc_change_grantee_priv.sql | 18 +
.../tests/ppas/11_plus/test_function.json | 51 ++
.../tests/ppas/11_plus/test_procedures.json | 86 +++-
.../12_plus/alter_function_change_grantee_acl.msql | 3 +
.../12_plus/alter_function_change_grantee_acl.sql | 21 +
.../12_plus/alter_proc_change_grantee_priv.msql | 13 +
.../12_plus/alter_proc_change_grantee_priv.sql | 18 +
.../tests/ppas/12_plus/test_function.json | 53 +-
.../tests/ppas/12_plus/test_procedures.json | 86 +++-
.../alter_function_change_grantee_acl.msql | 3 +
.../9.5_plus/alter_function_change_grantee_acl.sql | 21 +
.../tests/ppas/9.5_plus/test_function.json | 53 +-
.../alter_function_change_grantee_acl.msql | 3 +
.../9.6_plus/alter_function_change_grantee_acl.sql | 21 +
.../9.6_plus/alter_proc_change_grantee_priv.msql | 13 +
.../9.6_plus/alter_proc_change_grantee_priv.sql | 18 +
.../tests/ppas/9.6_plus/test_function.json | 53 +-
.../tests/ppas/9.6_plus/test_procedures.json | 34 ++
.../templates/sequences/sql/default/update.sql | 4 +
.../pg/default/alter_seq_change_grantee_privs.sql | 19 +
.../alter_seq_change_grantee_privs_msql.sql | 2 +
.../tests/pg/default/alter_seq_privs_remove.sql | 2 -
.../tests/pg/default/test_sequences_pg.json | 46 +-
.../default/alter_seq_change_grantee_privs.sql | 19 +
.../alter_seq_change_grantee_privs_msql.sql | 2 +
.../tests/ppas/default/alter_seq_privs_remove.sql | 2 -
.../tests/ppas/default/test_sequences_ppas.json | 46 +-
.../pg/10_plus/alter_column_alt_privilege.msql | 2 +
.../pg/10_plus/alter_column_alt_privilege.sql | 13 +
.../tests/{ => pg}/10_plus/alter_column_char.msql | 0
.../tests/{ => pg}/10_plus/alter_column_char.sql | 0
.../10_plus/alter_column_drop_identity.msql | 0
.../10_plus/alter_column_drop_identity.sql | 0
.../alter_column_drop_identity_by_default.msql | 0
.../alter_column_drop_identity_by_default.sql | 0
.../{ => pg}/10_plus/alter_column_identity.msql | 0
.../{ => pg}/10_plus/alter_column_identity.sql | 0
.../10_plus/alter_column_identity_by_default.msql | 0
.../10_plus/alter_column_identity_by_default.sql | 0
.../alter_column_identity_for_comb_start_seq.msql | 0
.../alter_column_identity_for_comb_start_seq.sql | 0
.../alter_column_identity_for_restart_seq.msql | 0
.../alter_column_identity_for_restart_seq.sql | 0
.../tests/{ => pg}/10_plus/alter_column_int.msql | 0
.../tests/{ => pg}/10_plus/alter_column_int.sql | 0
.../{ => pg}/10_plus/alter_column_numeric.msql | 0
.../{ => pg}/10_plus/alter_column_numeric.sql | 0
.../10_plus/alter_column_remove_length.msql | 0
.../10_plus/alter_column_remove_length.sql | 0
.../tests/{ => pg}/10_plus/create_column_char.msql | 0
.../tests/{ => pg}/10_plus/create_column_char.sql | 0
.../create_column_identity_for_restart_seq.msql | 0
.../create_column_identity_for_restart_seq.sql | 0
.../tests/{ => pg}/10_plus/create_column_int.msql | 0
.../tests/{ => pg}/10_plus/create_column_int.sql | 0
.../10_plus/create_column_int_identity.msql | 0
.../10_plus/create_column_int_identity.sql | 0
.../create_column_int_identity_by_default.msql | 0
.../create_column_int_identity_by_default.sql | 0
.../{ => pg}/10_plus/create_column_numeric.msql | 0
.../{ => pg}/10_plus/create_column_numeric.sql | 0
.../columns/tests/{ => pg}/10_plus/test.json | 34 ++
.../pg/12_plus/alter_column_alt_privilege.msql | 2 +
.../pg/12_plus/alter_column_alt_privilege.sql | 13 +
.../tests/{ => pg}/12_plus/alter_column_char.msql | 0
.../tests/{ => pg}/12_plus/alter_column_char.sql | 0
.../12_plus/alter_column_drop_identity.msql | 0
.../12_plus/alter_column_drop_identity.sql | 0
.../alter_column_drop_identity_by_default.msql | 0
.../alter_column_drop_identity_by_default.sql | 0
.../{ => pg}/12_plus/alter_column_generated.msql | 0
.../{ => pg}/12_plus/alter_column_generated.sql | 0
...ter_column_generated_with_existing_columns.msql | 0
...lter_column_generated_with_existing_columns.sql | 0
.../{ => pg}/12_plus/alter_column_identity.msql | 0
.../{ => pg}/12_plus/alter_column_identity.sql | 0
.../12_plus/alter_column_identity_by_default.msql | 0
.../12_plus/alter_column_identity_by_default.sql | 0
.../alter_column_identity_for_comb_start_seq.msql | 0
.../alter_column_identity_for_comb_start_seq.sql | 0
.../alter_column_identity_for_restart_seq.msql | 0
.../alter_column_identity_for_restart_seq.sql | 0
.../tests/{ => pg}/12_plus/alter_column_int.msql | 0
.../tests/{ => pg}/12_plus/alter_column_int.sql | 0
.../{ => pg}/12_plus/alter_column_numeric.msql | 0
.../{ => pg}/12_plus/alter_column_numeric.sql | 0
.../12_plus/alter_column_remove_length.msql | 0
.../12_plus/alter_column_remove_length.sql | 0
.../tests/{ => pg}/12_plus/create_column_char.msql | 0
.../tests/{ => pg}/12_plus/create_column_char.sql | 0
.../create_column_identity_for_restart_seq.msql | 0
.../create_column_identity_for_restart_seq.sql | 0
.../tests/{ => pg}/12_plus/create_column_int.msql | 0
.../tests/{ => pg}/12_plus/create_column_int.sql | 0
.../12_plus/create_column_int_generated.msql | 0
.../12_plus/create_column_int_generated.sql | 0
...column_int_generated_with_existing_columns.msql | 0
..._column_int_generated_with_existing_columns.sql | 0
.../12_plus/create_column_int_identity.msql | 0
.../12_plus/create_column_int_identity.sql | 0
.../create_column_int_identity_by_default.msql | 0
.../create_column_int_identity_by_default.sql | 0
.../{ => pg}/12_plus/create_column_numeric.msql | 0
.../{ => pg}/12_plus/create_column_numeric.sql | 0
.../columns/tests/{ => pg}/12_plus/test.json | 14 +
.../pg/default/alter_column_alt_privilege.msql | 2 +
.../pg/default/alter_column_alt_privilege.sql | 13 +
.../tests/{ => pg}/default/alter_column_char.msql | 0
.../tests/{ => pg}/default/alter_column_char.sql | 0
.../tests/{ => pg}/default/alter_column_int.msql | 0
.../tests/{ => pg}/default/alter_column_int.sql | 0
.../{ => pg}/default/alter_column_numeric.msql | 0
.../{ => pg}/default/alter_column_numeric.sql | 0
.../default/alter_column_remove_length.msql | 0
.../default/alter_column_remove_length.sql | 0
.../tests/{ => pg}/default/create_column_char.msql | 0
.../tests/{ => pg}/default/create_column_char.sql | 0
.../tests/{ => pg}/default/create_column_int.msql | 0
.../tests/{ => pg}/default/create_column_int.sql | 0
.../{ => pg}/default/create_column_numeric.msql | 0
.../{ => pg}/default/create_column_numeric.sql | 0
.../columns/tests/{ => pg}/default/test.json | 14 +
.../ppas/10_plus/alter_column_alt_privilege.msql | 2 +
.../ppas/10_plus/alter_column_alt_privilege.sql | 13 +
.../tests/ppas/10_plus/alter_column_char.msql | 15 +
.../tests/ppas/10_plus/alter_column_char.sql | 17 +
.../ppas/10_plus/alter_column_drop_identity.msql | 2 +
.../ppas/10_plus/alter_column_drop_identity.sql | 11 +
.../alter_column_drop_identity_by_default.msql | 2 +
.../alter_column_drop_identity_by_default.sql | 9 +
.../tests/ppas/10_plus/alter_column_identity.msql | 7 +
.../tests/ppas/10_plus/alter_column_identity.sql | 11 +
.../10_plus/alter_column_identity_by_default.msql | 8 +
.../10_plus/alter_column_identity_by_default.sql | 9 +
.../alter_column_identity_for_comb_start_seq.msql | 2 +
.../alter_column_identity_for_comb_start_seq.sql | 9 +
.../alter_column_identity_for_restart_seq.msql | 2 +
.../alter_column_identity_for_restart_seq.sql | 9 +
.../tests/ppas/10_plus/alter_column_int.msql | 9 +
.../tests/ppas/10_plus/alter_column_int.sql | 11 +
.../tests/ppas/10_plus/alter_column_numeric.msql | 9 +
.../tests/ppas/10_plus/alter_column_numeric.sql | 15 +
.../ppas/10_plus/alter_column_remove_length.msql | 2 +
.../ppas/10_plus/alter_column_remove_length.sql | 15 +
.../tests/ppas/10_plus/create_column_char.msql | 5 +
.../tests/ppas/10_plus/create_column_char.sql | 9 +
.../create_column_identity_for_restart_seq.msql | 5 +
.../create_column_identity_for_restart_seq.sql | 9 +
.../tests/ppas/10_plus/create_column_int.msql | 5 +
.../tests/ppas/10_plus/create_column_int.sql | 9 +
.../ppas/10_plus/create_column_int_identity.msql | 5 +
.../ppas/10_plus/create_column_int_identity.sql | 9 +
.../create_column_int_identity_by_default.msql | 5 +
.../create_column_int_identity_by_default.sql | 9 +
.../tests/ppas/10_plus/create_column_numeric.msql | 9 +
.../tests/ppas/10_plus/create_column_numeric.sql | 13 +
.../tables/columns/tests/ppas/10_plus/test.json | 531 ++++++++++++++++++++
.../ppas/12_plus/alter_column_alt_privilege.msql | 2 +
.../ppas/12_plus/alter_column_alt_privilege.sql | 13 +
.../tests/ppas/12_plus/alter_column_char.msql | 15 +
.../tests/ppas/12_plus/alter_column_char.sql | 17 +
.../ppas/12_plus/alter_column_drop_identity.msql | 2 +
.../ppas/12_plus/alter_column_drop_identity.sql | 11 +
.../alter_column_drop_identity_by_default.msql | 2 +
.../alter_column_drop_identity_by_default.sql | 9 +
.../tests/ppas/12_plus/alter_column_generated.msql | 7 +
.../tests/ppas/12_plus/alter_column_generated.sql | 11 +
...ter_column_generated_with_existing_columns.msql | 7 +
...lter_column_generated_with_existing_columns.sql | 11 +
.../tests/ppas/12_plus/alter_column_identity.msql | 7 +
.../tests/ppas/12_plus/alter_column_identity.sql | 11 +
.../12_plus/alter_column_identity_by_default.msql | 8 +
.../12_plus/alter_column_identity_by_default.sql | 9 +
.../alter_column_identity_for_comb_start_seq.msql | 2 +
.../alter_column_identity_for_comb_start_seq.sql | 9 +
.../alter_column_identity_for_restart_seq.msql | 2 +
.../alter_column_identity_for_restart_seq.sql | 9 +
.../tests/ppas/12_plus/alter_column_int.msql | 9 +
.../tests/ppas/12_plus/alter_column_int.sql | 11 +
.../tests/ppas/12_plus/alter_column_numeric.msql | 9 +
.../tests/ppas/12_plus/alter_column_numeric.sql | 15 +
.../ppas/12_plus/alter_column_remove_length.msql | 2 +
.../ppas/12_plus/alter_column_remove_length.sql | 15 +
.../tests/ppas/12_plus/create_column_char.msql | 5 +
.../tests/ppas/12_plus/create_column_char.sql | 9 +
.../create_column_identity_for_restart_seq.msql | 5 +
.../create_column_identity_for_restart_seq.sql | 9 +
.../tests/ppas/12_plus/create_column_int.msql | 5 +
.../tests/ppas/12_plus/create_column_int.sql | 9 +
.../ppas/12_plus/create_column_int_generated.msql | 5 +
.../ppas/12_plus/create_column_int_generated.sql | 9 +
...column_int_generated_with_existing_columns.msql | 5 +
..._column_int_generated_with_existing_columns.sql | 9 +
.../ppas/12_plus/create_column_int_identity.msql | 5 +
.../ppas/12_plus/create_column_int_identity.sql | 9 +
.../create_column_int_identity_by_default.msql | 5 +
.../create_column_int_identity_by_default.sql | 9 +
.../tests/ppas/12_plus/create_column_numeric.msql | 9 +
.../tests/ppas/12_plus/create_column_numeric.sql | 13 +
.../tables/columns/tests/ppas/12_plus/test.json | 539 +++++++++++++++++++++
.../ppas/default/alter_column_alt_privilege.msql | 2 +
.../ppas/default/alter_column_alt_privilege.sql | 13 +
.../tests/ppas/default/alter_column_char.msql | 15 +
.../tests/ppas/default/alter_column_char.sql | 17 +
.../tests/ppas/default/alter_column_int.msql | 9 +
.../tests/ppas/default/alter_column_int.sql | 11 +
.../tests/ppas/default/alter_column_numeric.msql | 9 +
.../tests/ppas/default/alter_column_numeric.sql | 15 +
.../ppas/default/alter_column_remove_length.msql | 2 +
.../ppas/default/alter_column_remove_length.sql | 15 +
.../tests/ppas/default/create_column_char.msql | 5 +
.../tests/ppas/default/create_column_char.sql | 9 +
.../tests/ppas/default/create_column_int.msql | 5 +
.../tests/ppas/default/create_column_int.sql | 9 +
.../tests/ppas/default/create_column_numeric.msql | 9 +
.../tests/ppas/default/create_column_numeric.sql | 13 +
.../tables/columns/tests/ppas/default/test.json | 193 ++++++++
.../templates/columns/sql/10_plus/update.sql | 9 +
.../templates/columns/sql/9.2_plus/update.sql | 9 +
.../templates/columns/sql/default/update.sql | 9 +
.../tables/templates/tables/sql/11_plus/update.sql | 4 +
.../tables/templates/tables/sql/12_plus/update.sql | 4 +
.../templates/tables/sql/9.6_plus/update.sql | 4 +
.../tables/templates/tables/sql/default/update.sql | 4 +
.../templates/schemas/pg/9.2_plus/sql/update.sql | 8 +
.../templates/schemas/pg/default/sql/update.sql | 8 +
.../templates/schemas/ppas/9.1_plus/sql/update.sql | 8 +
.../templates/schemas/ppas/9.2_plus/sql/update.sql | 8 +
.../tests/pg/9.2_plus/alter_schema_drop_priv.sql | 2 +-
.../tests/pg/9.2_plus/alter_schema_update_priv.sql | 22 +
.../pg/9.2_plus/alter_schema_update_priv_msql.sql | 12 +
.../schemas/tests/pg/9.2_plus/test_schema.json | 46 +-
.../tests/ppas/9.2_plus/alter_schema_drop_priv.sql | 2 +-
.../ppas/9.2_plus/alter_schema_update_priv.sql | 22 +
.../9.2_plus/alter_schema_update_priv_msql.sql | 14 +
.../schemas/tests/ppas/9.2_plus/test_schema.json | 43 ++
.../types/templates/types/sql/default/update.sql | 4 +
.../tests/pg/default/alter_acl_grantee_change.sql | 17 +
.../pg/default/alter_acl_grantee_change_msql.sql | 2 +
.../alter_composite_type_acl_description.msql | 0
.../alter_composite_type_acl_description.sql | 0
.../default/alter_composite_type_add_member.msql | 0
.../default/alter_composite_type_add_member.sql | 0
.../alter_composite_type_remove_member.msql | 0
.../default/alter_composite_type_remove_member.sql | 0
.../default/alter_enum_type_acl_description.msql | 0
.../default/alter_enum_type_acl_description.sql | 0
.../{ => pg}/default/alter_enum_type_label.msql | 0
.../{ => pg}/default/alter_enum_type_label.sql | 0
.../default/alter_range_type_acl_description.msql | 0
.../default/alter_range_type_acl_description.sql | 0
.../default/alter_shell_type_acl_description.msql | 0
.../default/alter_shell_type_acl_description.sql | 0
.../{ => pg}/default/create_composite_type.msql | 0
.../{ => pg}/default/create_composite_type.sql | 0
.../default/create_composite_type_one_field.sql | 0
.../tests/{ => pg}/default/create_enum_type.msql | 0
.../tests/{ => pg}/default/create_enum_type.sql | 0
.../tests/{ => pg}/default/create_range_type.msql | 0
.../tests/{ => pg}/default/create_range_type.sql | 0
.../tests/{ => pg}/default/create_shell_type.msql | 0
.../tests/{ => pg}/default/create_shell_type.sql | 0
.../types/tests/{ => pg}/default/tests.json | 51 ++
.../ppas/default/alter_acl_grantee_change.sql | 17 +
.../ppas/default/alter_acl_grantee_change_msql.sql | 2 +
.../alter_composite_type_acl_description.msql | 4 +
.../alter_composite_type_acl_description.sql | 20 +
.../default/alter_composite_type_add_member.msql | 2 +
.../default/alter_composite_type_add_member.sql | 14 +
.../alter_composite_type_remove_member.msql | 2 +
.../default/alter_composite_type_remove_member.sql | 13 +
.../default/alter_enum_type_acl_description.msql | 4 +
.../default/alter_enum_type_acl_description.sql | 16 +
.../tests/ppas/default/alter_enum_type_label.msql | 2 +
.../tests/ppas/default/alter_enum_type_label.sql | 9 +
.../default/alter_range_type_acl_description.msql | 4 +
.../default/alter_range_type_acl_description.sql | 19 +
.../default/alter_shell_type_acl_description.msql | 2 +
.../default/alter_shell_type_acl_description.sql | 11 +
.../tests/ppas/default/create_composite_type.msql | 6 +
.../tests/ppas/default/create_composite_type.sql | 13 +
.../default/create_composite_type_one_field.sql | 11 +
.../types/tests/ppas/default/create_enum_type.msql | 2 +
.../types/tests/ppas/default/create_enum_type.sql | 9 +
.../tests/ppas/default/create_range_type.msql | 4 +
.../types/tests/ppas/default/create_range_type.sql | 12 +
.../tests/ppas/default/create_shell_type.msql | 1 +
.../types/tests/ppas/default/create_shell_type.sql | 8 +
.../schemas/types/tests/ppas/default/tests.json | 284 +++++++++++
.../templates/mviews/pg/9.3_plus/sql/update.sql | 4 +
.../templates/mviews/pg/9.4_plus/sql/update.sql | 4 +
.../templates/mviews/ppas/9.3_plus/sql/update.sql | 4 +
.../templates/views/pg/9.1_plus/sql/update.sql | 4 +
.../templates/views/pg/9.2_plus/sql/update.sql | 4 +
.../templates/views/pg/9.3_plus/sql/update.sql | 4 +
.../templates/views/pg/9.4_plus/sql/update.sql | 4 +
.../templates/views/ppas/9.1_plus/sql/update.sql | 4 +
.../templates/views/ppas/9.2_plus/sql/update.sql | 4 +
.../templates/views/ppas/9.3_plus/sql/update.sql | 4 +
.../templates/views/ppas/9.4_plus/sql/update.sql | 4 +
.../9.4_plus/alter_mview_change_grantee_priv.sql | 17 +
.../alter_mview_change_grantee_priv_msql.sql | 2 +
.../tests/pg/9.4_plus/alter_view_delete_priv.sql | 1 -
.../tests/pg/9.4_plus/alter_view_update_priv.sql | 19 +
.../pg/9.4_plus/alter_view_update_priv_msql.sql | 2 +
.../views/tests/pg/9.4_plus/test_mview.json | 57 +++
.../schemas/views/tests/pg/9.4_plus/tests.json | 71 ++-
.../9.4_plus/alter_mview_change_grantee_priv.sql | 17 +
.../alter_mview_change_grantee_priv_msql.sql | 2 +
.../tests/ppas/9.4_plus/alter_view_delete_priv.sql | 2 -
.../tests/ppas/9.4_plus/alter_view_update_priv.sql | 19 +
.../ppas/9.4_plus/alter_view_update_priv_msql.sql | 2 +
.../views/tests/ppas/9.4_plus/test_mview.json | 57 +++
.../schemas/views/tests/ppas/9.4_plus/tests.json | 65 +++
.../schemas/views/tests/view_test_data.json | 1 +
.../server_groups/servers/static/js/privilege.js | 5 +-
web/pgadmin/browser/server_groups/servers/utils.py | 14 +-
357 files changed, 4696 insertions(+), 45 deletions(-)