Re: [pgAdmin4] Changes required to support PyCrypto

2018-04-10 Thread Dave Page
On Tue, Apr 10, 2018 at 12:49 PM, Murtuza Zabuawala < murtuza.zabuaw...@enterprisedb.com> wrote: > Hi, > > Please find patch to revert back the changes we have made for pycryptodom > module, This will fix encrypt & decrypt functionality in python3. > Wait, what? Are you telling me that

Re: pgAdmin 3.0 builds

2018-04-10 Thread Fahar Abbas
Yes Akshay is not able to reproduce this issue on VMWare Fusion 6.0.6 while Khushboo is facing same issue on Windows 07 64. I am using VMware Fusion 10 version and facing issue on Windows 2012 and Windows 07 64 bit. Kindly suggest, how i can proceed further on this, should we document this?

pgAdmin 4 commit: Cleanup node modules when cleaning

2018-04-10 Thread Dave Page
Cleanup node modules when cleaning Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=be2ac25fed77963cf0e62834b074a3534e06acfa Modified Files -- Makefile | 5 - 1 file changed, 4 insertions(+), 1 deletion(-)

pgAdmin 4 commit: Ensure the query tool will open properly under MS Edg

2018-04-10 Thread Dave Page
Ensure the query tool will open properly under MS Edge. Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=f5e474e4ece886830c2c7a870bee1fa8af90f4a5 Author: Murtuza Zabuawala Modified Files --

pgAdmin 3.0 builds

2018-04-10 Thread Dave Page
The pgAdmin 3.0 builds are at https://developer.pgadmin.org/~dpage/v3.0/. Barring any show-stopper issues, these are what will be released on Thursday. As a reminder to packagers: - This release removes dependencies on QtWebKit and/or QtWebEngine, and uses the users default browser instead. -

Re: pgAdmin 3.0 builds

2018-04-10 Thread Sandeep Thakkar
Hi Dave, On Tue, Apr 10, 2018 at 4:21 PM, Dave Page wrote: > The pgAdmin 3.0 builds are at https://developer.pgadmin.org/~dpage/v3.0/. > > Barring any show-stopper issues, these are what will be released on > Thursday. > > As a reminder to packagers: > > - This release

Re: pgAdmin 3.0 builds

2018-04-10 Thread Dave Page
On Tue, Apr 10, 2018 at 12:52 PM, Sandeep Thakkar < sandeep.thak...@enterprisedb.com> wrote: > Hi Dave, > > On Tue, Apr 10, 2018 at 4:21 PM, Dave Page wrote: > >> The pgAdmin 3.0 builds are at https://developer.pgadmin.org/~dpage/v3.0/. >> >> Barring any show-stopper issues,

Re: pgAdmin 3.0 builds

2018-04-10 Thread Fahar Abbas
Hi Dave, I am still able to reproduce pgadmin4 crash issue on Windows 07 64 and Windows 2012 R2 64 on fresh VMs: https://redmine.postgresql.org/issues/3227 Kind Regards, On Tue, Apr 10, 2018 at 3:51 PM, Dave Page wrote: > The pgAdmin 3.0 builds are at

Re: pgAdmin 3.0 builds

2018-04-10 Thread Dave Page
On Tue, Apr 10, 2018 at 1:02 PM, Fahar Abbas wrote: > Yes Akshay is not able to reproduce this issue on VMWare Fusion 6.0.6 > while Khushboo is facing same issue on Windows 07 64. > > I am using VMware Fusion 10 version and facing issue on Windows 2012 and > Windows

Re: pgAdmin 3.0 builds

2018-04-10 Thread Dave Page
On Tue, Apr 10, 2018 at 12:54 PM, Fahar Abbas wrote: > Hi Dave, > > I am still able to reproduce pgadmin4 crash issue on Windows 07 64 and > Windows 2012 R2 64 on fresh VMs: > > https://redmine.postgresql.org/issues/3227 > Did you ever correlate it to VMware

