Ticket #2301 (closed tweak: obsolete)

Opened 9 years ago

Last modified 8 years ago

copy-paste-files-to – Should be able to paste files in Sophie

Reported by: deyan Owned by: deni
Priority: major Milestone: X3
Component: uncategorized Version: 2.0
Keywords: Cc:
Category: unknown Effort:
Importance: Ticket_group:
Estimated Number of Hours: 0 Add Hours to Ticket: 0
Billable?: yes Total Hours: 0
Analysis_owners: deyan Design_owners: deni
Imp._owners: deni Test_owners:
Analysis_reviewers: deni Changelog:
Design_reviewers: meddle Imp._reviewers: meddle, todor, pap
Test_reviewers: Analysis_score: 4
Design_score: 4 Imp._score: 4
Test_score: 0

Description

Should be able to paste files into Sophie. The behavior should be like it is for the drag and drop from the OS.

Attachments

copy-paste-files-to.patch (7.2 KB) - added by deni 9 years ago.

Change History

comment:1 Changed 9 years ago by deni

  • Category set to unknown
  • Design_owners set to deni
  • Imp._owners set to deni
  • Analysis_reviewers set to deni
  • Analysis_score set to 4
  • Test_score set to 0
  • Design_score set to 0
  • Status changed from new to s1a_analysis_started
  • Owner set to deni
  • Imp._score set to 0

comment:2 Changed 9 years ago by deni

  • Status changed from s1a_analysis_started to s1b_analysis_finished

comment:3 Changed 9 years ago by deni

  • Status changed from s1b_analysis_finished to s1c_analysis_ok

comment:4 Changed 9 years ago by deni

  • Status changed from s1c_analysis_ok to s2a_design_started

comment:5 Changed 9 years ago by deni

  • Status changed from s2a_design_started to s2b_design_finished
  • Move the logic for inserting resources from files to new frames from ResourceImportLogic.ON_DROP_FILE to a helper method - private static void importFilesInNewFrames(FileListData, PageWorkArea, ImmPoint)
  • Create a new operation in ResourceImportLogic - ON_PASTE_FILE which handles pasting one or multiple files in the current edit scope.
    • Its source class is VisualElement and its eventId - SophieDragDropHandler.TransferEventIds.PASTE.
    • Its handle method extracts the necessary information from the clipboard and calls the importFilesInNewFrames.

comment:6 Changed 9 years ago by deni

  • Status changed from s2b_design_finished to s2c_design_ok

comment:7 Changed 9 years ago by deni

  • Status changed from s2c_design_ok to s3a_implementation_started

Changed 9 years ago by deni

comment:8 Changed 9 years ago by deni

  • Status changed from s3a_implementation_started to s3b_implementation_finished

comment:9 Changed 9 years ago by meddle

  • Status changed from s3b_implementation_finished to s3c_implementation_ok
  • Design_reviewers set to meddle
  • Imp._reviewers set to meddle, todor, pap

Merged to the trunk at [8618].

You should have extracted the anonymous class for matching the import provider into a true inner one, I know that that is old code, but you can make the code `uality better :)

4p

comment:10 Changed 9 years ago by meddle

  • Design_score changed from 0 to 4
  • Imp._score changed from 0 to 4

comment:11 Changed 8 years ago by meddle

  • Status changed from s3c_implementation_ok to closed
  • Resolution set to obsolete

Closing all the tickets before M Y1

Note: See TracTickets for help on using tickets.