Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 155428

Summary: Multi-row selection copy-cut/paste does not work in Datapool Editor.
Product: z_Archived Reporter: Alex Bernstein <alexberns>
Component: TPTPAssignee: Paul Slauenwhite <paulslau>
Status: CLOSED FIXED QA Contact:
Severity: major    
Priority: P2 CC: mayzhu, paulslau
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Windows XP   
Whiteboard: closed471
Bug Depends on: 283713    
Bug Blocks: 293690    
Attachments:
Description Flags
Patch. none

Description Alex Bernstein CLA 2006-08-28 13:01:22 EDT
If you select several cells and "copy" them, you cannot paste multiple selection. It does not matter if you select one cell or seveal cells, what gets pasted is only one source cell value.
Comment 1 Paul Slauenwhite CLA 2008-05-29 16:14:11 EDT
In addition:

[Defect #231820]:
Multi-select copy/cut and paste does not work in the datapool editor.

The user can select multiple cells (shaded) in the datapool editor, however one
cell is the active cell (unshaded with no border).  As a result, the context
menu is isolated to the active cell and the user cannot multi-select copy/cut
and paste.
Comment 2 Paul Slauenwhite CLA 2008-05-29 16:14:18 EDT
*** Bug 231820 has been marked as a duplicate of this bug. ***
Comment 3 Paul Slauenwhite CLA 2008-06-27 13:47:15 EDT
This defect is still reproducible in TPTP 4.5.0 (http://www.eclipse.org/tptp/home/downloads/?ver=4.5.0).  
Comment 4 Paul Slauenwhite CLA 2008-06-27 13:52:49 EDT
In addition, selecting a cell will outline the cell with a dotted box but the entire row is grayed (excepted the selected cell).  As well, selecting a cell with CTRL is pressed, all of the selected cells are grayed.  This should be removed unless the row number is selected (first column).  We should be able to select or added to an existing selection (using CTRL) individual cells.
Comment 5 Alex Bernstein CLA 2008-06-27 13:58:28 EDT
My understanding is that people expect a behavior from Datapool Editor to be similar, if not exactly the same, as that of any well-known spreadsheet application. Perhaps we can draw some user experiences from there?
Comment 6 Paul Slauenwhite CLA 2008-06-27 14:10:09 EDT
(In reply to comment #5)
> My understanding is that people expect a behavior from Datapool Editor to be
> similar, if not exactly the same, as that of any well-known spreadsheet
> application. Perhaps we can draw some user experiences from there?
> 

Yes, we can use a similar user experience.
Comment 7 Paul Slauenwhite CLA 2009-05-28 07:36:54 EDT

*** This bug has been marked as a duplicate of bug 252958 ***
Comment 8 Paul Slauenwhite CLA 2009-07-16 10:29:49 EDT
Reopening to cover the following issues (see 252958):

1) When selecting and copying multiple rows, only the first row is copied.
Comment 9 Paul Slauenwhite CLA 2009-07-16 10:42:38 EDT
Requires changes to /org.eclipse.hyades.test.doc.user/concepts/cdatapooleditor.htm to reflect the support added by this defect.
Comment 10 Paul Slauenwhite CLA 2009-08-18 15:06:31 EDT
In addition, support multi-line pasting.  That is, the user would create the new variable in the datapool, copy multiple cells from Excel or multiple lines from a text editor, selecting the same number of rows in the variable of the datapool, and pasting the multiple cells.  
Comment 11 Paul Slauenwhite CLA 2009-12-14 15:09:45 EST
Deferring to TPTP 4.7.0.
Comment 12 Paul Slauenwhite CLA 2010-03-22 12:23:19 EDT
This defect will resolve copy-cut/paste of multi-rows or records instead of multi-cells (see https://bugs.eclipse.org/bugs/show_bug.cgi?id=283713#c7).
Comment 13 Paul Slauenwhite CLA 2010-03-29 19:44:19 EDT
(In reply to comment #10)
> In addition, support multi-line pasting.  That is, the user would create the
> new variable in the datapool, copy multiple cells from Excel or multiple lines
> from a text editor, selecting the same number of rows in the variable of the
> datapool, and pasting the multiple cells.

This will only be supported for data copied from other datapool cells/rows since pasted data must be enclosed in <Literal></Literal> elements to determine cell delineation.
Comment 14 Paul Slauenwhite CLA 2010-03-29 20:03:42 EDT
Created attachment 163348 [details]
Patch.
Comment 15 Paul Slauenwhite CLA 2010-03-29 20:03:55 EDT
Added 5 new test cases and refactored 2 existing test cases in /org.eclipse.hyades.test.ui.datapool.tests/manual/datapool/Test.UI.DatapoolEditor_editing.testsuite to cover the symptoms in this defect.

Updated the Datapool Editor documentation.

The attached patch is checked into CVS (HEAD).
Comment 16 Kathy Chan CLA 2010-11-18 18:52:21 EST
As of TPTP 4.6.0, TPTP is in maintenance mode and focusing on improving quality by resolving relevant enhancements/defects and increasing test coverage through test creation, automation, Build Verification Tests (BVTs), and expanded run-time execution. As part of the TPTP Bugzilla housecleaning process (see http://wiki.eclipse.org/Bugzilla_Housecleaning_Processes), this enhancement/defect is verified/closed by the Project Lead since this enhancement/defect has been resolved and unverified for more than 1 year and considered to be fixed. If this enhancement/defect is still unresolved and reproducible in the latest TPTP release (http://www.eclipse.org/tptp/home/downloads/), please re-open.