Ticket #2355 (closed bug: obsolete)

Opened 2 years ago

Last modified 2 months ago

resource-delete -- Cannot delete resources from books

Reported by: deyan Owned by:
Priority: major Milestone: X3
Component: uncategorized Version: 2.0
Keywords: Cc:
Ticket_group: Effort:
Category: unknown Importance: 95
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:
Design_reviewers: meddle Analysis_reviewers: deni
Changelog: Changelog Imp._reviewers: deyan, todor, meddle
Test_reviewers: Analysis_score: 3.5
Design_score: 4 Imp._score: 4
Test_score: 0

Description

In 2.0.4 an exception is trown when trying to delete an unused resource from the resource palette

Attachments

resource-delete-exceptions.patch Download (4.3 KB) - added by deni 23 months ago.

Change History

Changed 2 years ago by deyan

  • status changed from new to s1b_analysis_finished

Changed 2 years ago by deni

  • status changed from s1b_analysis_finished to s2a_design_started

Changed 23 months ago by deni

  • design_owners set to deni
  • implementation_owners set to deni
  • analysis_score changed from 0 to 3.5
  • An exception is thrown sometimes, not always. Besides, there are different exceptions thrown in different situations.

Changed 23 months ago by deni

  • status changed from s2a_design_started to s3b_implementation_finished
  • An exception is sometimes thrown when we have several open books:
    • In ResourceDeleteLogic.ON_DELETE_RESOURCE we should use AppViewUtil.getCurrentBookDocView(palette) instead of palette.findNearestElement(null, BookDocView.class) to get the current book doc view.
  • An exception is thrown when trying to delete a group:
    • In GroupHalosLogic.ON_GROUP_DELETED the arguments passed to ResourceRefR4.getRelativeRef(ResourceRefR4, ResourceRefR4) should be swapped.
  • In CopyUtil.decapsulate(...) the reference passed when decapsulating the children resources was wrong - it should be the child reference appended to the current one, not just the child reference. This didn't cause any visible problems but it really was wrong :-).
  • I changed FileListData.textURIListToFileList(String) to use String.split(String) instead of StringTokenizer. Please test it on Linux.

Changed 23 months ago by deni

Changed 23 months ago by meddle

  • status changed from s3b_implementation_finished to s3c_implementation_ok
  • analysis_reviewers set to deni
  • implementation_reviewers set to deyan, todor, meddle
  • design_score changed from 0 to 4
  • design_reviewers set to meddle
  • implementation_score changed from 0 to 4

Merged to the trunk at [8854].
I think under Linux there is no problem now.
Good that you commented the analysis and mark it - so you are the analysis reviewer?

Changed 2 months 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.