Re: [pgadmin-hackers] pgAdmin 4 commit: Improve handling of nulls anddefault values in the d - Mailing list pgadmin-hackers

From Dave Page
Subject Re: [pgadmin-hackers] pgAdmin 4 commit: Improve handling of nulls anddefault values in the d
Date
Msg-id CA+OCxoydonNXGzXe5LrzGG+XSisEcYDhOTENe0F3hd5xjmERww@mail.gmail.com
Whole thread Raw
In response to Re: [pgadmin-hackers] pgAdmin 4 commit: Improve handling of nulls anddefault values in the d  (Harshal Dhumal <harshal.dhumal@enterprisedb.com>)
Responses Re: [pgadmin-hackers] pgAdmin 4 commit: Improve handling of nulls anddefault values in the d
List pgadmin-hackers


On Fri, May 12, 2017 at 12:19 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:
Hi,

Below code snippet from above commit assumes that we have to disable last row after saving.

@@ -2320,6 +2340,10 @@ define(
                       grid.setSelectedRows([]);
                     }
 
+                    // Add last row(new row) to keep track of it
+                    if (is_added) {
+                      self.rows_to_disable.push(grid.getDataLength()-1);
+                    }

 However this is not the case all the time

For e.g 
Table has some data already and If user adds new row (do not save) and then copy past few exiting rows (at this point  newly added row no longer remains at last position). And after saving it disable last row which is not the newly added row.

Hmm, good point. Can you/Surinder work up a fix for that in time for the release please? 

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

pgadmin-hackers by date:

Previous
From: Harshal Dhumal
Date:
Subject: Re: [pgadmin-hackers] pgAdmin 4 commit: Improve handling of nulls anddefault values in the d
Next
From: Surinder Kumar
Date:
Subject: Re: [pgadmin-hackers] pgAdmin 4 commit: Improve handling of nulls anddefault values in the d