Re: [pgAdmin] RM6117 Unable to update column level privileges from security tab.

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(dot)toshniwal(at)enterprisedb(dot)com> wrote:

> Hi Rahul,
>
> On Sat, Feb 6, 2021 at 1:25 AM Rahul Shirsat <
> rahul(dot)shirsat(at)enterprisedb(dot)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*
> <http://edbpostgres.com>
> "Don't Complain about Heat, Plant a TREE"
>

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