Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mangahere latest chapter broken #156

Open
Snipeon opened this issue Sep 26, 2017 · 80 comments
Open

Mangahere latest chapter broken #156

Snipeon opened this issue Sep 26, 2017 · 80 comments

Comments

@Snipeon
Copy link

Snipeon commented Sep 26, 2017

AMR is wrongly updating all Mangahere latest chapters. All the latest chapters from Mangahere seem to have a file:// added in at the beginning.

With ref to here.

@Snipeon
Copy link
Author

Snipeon commented Sep 27, 2017

Seems like Mangafox has also been struck with the same problem.

@twluo
Copy link

twluo commented Sep 27, 2017

From what I can tell, it might be because MangaHere just switched to htpps. Would need to change the rule here https://community.allmangasreader.com/latest_v2/MangaHere.js

@twluo
Copy link

twluo commented Sep 27, 2017

I am writing a really quick fix for it that is probably going to increase latency but at least it will fix the issue. I can share it once I'm at a point that's worth sharing if you're interested.

@twluo
Copy link

twluo commented Sep 27, 2017

Another fix could be changing the .attr("href") to just .href in https://github.com/AllMangasReader-dev/mirrors/blob/master/MangaHere.js. That would make it so that it returns the complete url with the https:// as opposed to just // but I haven't had the chance to test that yet.

@Snipeon
Copy link
Author

Snipeon commented Sep 27, 2017

Nice, good work. Too bad I suck at java.

Meaning that I can't help much.

@braiam
Copy link
Member

braiam commented Sep 27, 2017 via email

@Snipeon
Copy link
Author

Snipeon commented Sep 27, 2017

Anything with java in the name, I suck at.

@Stubbs9410
Copy link

Hi, sorry to bother, but the chrome extension of All Mangas Reader versions 1.5.10.2 is not working can I know if you will make a new extension, fix I really love this extension been using it a long time and I check github but I don't know Java script.
Please respond to the message

@jacobmix
Copy link

jacobmix commented Sep 27, 2017

I use AMR as an unpacked extension, and only thing i've edited is this line of manifest.json:
http://prntscr.com/gqh0m7 | And use this to remove the Chrome start popup warning.

Do i have to edit some file with this?
https://github.com/AllMangasReader-dev/mirrors/blob/master/MangaHere.js
If so what file? If not is there anything else i can do to get the fix?

Edit: Using All Mangas Reader (Beta channel) 1.5.10.2 Pic of the extension folder:
http://prntscr.com/gqk68v

@twluo
Copy link

twluo commented Sep 28, 2017

So right now I was in the middle of a quick fix which was basically appending https: to every mangahere url since it seems that its returning as //www.mangahere.co/* as opposed to http://ww.mangahere.co. I'm not sure if it has to do with the fact that mangahere switched to https. MY solution fixes the issue but it might create some latency issues. Right now it detects that if you're in the last chapter then there's no new chapter but I'm still having some issue with updating what's the latest chapter. Once I get that part done I will fork it and share it.

@twluo
Copy link

twluo commented Sep 28, 2017

From the little investigating I did yesterday night, I noticed that if you go to a mangahere detail page such as https://www.mangahere.co/manga/14_sai_no_koi/ and then you can see from https://github.com/AllMangasReader-dev/mirrors/blob/84c4a8f370b116c9b0edd798116de60067207e1e/MangaHere.js#L52-L54 that it uses .attr('href') to fetch the url. If you go to the detail page and run $(".detail_list ul li span.left a").attr("href"); on the console you will get "//www.mangahere.co/manga/14_sai_no_koi/v02/c035.7/" back. Now if you were to run $(".detail_list ul li span.left a")[0].href then you get "https://www.mangahere.co/manga/14_sai_no_koi/v02/c035.7/" which is the complete url

Same issue on https://github.com/AllMangasReader-dev/mirrors/blob/84c4a8f370b116c9b0edd798116de60067207e1e/MangaHere.js#L75-L77

By using .attr("href") we are getting how they wrote it but if we use .href then we get the absolute path.

@twluo
Copy link

twluo commented Sep 28, 2017

https://github.com/twluo/AMR Is a very dirty fix but it gets stuff working. There's still some issues with the counter. It seems that the counter doesn't refresh properly but you can get it to refresh by just reloading the extension.

@Snipeon
Copy link
Author

Snipeon commented Sep 28, 2017

Good job. Though if you are able to grab the correct URL by changing the Mangahere.js, why are you editing the files in the main js folder? Pardon my noobness.

@twluo
Copy link

twluo commented Sep 28, 2017

I'm not sure how the mirrors work and I don't really want to spent an entire night investigating it. I try copying the MangaHere.js into my local and try to use the local mirror instead of the repo one but it kept failing so I gave up. Also I just updated my fork and should be able to refresh properly.

