Re: [pgAdmin4][Patch] - SonarQube Fixes

От

Akshay Joshi

Тема

Дата

4 августа 2020 г. в 06:40:17

Msg-id

CANxoLDf93wWrt8W94wVhtz1mzFJJe2Hb8UNDPmL+maxwnpNcrQ@mail.gmail.com

обсуждение

Ответ на

[pgAdmin4][Patch] - SonarQube Fixes (Khushboo Vashi)

Список

pgadmin-hackers

Дерево обсуждения

[pgAdmin4][Patch] - SonarQube Fixes Khushboo Vashi <khushboo.vashi@enterprisedb.com> 3 августа 2020 г. в 08:06:09

Re: [pgAdmin4][Patch] - SonarQube Fixes Akshay Joshi <akshay.joshi@enterprisedb.com> 4 августа 2020 г. в 06:40:17

Re: [pgAdmin4][Patch] - SonarQube Fixes Khushboo Vashi <khushboo.vashi@enterprisedb.com> 19 августа 2020 г. в 08:31:01

Re: [pgAdmin4][Patch] - SonarQube Fixes Akshay Joshi <akshay.joshi@enterprisedb.com> 19 августа 2020 г. в 08:56:57

Re: [pgAdmin4][Patch] - SonarQube Fixes Khushboo Vashi <khushboo.vashi@enterprisedb.com> 3 сентября 2020 г. в 11:09:05

Re: [pgAdmin4][Patch] - SonarQube Fixes Akshay Joshi <akshay.joshi@enterprisedb.com> 3 сентября 2020 г. в 13:26:32

Hi Khushboo

As per offline discussion, multiple developers declaring the constant for the same string and error messages. To avoid that only one developer should work on this task.

Suggestions:

We should have a common class for Constants (which are not module specific).
For module specific constants we should not declare them as Global variables if possible declare them as the Class variable.
For common error messages(if any) we can define functions in PGChildNodeView.

Please incorporate the changes send by Yogesh.

On Mon, Aug 3, 2020 at 1:36 PM Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote: