Re: [pgAdmin4][RM3294] User need to reset the layout to see thechanged preferences parameters

Hi Akshay,

On Fri, Jun 29, 2018 at 3:42 PM, Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:

Hi Aditya 

I have applied your patch and run pgAdmin4. I have found following two issue in the browser:

• Found error while open Preferences dialog. Refer Open_Preferences_Dialog.png

​This error occurs even with the latest pull without changes.​

 

• Set the preferences setting "Open in new browser tab" to True and open the query tool. Refer "Open_In_New_Broswer.png".

​Will look into this.​

 

I haven't review the code.

On Thu, Jun 28, 2018 at 8:04 PM, Aditya Toshniwal <aditya.toshniwal@enterprisedb.com> wrote:

Hi Hackers,

Attached is the patch for making preferences realtime and applying without reseting the layout. Please note, the patch is only for one module - SQL Editor and is the first part for the RM. There are lot of changes to be done to cover all and hence sending in parts. This will not affect/break existing code. Further patches will cover other modules.

Highlights of this patch include:

- Changes will affect SQL Editors in Create dialog boxes, SQL tab of the main screen, Query tool, History entries in the query tool, Query tool opened in New Tab/Window

- All the components of SQL editor will refer to single source of preferences which is cached in the Browser object. All other redundant ajax get preference calls are removed.

- SQL editor will not refer template JS variables anymore, once all the references are removed the template variables will also be removed.

- Code refactoring wherever possible.

- Covered JS test cases wherever possible.

Request you to kindly review.

--

Thanks and Regards,

Aditya Toshniwal

Software Engineer | EnterpriseDB Software Solutions | Pune

"Don't Complain about Heat, Plant a tree"

--

Akshay Joshi

Sr. Software Architect

Phone: +91 20-3058-9517
Mobile: +91 976-788-8246

--

Thanks and Regards,

Aditya Toshniwal

Software Engineer | EnterpriseDB Software Solutions | Pune

"Don't Complain about Heat, Plant a tree"