Update README.md

- Store the `Title` objects in `Library@title_hash`
- The `Title` objects only stores IDs to other titles

README.md

@@ -12,6 +12,7 @@ Mango is a self-hosted manga server and reader. Its features include
 - Multi-user support
 - Dark/light mode switch
 - Supports both `.zip` and `.cbz` formats
+- Supports nested folders in library
 - Automatically stores reading progress
 - Built-in [MangaDex](https://mangadex.org/) downloader
 - The web reader is responsive and works well on mobile, so there is no need for a mobile app
@@ -74,22 +75,6 @@ mangadex:
 - `scan_interval_minutes` can be any non-negative integer. Setting it to `0` disables the periodic scan
 - `log_level` can be `debug`, `info`, `warn`, `error`, `fatal` or `off`. Setting it to `off` disables the logging
 
-### Required Library Structure
-
-Please make sure that your library directory has the following structure:
-
-```
-.
-├── Manga 1
-│   └── Manga 1.cbz
-└── Manga 2
-    ├── Vol 0001.zip
-    ├── Vol 0002.zip
-    ├── Vol 0003.zip
-    ├── Vol 0004.zip
-    └── Vol 0005.zip
-```
-
 ### Initial Login
 
 On the first run, Mango would log the default username and a randomly generated password to STDOUT. You are advised to immediately change the password.

public/css/mango.css

@@ -20,7 +20,7 @@
 #scan-status {
 	cursor: auto;
 }
