From: | Yosry Muhammad <yosrym93(at)gmail(dot)com> |
---|---|
To: | Avin Kavish <avinkavish(at)gmail(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>, Dave Page <dpage(at)pgadmin(dot)org> |
Subject: | Re: JavaScript Errors in the Query Tool Leads to a False Message |
Date: | 2019-08-06 04:41:59 |
Message-ID: | CAFSMqn-FsykmZ3Z4-88PpbzSGb15Wk+HxB28OpBNu+5Eqrox2w@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Yes, if you undo this commit:
This was a fix of a bug that occurred when querying a table with no
columns. It displayed the same error mentioned in the original email and
was due to a JS error.
You can either checkout a previous commit or simply undo the difference in
sqleditor.js manually, it is only 2 lines of code. Then try querying a
table with no columns.
Your explanation seems legit to me but I am no expert in JS.
On Tue, Aug 6, 2019, 6:36 AM Avin Kavish <avinkavish(at)gmail(dot)com> wrote:
> Is there a way to reproduce this? i.e force a js error during query
> execution? One reason I can think of is using $.ajax({ async: false }) so
> the ajax request gets handled on the main thread and anything that throws
> on to the main thread gets caught.
>
> On Tue, Aug 6, 2019 at 4:07 AM Yosry Muhammad <yosrym93(at)gmail(dot)com> wrote:
>
>> Hi Hackers,
>>
>> I have noticed a strange behavior in the JS code of the Query Tool. When
>> a JS error occurs at any point during the execution of a query, the code is
>> traced back to the last ajax call and goes to the catch portion (as if the
>> ajax call failed). This leads to a "Not connected to the server" error
>> message to the user, which is wrong.
>>
>> I noticed this behavior before when fixing the bug where "Not connected
>> to the server" message appeared when the user queries a column with no
>> columns, which was due to a JS error being mistakenly identified as an ajax
>> error.
>>
>> Does anybody have any idea why this happens? This doesn't seem right.
>>
>> 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/
>>
>
From | Date | Subject | |
---|---|---|---|
Next Message | Yosry Muhammad | 2019-08-06 04:44:06 | Re: JavaScript Errors in the Query Tool Leads to a False Message |
Previous Message | Aditya Toshniwal | 2019-08-06 04:41:39 | Re: JavaScript Errors in the Query Tool Leads to a False Message |