calvinaquino / LNReader-Android

Light novel reader for android test project collab
Apache License 2.0
120 stars 36 forks source link

Bug on novel cover due to thumbnail handling. #259

Closed lyram21 closed 7 years ago

lyram21 commented 7 years ago

bitmap empty... and the chapter links are all red...

Nandaka commented 7 years ago

can you open the site from phone browser?

lyram21 commented 7 years ago

yes i can open it without any problems

belldandu commented 7 years ago

It appears that i have this exact same issue too and i load the site just fine in a browser. Samsung galaxy note 5 & samsung galaxy tab s2. Both are using Android 6.0.1 firmware

belldandu commented 7 years ago

App version 1.1.45

belldandu commented 7 years ago

It should be noted that all the chapters say they were last updated 6 years ago. And when you set (download hi res image) instead of getting "bitmap empy" you get a rediculously pixelated image for the cover.

belldandu commented 7 years ago

Should also be noted that some projects like this one https://www.baka-tsuki.org/project/index.php?title=Seikoku_no_Ryuu_Kishi are not shown in the main list of the app no matter how much i refresh. Which makes me beleive the app is pulling from an outdated source.

belldandu commented 7 years ago

Managed to get above project to show after playing around with settings but chapters still are red

belldandu commented 7 years ago

Aha here's a screenshot of an error tmp_28859-screenshot_20161031-0103171831514330

belldandu commented 7 years ago

tmp_28859-screenshot_20161031-005820-2087104504

belldandu commented 7 years ago

Just a heads up i am online.

belldandu commented 7 years ago

Ah as it turns out the app is trying to connect and grab from a dead server which is why its failing. They got a new server on october 10th.

Nandaka commented 7 years ago

From the screenshot, it shows that the connection is refused.

The ip address shown in the error message supposed to be the new server.

[image: Inline image 1]

[image: Inline image 2]

cover image = bitmap empty

This one is a bug because of changes on thumbnail handling.Using HiRes cover in Settings -> General can fix this.

lyram21 commented 7 years ago

the images are fixed sir ghen how about the chapters?

belldandu commented 7 years ago

Is there a reason you guys dont use the mediawikia api instead of scraping the site?

Nandaka commented 7 years ago

@belldandu I'm using both api and scraping.

API portion to retrieve body content and update info. Scraping from the api return to parse the novel structure/chapters from the novel detail page (this one I'm not sure how to retrieve the structure using api).

The contents itself is returned by api call. https://github.com/calvinaquino/LNReader-Android/blob/master/app/src/main/java/com/erakk/lnreader/dao/NovelsDao.java#L1103

https://github.com/calvinaquino/LNReader-Android/tree/master/app/src/main/java/com/erakk/lnreader/parser

yuinyaa commented 7 years ago

The image issue is already fixed but all my chapter are still redlink and cannot be seen at all, when I open the page, it shows " The page you specified doesn't exist"

danieltanfh95 commented 7 years ago

Just curious why is the IP address used instead of the domain name?

yuinyaa commented 7 years ago

@shadowys I am not too well verse in computer program language

danieltanfh95 commented 7 years ago

@yuinyaa Actually I was pinging @Nandaka sorry.