public inbox for [email protected]  

help / color / mirror / Atom feed

From: Pradip Parkale <[email protected]>
To: pgadmin-hackers <[email protected]>
Subject: [pgAdmin][RM7296]: Browser tree is not getting refresh after deleting the multiple node from collection properties
Date: Fri, 8 Apr 2022 12:29:29 +0530
Message-ID: <CAJ9T6SvSy4jwPGgPJq=a+xC=-ujNaMuMdVEx8MONO+eYLhfmFg@mail.gmail.com> (raw)

Hi Hackers,

Please find the attached patch for #7296.I have fixed the issue where the
browser tree did not refresh after multiple nodes were deleted.

-- 
Thanks & Regards,
Pradip Parkale
Software Engineer | EnterpriseDB Corporation


Attachments:

  [application/octet-stream] RM7296.patch (859B, 3-RM7296.patch)
  download | inline diff:
diff --git a/web/pgadmin/misc/properties/CollectionNodeProperties.jsx b/web/pgadmin/misc/properties/CollectionNodeProperties.jsx
index 05aeef26d..9d340914d 100644
--- a/web/pgadmin/misc/properties/CollectionNodeProperties.jsx
+++ b/web/pgadmin/misc/properties/CollectionNodeProperties.jsx
@@ -162,12 +162,13 @@ export function CollectionNodeView({
           if (res.success == 0) {
             pgBrowser.report_error(res.errormsg, res.info);
           }
+          pgAdmin.Browser.tree.refresh(selItem);
           setReload(!reload);
         })
         .catch(function (error) {
           Notify.alert(
             gettext('Error dropping %s', selectedItemData._label.toLowerCase()),
-            error.response.data.errormsg
+            _.isUndefined(error.response) ? error.message : error.response.data.errormsg
           );
         });
     };

view thread (4+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected]
  Subject: Re: [pgAdmin][RM7296]: Browser tree is not getting refresh after deleting the multiple node from collection properties
  In-Reply-To: <CAJ9T6SvSy4jwPGgPJq=a+xC=-ujNaMuMdVEx8MONO+eYLhfmFg@mail.gmail.com>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox