Re: [pgAdmin] RM6117 Unable to update column level privileges from security tab. - Mailing list pgadmin-hackers

From Rahul Shirsat
Subject Re: [pgAdmin] RM6117 Unable to update column level privileges from security tab.
Date
Msg-id CAKtn9dPisrrX3DCBU+AAwVZo2XmaUKCaQRBO+m5R-J6u_z1Ung@mail.gmail.com
Whole thread Raw
In response to Re: [pgAdmin] RM6117 Unable to update column level privileges from security tab.  (Aditya Toshniwal <aditya.toshniwal@enterprisedb.com>)
Responses Re: [pgAdmin] RM6117 Unable to update column level privileges from security tab.
List pgadmin-hackers
Akshay,

Removed some whitespace warnings. While this new patch is applied there are still 2 unknown whitespace warnings which are pointing to the wrong location.

Could you please let me know if this works?

On Mon, Feb 8, 2021 at 9:53 AM Aditya Toshniwal <aditya.toshniwal@enterprisedb.com> wrote:
Hi Rahul,

On Sat, Feb 6, 2021 at 1:25 AM Rahul Shirsat <rahul.shirsat@enterprisedb.com> wrote:
Hi Hackers,

Please find the attached patch which resolves the issue of unable to update column level privileges in the security tab.

Additionally, when the user tries to change grantee, the sql wasn't generating, this too issue is resolved.

This security privilege issue is resolved for the following nodes:

1.) Tables (test cases are skipped as there are no resql test cases based on pg & ppas)
2.) Columns
3.) Views              
4.) MViews              
5.) Types              
6.) Functions          
7.) Procedures          
8.) Trigger functions
9.) Sequences
10.) Schema

P.S for committer : Warning of whitespaces after applying patch should be ignored as it is a part of resql test case results.
Probably change the SQL generation logic to avoid the trailing whitespaces ? 

--
Rahul Shirsat
Senior Software Engineer | EnterpriseDB Corporation.


--
Thanks,
Aditya Toshniwal
pgAdmin hacker | Sr. Software Engineer | edbpostgres.com
"Don't Complain about Heat, Plant a TREE"


--
Rahul Shirsat
Senior Software Engineer | EnterpriseDB Corporation.
Attachment

pgadmin-hackers by date:

Previous
From: Yogesh Mahajan
Date:
Subject: Re: [pgAdmin4][Patch] - Feature Tests Failure Fix
Next
From: Dave Page
Date:
Subject: pgAdmin 4 commit: Update Docker container build to handle the new Rust