hanyoud / android-daisy-epub-reader

Automatically exported from code.google.com/p/android-daisy-epub-reader
1 stars 0 forks source link

Player assumes all books will contain audio, it crashes for other book formats. #6

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. Try to read a text-only' book e.g. the book titled "WBU WIPO Copyright
Treaty (draft)" and downloadable from http://www.daisy.org/z3986/samples/#t1

r40 and earlier have some hard-coded assumptions every book and section
will contain audio elements. The software crashes with a 'Force close'
error dialog that repeats in a tight loop until the phone is powered down
and restarted.

What is the expected output? What do you see instead?

The reader should be able to correctly play a range of the more popular
DAISY formats - ideally all the supported formats. If it encounters an
unsupported format, or unsupported segments within a book, it should not crash.

I've started working on some improvements to the code: to support the text
only books, and to add some robustness to the code when it encounters
unsupported or problematic content.

Original issue reported on code.google.com by julianharty on 13 Dec 2009 at 5:43

GoogleCodeExporter commented 8 years ago
r44 is a partial fix for handling text-based DAISY books. However I've currently
broken the navigation and progression for audio content (See new issue
http://code.google.com/p/android-daisy-epub-reader/issues/detail?id=8 )

I've got to add a view to display the text and fix the stuff I've broken.

Original comment by julianharty on 13 Dec 2009 at 9:54

GoogleCodeExporter commented 8 years ago
Around r185, and certainly in the current version, the player now displays a 
message explaining it doesn't support full-text books. It doesn't crash anymore 
- so this issue can be maked as fixed. Note: we'd still like to support 
full-text Daisy books, however that's a separate feature.

Original comment by julianharty on 29 Dec 2010 at 12:11