Re: [pgAdmin4][Patch]: Dependecies/Dependents functionality

It seems to me that this patch is implemented in a way to violates modularity - specifically, it adds smarts about a child node into the code for a parent node, rather than leaving each node to be self-contained. For example, web/pgadmin/browser/server_groups/servers/depends.py contains the SQL and other logic for displaying the dependencies and dependents of tablespace and role nodes.

I would expect to see role-specific code confined to the roles module, database-specific confined to the database module and so on.