Re: Can someone tell me what this code does ?

From: Murtuza Zabuawala <murtuza(dot)zabuawala(at)enterprisedb(dot)com>
To: Dave Cramer <davecramer(at)gmail(dot)com>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: Can someone tell me what this code does ?
Date: 2017-08-10 06:25:04
Message-ID: CAKKotZRgimD1BzVqgw4-Cutf12tUTsWtvRrvqEoDfdVS+D+0UQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi,

On Wed, Aug 9, 2017 at 8:19 PM, Dave Cramer <davecramer(at)gmail(dot)com> wrote:

> So what is the policy of the project? Ideally the cruft should be removed.
>
> Would a patch doing so be committed?
>
​Yes, I guess.​

Dave Cramer
>
> On 9 August 2017 at 00:28, Murtuza Zabuawala <murtuza.zabuawala@
> enterprisedb.com> wrote:
>
>> Yes, you are correct, at a moment that's all it does.
>>
>> On Tue, Aug 8, 2017 at 6:37 PM, Dave Cramer <davecramer(at)gmail(dot)com> wrote:
>>
>>> Hi,
>>>
>>> I guess my question was a bit vague. I get that it loads drivers. But
>>> note it does not actually put them anywhere.
>>>
>>> First it creates a dict
>>> sets the attribute in the app
>>> loads the drivers dynamically
>>> and returns an empty dict.
>>>
>>> From what I can tell this:
>>>
>>> DriverRegistry.load_drivers()
>>>
>>> is all it does?
>>>
>>>
>>>
>>>
>>>
>>>
>>> Dave Cramer
>>>
>>> On 7 August 2017 at 23:35, Murtuza Zabuawala <
>>> murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
>>>
>>>> Hi,
>>>>
>>>> This piece of code allow us to dynamically import all the available
>>>> driver modules from '../utils/driver/' directory into our application.
>>>>
>>>> --
>>>> Regards,
>>>> Murtuza Zabuawala
>>>> EnterpriseDB: http://www.enterprisedb.com
>>>> The Enterprise PostgreSQL Company
>>>>
>>>> [image: https://community.postgresrocks.net/]
>>>> <https://community.postgresrocks.net/>
>>>>
>>>> On Tue, Aug 8, 2017 at 4:20 AM, Dave Cramer <davecramer(at)gmail(dot)com>
>>>> wrote:
>>>>
>>>>> I'm fairly new to Python so excuse my naiveté.
>>>>>
>>>>> This code: in web/pgadmin/utils/driver/__init__.py does not appear
>>>>> to load the drivers into the drivers dictionary ? Or am I missing something
>>>>>
>>>>> def init_app(app):
>>>>> drivers = dict()
>>>>>
>>>>> setattr(app, '_pgadmin_server_drivers', drivers)
>>>>> DriverRegistry.load_drivers()
>>>>>
>>>>> return drivers
>>>>>
>>>>>
>>>>>
>>>>> Dave Cramer
>>>>>
>>>>
>>>>
>>>
>>
>

In response to

Browse pgadmin-hackers by date

  From Date Subject
Next Message Wenlin Zhang 2017-08-10 06:31:28 Re: [pgAdmin4][PATCH] Refactor and change of implementation of keyboard_shortcuts function dependencies
Previous Message Surinder Kumar 2017-08-10 06:00:31 Re: [pgAdmin4][patch] update the alert style in the sub-navigation