Ticket #2397 (closed bug: obsolete)

Opened 15 years ago

Last modified 13 years ago

Style change crashes the application

Reported by: dido Owned by:
Priority: major Milestone:
Component: TEXT_EDITING Version: 2.0
Keywords: text Cc:
Category: END_PROD Effort:
Importance: 70 Ticket_group:
Estimated Number of Hours: 0 Add Hours to Ticket: 0
Billable?: yes Total Hours: 0
Analysis_owners: dido Design_owners: diana
Imp._owners: diana Test_owners:
Analysis_reviewers: diana Changelog: Changelog
Design_reviewers: pap Imp._reviewers: todor, pap
Test_reviewers: Analysis_score: 0
Design_score: 4 Imp._score: 3.5
Test_score: 0

Description

Step to reproduce

  • Create a book with text frame and fill some content inside
  • Save the book
  • Close it
  • Open it
  • Select some text
  • Open character hud
  • Change one of the styles (make the text bold or italic for example)

As result the application crashes.
NOTE: The above is true only if you first change the style of the text. If after opening the book users do some change before that one on the style, for example move the frame or add another etc. the crash won't appear.
Attached is the exception from the bug report form and a book that could be used for reproducing the bug (first two steps are done).

Attachments

bugFormOutput.txt​ (205.5 KB) - added by dido 15 years ago.
defaultTextFrame.book.s2​ (1.9 KB) - added by dido 15 years ago.
2397.patch​ (729 bytes) - added by diana 15 years ago.

Change History

Changed 15 years ago by dido

Changed 15 years ago by dido

comment:1 Changed 15 years ago by dido

  • Status changed from new to s1b_analysis_finished

comment:2 Changed 15 years ago by diana

  • Design_owners set to diana
  • Status changed from s1b_analysis_finished to s2a_design_started
  • Imp._owners set to diana

Changed 15 years ago by diana

comment:3 Changed 15 years ago by diana

  • Status changed from s2a_design_started to s3b_implementation_finished
  • In HotTextLogic:SET_TEXT:handle check if the last change made is null (it is null because the history is lost when the book is open and getLastChange returns null) .

comment:4 Changed 15 years ago by pap

  • Status changed from s3b_implementation_finished to s3c_implementation_ok
  • Analysis_reviewers set to diana
  • Imp._reviewers set to todor, pap
  • Design_score changed from 0 to 4
  • Design_reviewers set to pap
  • Imp._score changed from 0 to 3.5
  • Works correctly and commited in [9018].
  • You should've extracted the last change into a local variable. This makes the code shorter and easier to read and understand.
  • Oh and you should write analysis_reviewer score and a short note.

comment:5 Changed 13 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.