ailiboy / ebookdroid

Automatically exported from code.google.com/p/ebookdroid
0 stars 0 forks source link

Ошибка при нажатии кнопки «Просмотр» в gmail #121

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Захожу в письмо, содержащее pdf-файл.
2. Нажимаю кнопку Просмотр и выбираю в 
списке программ для просмотр ebookdroid.
3. Загружается программа и выдает ошибку 
«PDF-файл не найден или поврежден». Другие 
программы нормально открывают этот файл.

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

What version of the product are you using? On what operating system?
1.3, Android 3.2 

What device are you using?
Samsung Galaxy Tab II

Please provide any additional information below.

Original issue reported on code.google.com by abakurev...@gmail.com on 12 Jan 2012 at 4:37

GoogleCodeExporter commented 9 years ago
Сейчас мы не поддерживаем вложения в 
письмах.
Будет позже.

Original comment by mc.creat on 12 Jan 2012 at 5:08

GoogleCodeExporter commented 9 years ago

Original comment by Alexander.V.Kasatkin@gmail.com on 14 Jan 2012 at 1:52

GoogleCodeExporter commented 9 years ago
This should be easy by following 
http://carvingcode.blogspot.com/2009/08/how-to-open-gmail-attachments-with.html 
and looking at 
http://code.google.com/p/bites-android/source/browse/trunk/Bites/src/caldwell/be
n/bites/Bites.java#149  and 
http://code.google.com/p/card-catcher/source/browse/trunk/src/se/rende/cardcatch
er/CardCatcher.java#35

You basically have to load the file from an InputStream instead of an actual 
file.

Original comment by zhen...@gmail.com on 15 Jan 2012 at 12:37

GoogleCodeExporter commented 9 years ago
На новых прошивках глюк в маркете - оно все 
равно не установится.

Original comment by Alexander.V.Kasatkin@gmail.com on 15 Jan 2012 at 1:18

GoogleCodeExporter commented 9 years ago
See patch at [1], not too nice, but works for me with K9 Mail.
Btw, google says "Issue attachment storage quota exceeded."

[1] 
http://pinky.die-welt.net/~evgeni/tmp/0001-open-content-files-by-saving-them-to-
a-temporary-loc.patch

Original comment by zhen...@gmail.com on 15 Jan 2012 at 3:43

GoogleCodeExporter commented 9 years ago
Из-за того, что используются нативные 
библиотеки использовать InputStream 
проблематично.
Мы можем работать с полным путем к файлу 
или файловым дескриптором.
Раньше и сейчас мы получаем полный путь 
файла, но из-за ошибки в маркете, пришлось 
убрать permission для доступа к аттачам в 
письмах.
Уже есть наработки по получению 
дескриптора файла в нативной части, но еще 
работы много...
Т.е. сделаем, только позже. Скорее всего в 1.4. 
Но может и раньше...

Original comment by mc.creat on 15 Jan 2012 at 4:41

GoogleCodeExporter commented 9 years ago
That's why my patch takes the InputSteam, opens a temp file, writes the stream 
in it, and then calls the native viewer on the temp file.

Original comment by zhen...@gmail.com on 16 Jan 2012 at 8:25

GoogleCodeExporter commented 9 years ago
Save to file only temporary fix. This fix have a lot of questions.
When file will be deleted? How to detect this is temporary file or not? If 
temporary file we must no store bookmarks...

Original comment by mc.creat on 16 Jan 2012 at 8:39

GoogleCodeExporter commented 9 years ago
When CacheManager.clean() is run. It has a distinguishable prefix and suffix. 
Uff, well, wasn't an issue for me till now, as I needed a viewer only and never 
tried bookmarks.

Original comment by zhen...@gmail.com on 16 Jan 2012 at 9:19

GoogleCodeExporter commented 9 years ago
We applied your path. With some modifications.
Will be in 1.3.1 release.

Original comment by mc.creat on 16 Jan 2012 at 1:24

GoogleCodeExporter commented 9 years ago

Original comment by mc.creat on 16 Jan 2012 at 1:25