@Snipeon
Copy link
Author

Snipeon commented Sep 28, 2017

Nice. Good job.

@twluo
Copy link

twluo commented Sep 28, 2017

Did you use my version? Help me test :) If you find any issue besides latency notify me.

@Vuzrak
Copy link

Vuzrak commented Sep 28, 2017

On your version I'm having 2 issues, the mark latest chapter as read doesn't seem to be working even with reloading the extension. Also the options button doesn't work and just brings it to "https://options.html/", the options button under chrome://extensions/ does work fine though.

@Vuzrak
Copy link

Vuzrak commented Sep 28, 2017

Actually I don't think any of the buttons work anymore in the menu.

@Snipeon
Copy link
Author

Snipeon commented Sep 28, 2017

testing it right now

@twluo
Copy link

twluo commented Sep 28, 2017

Maybe I messed up when I was cleaning the code. Because it works on the one I have loaded which was pre-cleaned up. Oh I never used those buttons so I didn't bother testing I know what's wrong. Also the mark latest I might have fixed it on the most recent change. When did you pull my changes?

@Snipeon
Copy link
Author

Snipeon commented Sep 28, 2017

for me no problems yet.

@Vuzrak
Copy link

Vuzrak commented Sep 28, 2017

I pulled the 6cfd713 commit.

@twluo
Copy link

twluo commented Sep 28, 2017

@Vuzrak I fixed the settings issue. Tried to make it a bit more generic but it didn't work out. So I specify it to MangaHere should work now.

@Vuzrak
Copy link

Vuzrak commented Sep 28, 2017

Also to use the mirror locally you would have to host the files yourself and modify the mgEntry.js to get it to work.

@Vuzrak
Copy link

Vuzrak commented Sep 28, 2017

Options working now, but the mark as seen still isn't.

@twluo
Copy link

twluo commented Sep 28, 2017

I did that but it wasn't working and I didn't really have the energy to look into it. I might try over the weekend. Anyways I can't reproduce the issue with the mark as read. Let me upload the "unclean" code.

@Snipeon
Copy link
Author

Snipeon commented Sep 28, 2017

ok, all Mangahere chapters have to be manually reloaded for there to be a working entry. The old one does not work, AMR would create a new working entry once you load the latest chapter.

However, Mangafox load full chapter option is broken.

@twluo
Copy link

twluo commented Sep 28, 2017

What I did was export the mangaList and then use a text editor to change all the urls to have https:// and then re import it.

@twluo
Copy link

twluo commented Sep 28, 2017

@Vuzrak have you worked with local mirrors? You could try what I suggested and see if it works.

@Vuzrak
Copy link

Vuzrak commented Sep 28, 2017

Weird, the mark read still doesn't work, gonna just try uninstalling extension and reinstalling it.

I tried your changes but they didn't seem to work, it's probably I didn't understand exactly all that was needed.

@twluo
Copy link

twluo commented Sep 28, 2017

I was using this https://raw.githubusercontent.com/twluo/mirrors/master/MangaHere.js

So is working fine now?

@Snipeon
Copy link
Author

Snipeon commented Sep 28, 2017

oh... Though I see that that are some differences between here and here

untitled

@twluo
Copy link

twluo commented Sep 28, 2017

There are some minor stuff, was just messing around to make it pass the lab test. For normal use I haven't seen any issue with the previous configuration

@Snipeon
Copy link
Author

Snipeon commented Sep 28, 2017

I see.

@Stubbs9410
Copy link

Im not at the computer right now, but can someone just archive thier chrome folder extension of all mangas reader with the fixes and put it online on drive or dropbox?

@Snipeon
Copy link
Author

Snipeon commented Sep 28, 2017

You can just download from twluo's gtihub here.

You know it's a good day when you have to manually reload 600+ manga.
internally screaming

@twluo
Copy link

twluo commented Sep 28, 2017

What do you mean by manually reload?

Btw I added support to use https://github.com/twluo/mirrors/blob/master/MangaHere.js as a Mirror so no need to have the local Mirror anymore. So I think it should work as is now.

@Snipeon
Copy link
Author

Snipeon commented Sep 28, 2017

For me, all the old entries of the manga does not work. I need to reload a chapter to get a new entry in, and delete the old entry...

@twluo
Copy link

twluo commented Sep 28, 2017

So for that I just export it and then do a replace all <"//www.mangahere.co> to <"https://www.mangahere.co"> and also a <"http://www.mangahere.co> to <"https://www.mangahere.co> and import it back. Try that it might work.

@twluo
Copy link

twluo commented Sep 28, 2017

I can try taking a look at Mangafox during the weekend. I think I spent too much time on JS for a bit. @.@

@Vuzrak
Copy link

