The changes to fix 4520 are in python side. I would suggest if you can separate it out from this patch, so that we can ask the users(affected by 4520) to just replace the python files in their installed pgAdmin4. This fix will not be available till next release, but we can give the workaround files. Most users are unable to upgrade psycopg2 on their systems.
Agreed. As a general rule, we try to keep to one fix per patch/commit as it makes it much easier to understand changes, especially after some time. Having a stated dependency between patches is fine if that's necessary.
Can you split the changes up please?
Thanks.
On Tue, Jul 30, 2019 at 8:10 PM Yosry Muhammad <yosrym93@gmail.com> wrote:
Hi,
Please find attached a patch with the following bug fixes for the query tool:
- View data not working correctly when the table/view does not contain any columns (#4492).
- Query tool does not produce results when psycopg2 version is below 2.8 (#4520).
- Command tags and number of affected rows not shown in the query result messages (#4525 and #4529 - duplicates).
Note that:
#4524 and #4522 are feature requests for already existing features.