Opened 8 years ago

Closed 7 years ago

Last modified 6 years ago

#1068 closed defect (fixed)

Phonelog crashes on startup

Reported by: Beini Owned by: TAsn
Priority: major Milestone:
Component: pyphonelog Version: SHR-testing
Keywords: Cc:

Description

Phonelog
Initialized global vars
Initializing gtk interface
Showing main window
Upright
Started caching contacts
Done caching contacts
Started populating list
Started getting missed
Finished getting missed
Traceback (most recent call last):
  File "/usr/bin/phonelog", line 775, in <module>
    populateListByPageNum(incoming, outgoing, missed, general, startup_page)
  File "/usr/bin/phonelog", line 672, in populateListByPageNum
    missed.populateList()
  File "/usr/bin/phonelog", line 227, in populateList
    if status == 2 and int(call['New']) != 0:

It used to work, and I'm not sure when/why it started crashing (Could be after a crash or after a new call or..). Logs don't seem to contain anything related.

If you need more info, let me know.

Change History (10)

comment:1 in reply to: ↑ description Changed 8 years ago by Beini

Sorry, forgot the most important last line:

KeyError: 'New'

comment:2 Changed 8 years ago by Heinervdm

  • Milestone set to MS1

comment:3 Changed 8 years ago by TAsn

  • Resolution set to fixed in unstable
  • Status changed from new to closed

Fixed in 7672065
Will be built in unstable soon.

comment:4 Changed 8 years ago by Beini

Did the fix by hand in -testing. Phonelog starts correctly now. That was fast :) Thanks.

comment:5 follow-up: Changed 8 years ago by TAsn

Np.
Btw, this probably means your phonelog data is corrupted in some way (but not sure) as New is pretty standard.

comment:6 in reply to: ↑ 5 Changed 8 years ago by Beini

Replying to TAsn:

Np.
Btw, this probably means your phonelog data is corrupted in some way (but not sure) as New is pretty standard.

Indeed, one entry was missing the new field. Just adding this for record.

opimd-cli calls delete 50

Failed because because looks like opimd is also sure there is a "New" field. But after doing:

opimd-cli calls edit 50 New 0

opimd could delete the entry just fine. I think my phone crashed just after getting this missed call, that may be related.

comment:7 Changed 8 years ago by TAsn

Opimd does not need the New field for deleting a call. Actually, the delete is pretty generic, I wonder what went wrong. I wish you could have provided the previous database file for me to check this out.

comment:8 Changed 7 years ago by jama

  • Resolution fixed in unstable deleted
  • Status changed from closed to reopened

comment:9 Changed 7 years ago by jama

  • Resolution set to fixed
  • Status changed from reopened to closed

comment:10 Changed 6 years ago by morphis

  • Milestone MS1 deleted

Milestone MS1 deleted

Note: See TracTickets for help on using tickets.