Vuzrak commented Sep 28, 2017

Yup, I did the same thing, export it, search/replace, import it back in.

@twluo
Copy link

twluo commented Sep 28, 2017

@Vuzrak I updated the repo to fetch the mirror rules from the other repo as opposed to local that way it will be easier to mantain. So if you haven't I suggest you update it.

@twluo
Copy link

twluo commented Sep 28, 2017

If you find any other issue either comment it here or raise it on my repo and I can take a look tomorrow.

@Snipeon
Copy link
Author

Snipeon commented Sep 28, 2017

@twluo @Vuzrak OH GOD I'M AN IDIOT!!! Didn't think of that at all...

@jacobmix
Copy link

So i just export my manga list. Then do this: https://prnt.sc/gqq51a , and: https://prnt.sc/gqq5k1
Then download this: https://codeload.github.com/twluo/AMR/zip/develop
Extract it, and override my other extension (Backed up old one first tho)
Then import? Because i get this error when i try to import: http://prntscr.com/gqq928
Am i missing something?

@Snipeon
Copy link
Author

Snipeon commented Sep 28, 2017

After you did the first replace, it's enough... there's no need to add in the second replace....

@Vuzrak
Copy link

Vuzrak commented Sep 28, 2017

That and the 2nd replaced you added an extra "

@jacobmix
Copy link

jacobmix commented Sep 28, 2017

Okay. Will try doing it over. But not adding the extra "
Just did as twluo said in his earlier comment.

So for that I just export it and then do a replace all <"//www.mangahere.co> to <"https://www.mangahere.co"> and also a <"http://www.mangahere.co> to <"https://www.mangahere.co> and import it back. Try that it might work.

@Vuzrak
Copy link

Vuzrak commented Sep 28, 2017

Just do a replace on "http://www.mangahere" to "https://www.mangahere" without the quotes.

@jacobmix
Copy link

jacobmix commented Sep 28, 2017

Doing "//www.mangahere.co to "https://www.mangahere.co
and also "http://www.mangahere.co to "https://www.mangahere.co worked.

But doing just "http://www.mangahere.co to "https://www.mangahere.co also works.

@noodle27
Copy link

noodle27 commented Oct 2, 2017

I'm having the problem for loading all pages on Mangahere again. I applied the fix by @twluo and it's actually fixed (thanks man), but yesterday the problem came back and I can't load all pages in the chapter now (inb4 MH dev is secretly lurking this place). I've already checked the "load scans in oder of chapter" and "save bandwidth" in the option. New chapter alert is still working tho.

@twluo
Copy link

twluo commented Oct 2, 2017

MangaHere is dead in regards to that, there are some server issues or something is happening. I haven't had the time to look into it

@noodle27
Copy link

noodle27 commented Oct 2, 2017

Update: just turn off and on my pc, and suddenly mangahere magically works, without any loading issue. That is that I guess. I feel dumb right now. Thanks for putting up with my crap

@yvains
Copy link

yvains commented Oct 5, 2017

i have the same problem as noodle27, i applied the fix by twluo, and it still can't load all pages in the chapter, (only 17%, 34% of it), i already checked the "load scans in oder of chapter" and "save bandwidth" in the option also

@noodle27
Copy link

noodle27 commented Oct 13, 2017

Sorry to dug this thread up, but I've encountered another serious problem with Mangahere.

So my AMR was having a problem, it won't save my progress after I quited browser. To make it easier to understand:

-Chapter 1 of manga A is marked as the last chapter read
-Chapter 2 is released, marked as read
-Quit browser
-Open browser again, progress set back to the previous state, which chapter 1 is still marked as last chapter read, not chapter 2

My only way to counter this was just export the reading list before quitting browser, then import it again the next time I open it.

This happens to ALL manga in my list. So I tried to remove AMR and reinstall it ( @twluo 's fix is applied), but after I'm done importing the list, another problem popped up:

-All manga in Mangahere is marked as unread (this is not really a problem, since I marked them as read and it's ok, still not sure if it'll be messed up if new chapter actual comes up tho)
-Mangahere doesn't load the whole chapter anymore (this is the main problem). Other sites like Batoto is still working well.

Not sure if the previous problem is fixed or not.

Anyone can help me on this?

@king37sniper
Copy link

@noodle27 Everyone has that problem, and don't know when @braiam start to fix it

@noodle27
Copy link

noodle27 commented Oct 17, 2017

Forgot to update. I cleared out everything of AMR then install it again using https://github.com/twluo/AMR and it worked out well, at least for me. Both reading progress marking problem and mangahere not loading problem are gone. Not sure if the reading progress marking will come back or not tho, because last time it suddenly came out of nowhere too

@king37sniper
Copy link

Yeah, just try it 1h ago
Working 100%

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

9 participants