Skip to content
This repository has been archived by the owner on Oct 31, 2020. It is now read-only.

Added KissManga.lua #1141

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

GhostWriterTNCS
Copy link

Hi!

I've created the KissManga module in lua.
It can get info from url, check new chapters and download chapters.
It cannot update the manga list and I still don't understand why it fails.
I've decided to create a pull request anyway because the core functions are working, so you don't need the workaround.

Thank you very much.

@Slasar41
Copy link
Member

Slasar41 commented Oct 9, 2020

I look at other modules and make it work with this:

function GetDirectoryPageNumber()
  if http.get(module.RootURL..'/manga_list') then
    page = tonumber(TXQuery.Create(http.Document).XPathString('//ul[@class="pager"]/li[last()]/a'):match('%((.-)%)'))
    return true
  else
    return false
  end
end

Also maybe you want add it to FMD2 repo too.

@Tmp341 Tmp341 added Lua Issue has to do with lua scripts. Site Request Request for adding a new website. Partially Implemented Issue has been partially implemented. There's still something to implement. labels Oct 10, 2020
@GhostWriterTNCS
Copy link
Author

I added the suggested script, but I think the method GetDirectoryPageNumber() is not called at all, in fact it always parses only the first page of the manga list (it parses the page correctly btw).

@Slasar41
Copy link
Member

It's working on my end.
image

@GhostWriterTNCS
Copy link
Author

I've re-downloaded the last release of FMD and it worked. Apparently there was a problem with my config files.
Thank you @Slasar41 for your help.

@GhostWriterTNCS GhostWriterTNCS changed the title Added KissManga.lua (partial) Added KissManga.lua Oct 12, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Lua Issue has to do with lua scripts. Partially Implemented Issue has been partially implemented. There's still something to implement. Site Request Request for adding a new website.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants