From: | Rahul Shirsat <rahul(dot)shirsat(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: [pgAdmin4][Patch] - SonarQube Fixes #2 |
Date: | 2020-08-20 11:17:00 |
Message-ID: | CAKtn9dMqy86MJ+jNUTd=rMtR1DtLU3gRGM7_GoXVzs1e4RphRQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi Akshay,
Please find the updated patch attached in this email.
On Wed, Aug 19, 2020 at 2:40 PM Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
wrote:
> Hi Rahul
>
> I have committed Khushboo's patch where she has introduced a new method
> 'not_found_error_msg', so instead of creating a new constant
> 'TABLE_NOT_FOUND_MSG' use self.not_found_error_msg.
>
Done
>
> Also, instead of
> _NODE_CSS = ‘node.css’
> _COLLECTION_CSS = ‘collection.css’
>
> We can use the following:
> _NODE_CSS = _BROWSER_CSS_PATH + ‘/node.css’
> _COLLECTION_CSS = _BROWSER_CSS_PATH + ‘/collection.css’
>
Done
>
>
> Make the above changes and resend the patch.
>
> On Fri, Aug 14, 2020 at 8:34 PM Rahul Shirsat <
> rahul(dot)shirsat(at)enterprisedb(dot)com> wrote:
>
>> Hi Hackers,
>>
>> Please find the attached patch to fix the SonarQube Code smells having
>> Rule:
>>
>>
>> 1. Define a constant instead of duplicating this literal
>> "browser/css/node.css" 4 times.
>> 2. Define a constant instead of duplicating this literal "The
>> specified table could not be found." 10 times.
>> 3. Define a constant instead of duplicating this literal
>> 'get_columns_for_table.sql' 3 times.
>>
>> --
>> *Rahul Shirsat*
>> Software Engineer | EnterpriseDB Corporation.
>>
>
>
> --
> *Thanks & Regards*
> *Akshay Joshi*
> *pgAdmin Hacker | Sr. Software Architect*
> *EDB Postgres <http://edbpostgres.com>*
>
> *Mobile: +91 976-788-8246*
>
--
*Rahul Shirsat*
Software Engineer | EnterpriseDB Corporation.
Attachment | Content-Type | Size |
---|---|---|
sonar_qube_table_init_v2.patch | application/octet-stream | 25.5 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Nikhil Mohite | 2020-08-20 12:01:36 | Re: [pgAdmin][RM5990]: Update the style guide on pgAdmin4 website |
Previous Message | Fahar Abbas | 2020-08-20 11:10:59 | Re: pgAdmin4 v4.25 candidate builds |