Re: Re-[Sonarqube]- Clumsy code

Поиск
Список
Период
Сортировка
От Satish V
Тема Re: Re-[Sonarqube]- Clumsy code
Дата
Msg-id CAOEjfmG080V9mhKWUjqTtxC48NgzucyUsipFG64FE9m+GabXxg@mail.gmail.com
обсуждение исходный текст
Ответ на Re: Re-[Sonarqube]- Clumsy code  (Akshay Joshi <akshay.joshi@enterprisedb.com>)
Ответы Re: Re-[Sonarqube]- Clumsy code
Список pgadmin-hackers
Hi Akshay,

Please find the updated Patch in the attachment which does not introduce more code smell.

Thanks 
Sathish V

On Thu, Jun 18, 2020 at 12:48 PM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi Satish

Fixes like below will create another code smell in SonarQube:
  • !(m.label == 'pg_global') should be (m.label != 'pg_global')
  • !(_.isUndefined(index) || index == '');
  • !(_.has(itemData, 'label') && itemData.label === '_RETURN');
  • .....
Please fix such errors correctly and resend the patch. Make sure functionality should work properly.

On Thu, Jun 18, 2020 at 11:27 AM Satish V <satish.v@enterprisedb.com> wrote:
Hi Hackers,

Please find the patch in the attachment which resolves below issue.
1.Replace this if-then-else statement by a single return statement.

Kindy review and share the changes, if required

Thanks,
Sathish V


--
Thanks & Regards
Akshay Joshi
Sr. Software Architect
EnterpriseDB Software India Private Limited
Mobile: +91 976-788-8246
Вложения

В списке pgadmin-hackers по дате отправления:

Предыдущее
От: Aditya Toshniwal
Дата:
Сообщение: Re: [pgAdmin][patch] Sonarqube issues tagged as redundant
Следующее
От: Akshay Joshi
Дата:
Сообщение: pgAdmin 4 commit: Fixed 'Comma operator should not be used' issue repor