Hi Yosry, This breaks the reconnect for query tool. Open a query tool, execute some query and then restart the python server. Go to the query tool and click execute. It will show a warning, on continuing it should connect again. It throws exception in browser console: Uncaught TypeError: Cannot read property 'apply' of undefined at Object.eval (VM69935 sqleditor.js:1769) at Object.callback (alertify.js:3347) at triggerCallback (alertify.js:1220) at Object.buttonsClickHandler (alertify.js:1241) at HTMLDivElement.eval (alertify.js:299)
On Thu, Aug 22, 2019 at 11:44 PM Yosry Muhammad <yosry...@gmail.com> wrote: > Please find an updated patch attached. > > On Mon, Aug 19, 2019 at 9:54 AM Yosry Muhammad <yosry...@gmail.com> wrote: > >> Jasmine tests passed on my machine, I will take another look once I have >> access to my machine. >> >> On Mon, Aug 19, 2019, 7:57 AM Akshay Joshi <akshay.jo...@enterprisedb.com> >> wrote: >> >>> Hi Yosry >>> >>> Jasmine tests are failing, can you please fix those and resend the >>> patch. >>> >>> On Fri, Aug 16, 2019 at 11:23 PM Yosry Muhammad <yosry...@gmail.com> >>> wrote: >>> >>>> Hi hackers, >>>> >>>> Please find attached a patch with minimal refactoring of: >>>> web/pgadmin/tools/sqleditor/static/js/sqleditor.js >>>> >>>> This includes merging 2 redundant functions into one and renaming some >>>> functions to have more expressive and consistent names. >>>> >>>> Please review ! >>>> Thanks. >>>> -- >>>> *Yosry Muhammad Yosry* >>>> >>>> Computer Engineering student, >>>> The Faculty of Engineering, >>>> Cairo University (2021). >>>> Class representative of CMP 2021. >>>> https://www.linkedin.com/in/yosrym93/ >>>> >>> >>> >>> -- >>> *Thanks & Regards* >>> *Akshay Joshi* >>> >>> *Sr. Software Architect* >>> *EnterpriseDB Software India Private Limited* >>> *Mobile: +91 976-788-8246* >>> >> > > -- > *Yosry Muhammad Yosry* > > Computer Engineering student, > The Faculty of Engineering, > Cairo University (2021). > Class representative of CMP 2021. > https://www.linkedin.com/in/yosrym93/ > -- Thanks and Regards, Aditya Toshniwal Software Engineer | EnterpriseDB India | Pune "Don't Complain about Heat, Plant a TREE"