Hi,
Issue caused because variable temp_new_rows was access before it was
initialised. Attached patch fixes this issue.
--
*Harshal Dhumal*
*Sr. Software Engineer*
EnterpriseDB India: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
On Mon, May 15, 2017 at 9:28 PM, Dave Page <dp...@pgadmin.org> wrote:
> Aww, nuts. I thought we had tests for that? Did we miss that particular
> case?
>
> Anyway, copying of a single value still works (with Cmd+C), so I'm not
> inclined to re-wrap everything for this unless someone can get a patch to
> me by ~9AM UK time tomorrow. Surinder?
>
> On Mon, May 15, 2017 at 4:50 PM, Joao Pedro De Almeida Pereira <
> jdealmeidapere...@pivotal.io> wrote:
>
>> Hi hackers,
>>
>> We were checking out the selection functionality after this commit--looks
>> like selection no longer works for single cells.
>>
>> If you press a cell in the grid the following message is console logged:
>>
>> sqleditor.js:869 Uncaught TypeError: Cannot read property 'indexOf' of
>> undefined
>> at SlickGrid.<anonymous> (sqleditor.js:869)
>> at Event.notify (slick.core.js:143)
>> at trigger (slick.grid.js:1067)
>> at setActiveCellInternal (slick.grid.js:2693)
>> at HTMLDivElement.handleClick (slick.grid.js:2469)
>> at HTMLDivElement.dispatch (jquery-1.11.2.js:4665)
>> at HTMLDivElement.$event.dispatch (jquery.event.drag-2.2.js:374)
>> at HTMLDivElement.elemData.handle (jquery-1.11.2.js:4333)
>>
>>
>>
>> Thanks
>> George & Joao
>>
>> On Mon, May 15, 2017 at 10:05 AM, Dave Page <dp...@pgadmin.org> wrote:
>>
>>> I've committed a modified version of this patch following some
>>> discussion on IM with Surinder.
>>>
>>> Thanks Surinder!
>>>
>>> On Mon, May 15, 2017 at 10:56 AM, Surinder Kumar <
>>> surinder.ku...@enterprisedb.com> wrote:
>>>
>>>> Hi
>>>>
>>>> The regression test cases for 'copy row' was failing.
>>>> This patch was assuming that each selected row will have primary key
>>>> due to selection was not working. Fixed.
>>>>
>>>> Please find revised patch.
>>>>
>>>> Thanks
>>>> Surinder Kumar
>>>>
>>>> On Mon, May 15, 2017 at 1:13 PM, Surinder Kumar <
>>>> surinder.ku...@enterprisedb.com> wrote:
>>>>
>>>>> Hi,
>>>>>
>>>>> This patch contains following changes:
>>>>>
>>>>> 1) When a new row is added, allow to delete if changes are not saved.
>>>>>
>>>>> 2) Disable new row selection if primary key is not given but changes
>>>>> are saved on server.
>>>>>
>>>>> 3) Copy and paste one or more rows, then add new row doesn't
>>>>> work(entered value doesn't appear).
>>>>>
>>>>> 4) After deleting a row, add new row doesn't work.
>>>>>
>>>>> 5) New row added with explicitly given primary key should not disabled.
>>>>>
>>>>>
>>>>> Please review.
>>>>>
>>>>> Thanks,
>>>>> Surinder Kumar
>>>>>
>>>>
>>>>
>>>>
>>>> --
>>>> Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
>>>> To make changes to your subscription:
>>>> http://www.postgresql.org/mailpref/pgadmin-hackers
>>>>
>>>>
>>>
>>>
>>> --
>>> Dave Page
>>> Blog: http://pgsnake.blogspot.com
>>> Twitter: @pgsnake
>>>
>>> EnterpriseDB UK: http://www.enterprisedb.com
>>> The Enterprise PostgreSQL Company
>>>
>>
>>
>
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
diff --git a/web/pgadmin/tools/sqleditor/templates/sqleditor/js/sqleditor.js b/web/pgadmin/tools/sqleditor/templates/sqleditor/js/sqleditor.js
index f3ca7b0..82b966f 100644
--- a/web/pgadmin/tools/sqleditor/templates/sqleditor/js/sqleditor.js
+++ b/web/pgadmin/tools/sqleditor/templates/sqleditor/js/sqleditor.js
@@ -1597,7 +1597,8 @@ define(
self.explain_costs = false;
self.explain_buffers = false;
self.explain_timing = false;
- self.is_new_browser_tab = is_new_browser_tab;
+ self.is_new_browser_tab = is_new_browser_tab
+ self.temp_new_rows = [];
// We do not allow to call the start multiple times.
if (self.gridView)
--
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers
