From: | Pradip Parkale <pradip(dot)parkale(at)enterprisedb(dot)com> |
---|---|
To: | Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [pgAdmin][SonarQube] : Remove duplicates string literals. |
Date: | 2020-09-03 13:49:09 |
Message-ID: | CAJ9T6Ss3trXEnFPTwi+BV0JGLjxN8-D3YsjtsnE7-MFSzN0_EA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi Akshay,
Please find the updated patch.
On Thu, Sep 3, 2020 at 6:24 PM Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
wrote:
> Hi Pradip
>
> Following are the review comments:
>
> - ROLE_NOT_FOUND already in wrapped in gettext() then no need to use
> like gone(_(ROLE_NOT_FOUND)). In fact, can we use
>
> self.not_found_error_msg()instead of new constant.
>
> - Same with ERROR_FETCHING_ROLE_INFORMATION it's already in gettext,
> also we can use this constant for string "
>
> Error retrieving roles from the database server"
>
>
>
> On Thu, Sep 3, 2020 at 4:30 PM Pradip Parkale <
> pradip(dot)parkale(at)enterprisedb(dot)com> wrote:
>
>> Hi Hackers,
>>
>> Attached is a patch for removing duplicates string literals.
>> /server_groups/servers/roles/__init__.py - 8 places
>>
>>
>>
>> --
>> Thanks & Regards,
>> Pradip Parkale
>> Software Engineer | EnterpriseDB Corporation
>>
>
>
> --
> *Thanks & Regards*
> *Akshay Joshi*
> *pgAdmin Hacker | Sr. Software Architect*
> *EDB Postgres <http://edbpostgres.com>*
>
> *Mobile: +91 976-788-8246*
>
--
Thanks & Regards,
Pradip Parkale
Software Engineer | EnterpriseDB Corporation
Attachment | Content-Type | Size |
---|---|---|
sonar_Qube_v2.patch | application/octet-stream | 6.5 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Akshay Joshi | 2020-09-03 13:52:03 | pgAdmin 4 commit: Modified the logic to check Port is in use for runtim |
Previous Message | Akshay Joshi | 2020-09-03 13:26:51 | Re: [pgAdmin][SonarQube] Reduce cognitive complexity |