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

From Akshay Joshi
Subject Re: [pgAdmin] RM6117 Unable to update column level privileges from security tab.
Date
Msg-id CANxoLDcdL=QBnU43Ntk0WtVxs=H11pEG8bbrcUYGgKLRXe47Yg@mail.gmail.com
Whole thread Raw
In response to Re: [pgAdmin] RM6117 Unable to update column level privileges from security tab.  (Rahul Shirsat <rahul.shirsat@enterprisedb.com>)
Responses Re: [pgAdmin] RM6117 Unable to update column level privileges from security tab.  (Nikhil Mohite <nikhil.mohite@enterprisedb.com>)
List pgadmin-hackers
Hi Nikhil

Can you please review the patch thoroughly?

On Mon, Feb 8, 2021 at 3:28 PM Rahul Shirsat <rahul.shirsat@enterprisedb.com> wrote:
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.


--
Thanks & Regards
Akshay Joshi
pgAdmin Hacker | Principal Software Architect
EDB Postgres
Mobile: +91 976-788-8246

pgadmin-hackers by date:

Previous
From: Nikhil Mohite
Date:
Subject: Re: [pgAdmin][RM-6157]: UI issue in delete records.
Next
From: Murtuza Zabuawala
Date:
Subject: Re: pgAdmin 4 commit: Use 'sys.executable' to get the absolute path of the