Обсуждение: [pgAdmin][SonarQube] : Remove duplicates string literals.
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
Software Engineer | EnterpriseDB Corporation
Вложения
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.parkale@enterprisedb.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 PostgresMobile: +91 976-788-8246
Hi Akshay,
Please find the updated patch.
On Thu, Sep 3, 2020 at 6:24 PM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi PradipFollowing 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.parkale@enterprisedb.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 & RegardsAkshay JoshipgAdmin Hacker | Sr. Software ArchitectEDB PostgresMobile: +91 976-788-8246
Thanks & Regards,
Pradip Parkale
Software Engineer | EnterpriseDB Corporation
Software Engineer | EnterpriseDB Corporation
Вложения
Thanks, patch applied.
On Thu, Sep 3, 2020 at 7:19 PM Pradip Parkale <pradip.parkale@enterprisedb.com> wrote:
Hi Akshay,Please find the updated patch.On Thu, Sep 3, 2020 at 6:24 PM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:Hi PradipFollowing 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.parkale@enterprisedb.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 & RegardsAkshay JoshipgAdmin Hacker | Sr. Software ArchitectEDB PostgresMobile: +91 976-788-8246--Thanks & Regards,Pradip Parkale
Software Engineer | EnterpriseDB Corporation
--
Thanks & Regards
Akshay Joshi
pgAdmin Hacker | Sr. Software Architect
EDB PostgresMobile: +91 976-788-8246