Ticket #2077 (closed bug: fixed)

Opened 10 years ago

Last modified 10 years ago

When frame is on focus and click add page from the PPP exceprion is thrown.

Reported by: todor Owned by: mira
Priority: critical Milestone: M12_RELEASE
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: todor Design_owners: mira
Imp._owners: mira Test_owners:
Analysis_reviewers: mira Changelog:
Design_reviewers: Imp._reviewers:
Test_reviewers: Analysis_score: 0
Design_score: 3 Imp._score: 4
Test_score: 0

Description

  1. Create book.
  2. Insert frame.
  3. The frame is on focus. Now click add page from the PPP.
  4. Exception is thrown instead of creating a new page.

Change History

comment:1 Changed 10 years ago by mira

  • Status changed from new to s1b_analysis_finished
  • Analysis_reviewers set to mira

comment:2 Changed 10 years ago by mira

  • Status changed from s1b_analysis_finished to s2a_design_started

comment:3 Changed 10 years ago by todor

  • Status changed from s2a_design_started to s2b_design_finished

comment:4 Changed 10 years ago by mira

  • Status changed from s2b_design_finished to s2c_design_ok

This bug has been handled before for going to next page in author when there is no such page. It is only presented when the current page is last and you are adding page after the current. I believe there is a bug when computing controlPoint() of the FrameRotateHaloMenu but for now it is fixed and tracking it down is not high priority.

I will extract the common logic for adding a new page in a hepler method in BookView. It should be here, not in BookH since it is seting the current page to be the new page. I discused with the analizators this should always be the case. This helper method will clear the selected elements so that the location of the halo is not recomputed.

comment:5 Changed 10 years ago by mira

  • Owner set to mira
  • Status changed from s2c_design_ok to s3a_implementation_started

comment:6 Changed 10 years ago by mira

  • Design_owners set to mira
  • Status changed from s3a_implementation_started to s3b_implementation_finished
  • Imp._owners set to mira

Implemented and commited in a new branch - [7071]

comment:7 Changed 10 years ago by pap

  • Status changed from s3b_implementation_finished to s3c_implementation_ok
  • Design_score changed from 0 to 3
  • Imp._score changed from 0 to 4
  • OK but as you know this deselecting is not nice
  • Commited in [7089]

comment:8 Changed 10 years ago by todor

  • Status changed from s3c_implementation_ok to closed
  • Resolution set to fixed
Note: See TracTickets for help on using tickets.