[pgAdmin4][RM#3257] Fix for explain functionality

2018-04-10 Thread Murtuza Zabuawala
Hi Dave, A minor fix for explain functionality. *Issue:* Due to multiline explain options in the SQL template CodeMirror was failing to highlight the exact line of error. -- Regards, Murtuza Zabuawala EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company RM_3257.diff

[pgAdmin4] Changes required to support PyCrypto

2018-04-10 Thread Murtuza Zabuawala
Hi, Please find patch to revert back the changes we have made for pycryptodom module, This will fix encrypt & decrypt functionality in python3. -- Regards, Murtuza Zabuawala EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company reverting_back_changes_for_pycrypto.diff

Re: [pgAdmin4] Changes required to support PyCrypto

2018-04-10 Thread Murtuza Zabuawala
On Tue, Apr 10, 2018 at 5:27 PM, Dave Page wrote: > > > On Tue, Apr 10, 2018 at 12:49 PM, Murtuza Zabuawala < > murtuza.zabuaw...@enterprisedb.com> wrote: > >> Hi, >> >> Please find patch to revert back the changes we have made for pycryptodom >> module, This will fix encrypt

Re: [pgAdmin4][RM#3257] Fix for explain functionality

2018-04-10 Thread Joao De Almeida Pereira
Hi Murtuza, Looks like the tests are failing due to this change. I also got some strange result on the linter, but I believe it is something related to this machine. Thanks Joao On Tue, Apr 10, 2018 at 6:20 AM Murtuza Zabuawala < murtuza.zabuaw...@enterprisedb.com> wrote: > Hi Dave, > > A minor

Re: pgAdmin 3.0 builds

2018-04-10 Thread Dave Page
Hi Windows 7 Professional here. On Tue, Apr 10, 2018 at 3:50 PM, Fahar Abbas wrote: > Hi Murtuza, > > I am using Windows 07 64 ultimate and updates are already applied and > crash is still reproducible when change default browsers to chrome. So you > are using

Re: pgAdmin 3.0 builds

2018-04-10 Thread Dave Page
On Tue, Apr 10, 2018 at 2:36 PM, Khushboo Vashi < khushboo.va...@enterprisedb.com> wrote: > > > On Tue, Apr 10, 2018 at 6:15 PM, Fahar Abbas > wrote: > >> Hi Neel, >> >> On Tue, Apr 10, 2018 at 5:43 PM, Neel Patel >> wrote: >> >>> Hi,

Re: pgAdmin 3.0 builds

2018-04-10 Thread Khushboo Vashi
On Tue, Apr 10, 2018 at 6:15 PM, Fahar Abbas wrote: > Hi Neel, > > On Tue, Apr 10, 2018 at 5:43 PM, Neel Patel > wrote: > >> Hi, >> >> On Tue, Apr 10, 2018 at 5:44 PM, Dave Page wrote: >> >>> >>> >>> On Tue, Apr 10,

Re: pgAdmin 3.0 builds

2018-04-10 Thread Neel Patel
Hi, On Tue, Apr 10, 2018 at 5:44 PM, Dave Page wrote: > > > On Tue, Apr 10, 2018 at 1:02 PM, Fahar Abbas > wrote: > >> Yes Akshay is not able to reproduce this issue on VMWare Fusion 6.0.6 >> while Khushboo is facing same issue on Windows 07

Re: pgAdmin 3.0 builds

2018-04-10 Thread Khushboo Vashi
On Tue, 10 Apr 2018, 20:20 Fahar Abbas, wrote: > Hi Murtuza, > > I am using Windows 07 64 ultimate and updates are already applied and > crash is still reproducible when change default browsers to chrome. So you > are using Windows 07 Home Premium as per the

Re: pgAdmin 3.0 builds

2018-04-10 Thread Fahar Abbas
Hi Neel, On Tue, Apr 10, 2018 at 5:43 PM, Neel Patel wrote: > Hi, > > On Tue, Apr 10, 2018 at 5:44 PM, Dave Page wrote: > >> >> >> On Tue, Apr 10, 2018 at 1:02 PM, Fahar Abbas < >> fahar.ab...@enterprisedb.com> wrote: >> >>> Yes Akshay is not

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree

2018-04-10 Thread Joao De Almeida Pereira
Hello Khushboo On Mon, Apr 9, 2018 at 1:59 AM Khushboo Vashi < khushboo.va...@enterprisedb.com> wrote: > Hi Joao, > > I have reviewed your patch and have some suggestions. > > On Sat, Apr 7, 2018 at 12:43 AM, Joao De Almeida Pereira < > jdealmeidapere...@pivotal.io> wrote: > >> Hello

Re: pgAdmin 3.0 builds

2018-04-10 Thread Murtuza Zabuawala
I don't think sub version matters here. On Tue, 10 Apr 2018, 22:08 Khushboo Vashi, wrote: > > > On Tue, 10 Apr 2018, 20:20 Fahar Abbas, > wrote: > >> Hi Murtuza, >> >> I am using Windows 07 64 ultimate and updates are already

Re: [pgAdmin4] Changes required to support PyCrypto

2018-04-10 Thread Dave Page
*grumble*. Thanks, applied. On Tue, Apr 10, 2018 at 1:52 PM, Murtuza Zabuawala < murtuza.zabuaw...@enterprisedb.com> wrote: > On Tue, Apr 10, 2018 at 5:27 PM, Dave Page wrote: > >> >> >> On Tue, Apr 10, 2018 at 12:49 PM, Murtuza Zabuawala < >>

pgAdmin 4 commit: Fix non-ASCII password decryption/encryption under Py

2018-04-10 Thread Dave Page
Fix non-ASCII password decryption/encryption under Python 3 Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=d0a640442cd6dc7bbfdb2647cf1775c13ad79a96 Author: Murtuza Zabuawala Modified Files --

Re: [pgAdmin4] Changes required to support PyCrypto

2018-04-10 Thread Dave Page
On Tue, Apr 10, 2018 at 4:51 PM, Dave Page wrote: > *grumble*. Thanks, applied. > And reverted. When I went back to 2.7, it got very upset: ERROR runTest (pgadmin.browser.server_groups.servers.databases.schemas.tables.tests.test_table_put.TableUpdateTestCase) Create

Re: [pgAdmin4] Changes required to support PyCrypto

2018-04-10 Thread Murtuza Zabuawala
​This is the exact code we had before pycrptodom changes, Is it fresh virtual environment? I suspect ​some conflicts from pycrptodom files. -- Regards, Murtuza Zabuawala EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company On Tue, Apr 10, 2018 at 9:51 PM, Dave Page

Re: [pgAdmin4][RM#3257] Fix for explain functionality

2018-04-10 Thread Murtuza Zabuawala
Thanks for reviewing Joao. Yeah missed to run tests. I assumed its just a SQL formatting it wouldn't hurt anything but I was wrong :) Here's updated patch. -- Regards, Murtuza Zabuawala EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company On Tue, Apr 10, 2018 at 8:18 PM,

Ignoring web/yarn-error.log and runtime/moc_LogWindow.cpp

2018-04-10 Thread Rahul Soshte
These are generated files. We can choose to ignore them. diff --git a/.gitignore b/.gitignore index 467505c7..3bec41ff 100644 --- a/.gitignore +++ b/.gitignore @@ -40,3 +40,4 @@ web/regression/test_config.json node_modules/ web/pgAdmin/static/js/generated web/pgadmin/static/js/generated

pgAdmin 4 commit: Revert "Fix non-ASCII password decryption/encryption

2018-04-10 Thread Dave Page
Revert "Fix non-ASCII password decryption/encryption under Python 3" This reverts commit d0a640442cd6dc7bbfdb2647cf1775c13ad79a96. Seems to anger Python 2.7 Branch -- master Details ---

Re: [pgAdmin4][Patch][Feature #1998] Appends .sql if extension not given when using 'save' or 'save as' feature

2018-04-10 Thread Rahul Soshte
I am trying to access the value in the format combobox for selected file type to put it here.. This snippet happens to be in web/pgadmin/tools/sqleditor/static/js/sqleditor.js How do I get access to that format combobox value to determine whether it is 'sql' or 'All Files' ? Also What is e

pgAdmin 4 commit: Don't use pycryptodome at the moment, as it doesn't p

2018-04-10 Thread Dave Page
Don't use pycryptodome at the moment, as it doesn't play well on Windows. Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=051fd645d7f134af83215f783a155c8ef6db184b Modified Files -- requirements.txt | 3 +-- 1 file changed, 1

pgAdmin 4 commit: Only avoid pre-built binaries for psycopg2 when build

2018-04-10 Thread Dave Page
Only avoid pre-built binaries for psycopg2 when building the Mac package, as some others (e.g. pbr) don't like being forced to source builds at the moment. Branch -- master Details ---

Re: pgAdmin 3.0 builds

2018-04-10 Thread Fahar Abbas
Hi Dave/Team! I just downloaded fresh iso and created new VM of Windows 2012 R2 server from MSDN subscription and crash issue is still reproduce. Now checking on Windows 07 Home basic 32 bit and Windows 07 Professional 64 bit. Kind Regards On Tue, Apr 10, 2018 at 10:15 PM, Murtuza Zabuawala <

Re: pgAdmin 3.0 builds

2018-04-10 Thread Fahar Abbas
Hi Dave, Now i know the actual root-cause of the issue. After further investigation, i realized that pgAdmin4 applications works fine on same Windows 2012 R2 and Windows 07 64 bit VM once Administrator user does not contain any space. Normally I use Fahar Abbas with Administrator user privileges

Re: [pgAdmin4][Patch][Feature #1998] Appends .sql if extension not given when using 'save' or 'save as' feature

2018-04-10 Thread Murtuza Zabuawala
Hi, We are passing the file name when we trigger an event from File manager whenever user selects any file ../pgadmin/misc/file_manager/static/js/file_manager.js Line:302 OR creates any new file ../pgadmin/misc/file_manager/static/js/file_manager.js Line:583, Line:678 You can use jQuery