Re: Refactor: Registry Classes

From: Ashesh Vashi <ashesh(dot)vashi(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: Refactor: Registry Classes
Date: 2021-06-19 05:57:19
Message-ID: CAG7mmoxQfhVhGUC2ArMkCPUoWiBusu-z6=0PR8OiZfOMQpnWwg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

On Sat, Jun 19, 2021 at 1:39 AM Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com>
wrote:

> Hi Akshay,
>
> We do have a couple of classes, which does automatic registration of the
> base classes,
> and which creates single-ton objects for these base classes, when needed.
>
> I would be working on a patch sooner, which will be using similar
> functionality for loading
> the multi-factor authentication.
>
> I realized - it will be a duplicate code at three places for the same
> functionalities.
> Hence - I worked on refactoring this registry class.
>
> Please find the patch for the same.
>
Found issues - some test files were using the old function
'Driver.load_drivers(...)'.
They're fixed now.

-- Thanks, Ashesh

>
> --
>
> Thanks & Regards,
>
> Ashesh Vashi
> EnterpriseDB INDIA: Enterprise PostgreSQL Company
> <http://www.enterprisedb.com>
>
>
> *http://www.linkedin.com/in/asheshvashi*
> <http://www.linkedin.com/in/asheshvashi>
>

Attachment Content-Type Size
refactor_registry_class_v2.patch application/octet-stream 19.7 KB

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Aditya Toshniwal 2021-06-21 04:05:13 Re: [pgAdmin][RM6130] React based framework for properties dialog and port Server Group, Server and Database dialogs
Previous Message Ashesh Vashi 2021-06-18 20:09:47 Refactor: Registry Classes