-.uk-card-title {
+.break-word {
 	word-wrap: break-word;
 }
 .uk-logo > img {

public/js/sort-items.js

@@ -93,8 +93,8 @@ $(() => {
 				else if (by === 'date')
 					res = $(a).attr('data-mtime') > $(b).attr('data-mtime');
 				else if (by === 'progress') {
-					const ap = $(a).attr('data-progress');
+					const ap = parseFloat($(a).attr('data-progress'));
-					const bp = $(b).attr('data-progress');
+					const bp = parseFloat($(b).attr('data-progress'));
 					if (ap === bp)
 						// if progress is the same, we compare by name
 						res = $(a).find('.uk-card-title').text() > $(b).find('.uk-card-title').text();

public/js/title.js

@@ -15,7 +15,7 @@ function showModal(encodedPath, pages, percentage, encodedeTitle, encodedEntryTi
 	if (percentage === 100) {
 		$('#read-btn').attr('hidden', '');
 	}
-	$('#modal-title').text(title);
+	$('#modal-title').text(entry);
 	$('#path-text').text(zipPath);
 	$('#pages-text').text(pages + ' pages');
 

shard.yml

@@ -1,5 +1,5 @@
 name: mango
-version: 0.1.0
+version: 0.2.1
 
 authors:
   - Alex Ling <hkalexling@gmail.com>

src/library.cr

@@ -15,7 +15,7 @@ struct Image
 end
 
 class Entry
-	JSON.mapping zip_path: String, book_title: String, title: String,
+	property zip_path : String, book_title : String, title : String,
 		size : String, pages : Int32, cover_url : String, id : String,
 		title_id : String, encoded_path : String, encoded_title : String,
 		mtime : Time
@@ -38,6 +38,19 @@ class Entry
 		@cover_url = "/api/page/#{@title_id}/#{@id}/1"
 		@mtime = File.info(@zip_path).modification_time
 	end
+
+	def to_json(json : JSON::Builder)
+		json.object do
+			{% for str in ["zip_path", "book_title", "title", "size",
+				"cover_url", "id", "title_id", "encoded_path",
+				"encoded_title"] %}
+				json.field {{str}}, @{{str.id}}
+			{% end %}
+			json.field "pages" {json.number @pages}
+			json.field "mtime" {json.number @mtime.to_unix}
+		end
+	end
+
 	def read_page(page_num)
 		Zip::File.open @zip_path do |file|
 			page = file.entries
@@ -63,27 +76,94 @@ class Entry
 end
 
 class Title
-	JSON.mapping dir: String, entries: Array(Entry), title: String,
+	property dir : String, parent_id : String, title_ids : Array(String),
-		id: String, encoded_title: String, mtime: Time, logger: MLogger
+		entries : Array(Entry), title : String, id : String,
+		encoded_title : String, mtime : Time
 
-	def initialize(dir : String, storage, @logger : MLogger)
+	def initialize(dir : String, @parent_id, storage,
+				   @logger : MLogger, @library : Library)
 		@dir = dir
 		@id = storage.get_id @dir, true
 		@title = File.basename dir
 		@encoded_title = URI.encode @title
-		@entries = (Dir.entries dir)
+		@title_ids = [] of String
-			.select { |path| [".zip", ".cbz"].includes? File.extname path }
+		@entries = [] of Entry
-			.map { |path| File.join dir, path }
+
-			.select { |path| valid_zip path }
+		Dir.entries(dir).each do |fn|
-			.map { |path|
+			next if fn.starts_with? "."
-				Entry.new path, @title, @id, storage
+			path = File.join dir, fn
-			}
+			if File.directory? path
-			.select { |e| e.pages > 0 }
+				title = Title.new path, @id, storage, @logger, library
-			.sort { |a, b| a.title <=> b.title }
+				next if title.entries.size == 0 && title.titles.size == 0
+				@library.title_hash[title.id] = title
+				@title_ids << title.id
+				next
+			end
+			if [".zip", ".cbz"].includes? File.extname path
+				next if !valid_zip path
+				entry = Entry.new path, @title, @id, storage
+				@entries << entry if entry.pages > 0
+			end
+		end
+
+		@title_ids.sort! do |a, b|
+			compare_alphanumerically @library.title_hash[a].title,
+				@library.title_hash[b].title
+		end
+		@entries.sort! do |a, b|
+			compare_alphanumerically a.title, b.title
+		end
+
 		mtimes = [File.info(dir).modification_time]
+		mtimes += @title_ids.map{|e| @library.title_hash[e].mtime}
 		mtimes += @entries.map{|e| e.mtime}
 		@mtime = mtimes.max
 	end
+
+	def to_json(json : JSON::Builder)
+		json.object do
+			{% for str in ["dir", "title", "id", "encoded_title"] %}
+				json.field {{str}}, @{{str.id}}
+			{% end %}
+			json.field "mtime" {json.number @mtime.to_unix}
+			json.field "titles" do
+				json.raw self.titles.to_json
+			end
+			json.field "entries" do
+				json.raw @entries.to_json
+			end
+			json.field "parents" do
+				json.array do
+					self.parents.each do |title|
+						json.object do
+							json.field "title", title.title
+							json.field "id", title.id
+						end
+					end
+				end
+			end
+		end
+	end
+
+	def titles
+		@title_ids.map {|tid| @library.get_title! tid}
+	end
+
+	def parents
+		ary = [] of Title
+		tid = @parent_id
+		while !tid.empty?
+			title = @library.get_title! tid
+			ary << title
+			tid = title.parent_id
+		end
+		ary
+	end
+
+	def size
+		@entries.size + @title_ids.size
+	end
+
 	# When downloading from MangaDex, the zip/cbz file would not be valid
 	#	before the download is completed. If we scan the zip file,
 	#	Entry.new would throw, so we use this method to check before
@@ -132,6 +212,7 @@ class Title
 		page / entry_obj.pages
 	end
 	def load_percetage(username)
+		return 0 if @entries.empty?
 		read_pages = total_pages = 0
 		@entries.each do |e|
 			read_pages += load_progress username, e.title
@@ -150,10 +231,7 @@ class TitleInfo
 	# { user1: { entry1: 10, entry2: 0 } }
 	include JSON::Serializable
 
-	@[JSON::Field(key: "comment")]
 	property comment = "Generated by Mango. DO NOT EDIT!"
-
-	@[JSON::Field(key: "progress")]
 	property progress : Hash(String, Hash(String, Int32))
 
 	def initialize(title_dir)
@@ -175,13 +253,14 @@ class TitleInfo
 end
 
 class Library
-	JSON.mapping dir: String, titles: Array(Title), scan_interval: Int32,
+	property dir : String, title_ids : Array(String), scan_interval : Int32,
-		logger: MLogger, storage: Storage
+		logger : MLogger, storage : Storage, title_hash : Hash(String, Title)
 
 	def initialize(@dir, @scan_interval, @logger, @storage)
 		# explicitly initialize @titles to bypass the compiler check. it will
 		#	be filled with actual Titles in the `scan` call below
-		@titles = [] of Title
+		@title_ids = [] of String
+		@title_hash = {} of String => Title
 
 		return scan if @scan_interval < 1
 		spawn do
@@ -189,13 +268,27 @@ class Library
 				start = Time.local
 				scan
 				ms = (Time.local - start).total_milliseconds
-				@logger.info "Scanned #{@titles.size} titles in #{ms}ms"
+				@logger.info "Scanned #{@title_ids.size} titles in #{ms}ms"
 				sleep @scan_interval * 60
 			end
 		end
 	end
+	def titles
+		@title_ids.map {|tid| self.get_title!(tid) }
+	end
+	def to_json(json : JSON::Builder)
+		json.object do
+			json.field "dir", @dir
+			json.field "titles" do
+				json.raw self.titles.to_json
+			end
+		end
+	end
 	def get_title(tid)
-		@titles.find { |t| t.id == tid }
+		@title_hash[tid]?
+	end
+	def get_title!(tid)
+		@title_hash[tid]
 	end
 	def scan
 		unless Dir.exists? @dir
@@ -203,11 +296,18 @@ class Library
 				"Attempting to create it"
 			Dir.mkdir_p @dir
 		end
-		@titles = (Dir.entries @dir)
+		@title_ids.clear
-			.select { |path| File.directory? File.join @dir, path }
+		(Dir.entries @dir)
-			.map { |path| Title.new File.join(@dir, path), @storage, @logger }
+			.select { |fn| !fn.starts_with? "." }
-			.select { |title| !title.entries.empty? }
+			.map { |fn| File.join @dir, fn }
+			.select { |path| File.directory? path }
+			.map { |path| Title.new path, "", @storage, @logger, self }
+			.select { |title| !(title.entries.empty? && title.titles.empty?) }
 			.sort { |a, b| a.title <=> b.title }
+			.each do |title|
+				@title_hash[title.id] = title
+				@title_ids << title.id
+			end
 		@logger.debug "Scan completed"
 	end
 end

src/mango.cr

@@ -3,7 +3,7 @@ require "./context"
 require "./mangadex/*"
 require "option_parser"
 
-VERSION = "0.2.0"
+VERSION = "0.2.1"
 
 config_path = nil
 

src/routes/api.cr

@@ -26,7 +26,7 @@ class APIRouter < Router
 			end
 		end
 
-		get "/api/book/:title" do |env|
+		get "/api/book/:tid" do |env|
 			begin
 				tid = env.params.url["tid"]
 				title = @context.library.get_title tid

src/routes/main.cr

@@ -26,6 +26,7 @@ class MainRouter < Router
 					.not_nil!
 
 				cookie = HTTP::Cookie.new "token", token
+				cookie.expires = Time.local.shift years: 1
 				env.response.cookies << cookie
 				env.redirect "/"
 			rescue
@@ -46,6 +47,8 @@ class MainRouter < Router
 				username = get_username env
 				percentage = title.entries.map { |e|
 					title.load_percetage username, e.title }
+				titles_percentage = title.titles.map { |t|
+					title.load_percetage username, t.title }
 				layout "title"
 			rescue e
 				@context.error e

src/views/index.ecr

@@ -26,12 +26,18 @@
       <a class="acard" href="/book/<%= t.id %>">
         <div class="uk-card uk-card-default">
             <div class="uk-card-media-top">
+			<%- if t.entries.size > 0 -%>
               <img src="<%= t.entries[0].cover_url %>" alt="">
+			<%- else -%>
+              <img src="/img/icon.png" alt="">
+			<%- end -%>
             </div>
             <div class="uk-card-body">
+			<%- if t.entries.size > 0 -%>
               <div class="uk-card-badge uk-label"><%= (percentage[i] * 100).round(1) %>%</div>
-              <h3 class="uk-card-title"><%= t.title %></h3>
+			<%- end -%>
-              <p><%= t.entries.size %> entries</p>
+              <h3 class="uk-card-title break-word"><%= t.title %></h3>
+              <p><%= t.size %> entries</p>
             </div>
         </div>
       </a>

src/views/title.ecr

@@ -1,5 +1,12 @@
 <h2 class=uk-title><%= title.title %></h2>
-<p class="uk-text-meta"><%= title.entries.size %> entries found</p>
+<ul class="uk-breadcrumb">
+	<li><a href="/">Library</a></li>
+	<%- title.parents.each do |t| -%>
+	<li><a href="/book/<%= t.id %>"><%= t.title %></a></li>
+	<%- end -%>
+	<li class="uk-disabled"><a><%= title.title %></a></li>
+</ul>
+<p class="uk-text-meta"><%= title.size %> entries found</p>
 <div class="uk-grid-small" uk-grid>
 	<div class="uk-margin-bottom uk-width-3-4@s">
 		<form class="uk-search uk-search-default">
@@ -23,6 +30,28 @@
 	</div>
 </div>
 <div id="item-container" class="uk-child-width-1-4@m uk-child-width-1-2" uk-grid>
+	<%- title.titles.each_with_index do |t, i| -%>
+	<div class="item" data-mtime="<%= t.mtime.to_unix %>" data-progress="<%= titles_percentage[i] %>">
+      <a class="acard" href="/book/<%= t.id %>">
+        <div class="uk-card uk-card-default">
+            <div class="uk-card-media-top">
+			<%- if t.entries.size > 0 -%>
+              <img src="<%= t.entries[0].cover_url %>" alt="">
+			<%- else -%>
+              <img src="/img/icon.png" alt="">
+			<%- end -%>
+            </div>
+            <div class="uk-card-body">
+			<%- if t.entries.size > 0 -%>
+              <div class="uk-card-badge uk-label"><%= (titles_percentage[i] * 100).round(1) %>%</div>
+			<%- end -%>
+              <h3 class="uk-card-title break-word"><%= t.title %></h3>
+              <p><%= t.size %> entries</p>
+            </div>
+        </div>
+      </a>
+	</div>
+	<%- end -%>
   <%- title.entries.each_with_index do |e, i| -%>
   <div class="item" data-mtime="<%= e.mtime.to_unix %>" data-progress="<%= percentage[i] %>">
       <a class="acard">
@@ -44,8 +73,8 @@
   <div class="uk-modal-dialog uk-margin-auto-vertical">
     <button class="uk-modal-close-default" type="button" uk-close></button>
     <div class="uk-modal-header">
-      <h3 class="uk-modal-title" id="modal-title"></h3>
+      <h3 class="uk-modal-title break-word" id="modal-title"></h3>
-      <p class="uk-text-meta uk-margin-remove-bottom" id="path-text"></p>
+      <p class="uk-text-meta uk-margin-remove-bottom break-word" id="path-text"></p>
       <p class="uk-text-meta uk-margin-remove-top" id="pages-text"></p>
     </div>
     <div class="uk-modal-body">