better handling of book details, use custom columns

bouquins.js

@@ -42,6 +42,7 @@ var db = new sqlite3.Database(dbPath);
 
 app.use(function(req, res, next) {
 	req.db = db;
+	req.locals = app.locals;
 	next();
 });
 

config/development.json

@@ -1,5 +1,16 @@
 {
 	"calibre_path": "./public/calibre",
 	"calibre_db_path": "./public/calibre/metadata.db",
-	"trust proxy": true
+	"trust proxy": true,
+	"book_details_custom": [
+		{
+			"name": "a_lire",
+			"query": "SELECT custom_columns.name AS label, custom_column_2.value AS value FROM custom_column_2 LEFT OUTER JOIN custom_columns ON custom_columns.id = 2 WHERE custom_column_2.book = ?",
+			"type": "bool"
+		}, {
+			"name": "phy",
+			"query": "SELECT custom_columns.name AS label, custom_column_3.value AS value FROM custom_column_3 LEFT OUTER JOIN custom_columns ON custom_columns.id = 3 WHERE custom_column_3.book = ?",
+			"type": "bool"
+		}
+	]
 }

routes/book.js

@@ -151,7 +151,8 @@ router.post('/', function(req,res) {
 /* Single book */
 router.get('/:id', function(req, res) {
 	var book, authors, tags, details;
-	var queries = 4;
+	var queries = 3;
+	if (req.locals.book_details_custom) queries+=req.locals.book_details_custom.length;
 	var callback = function() {
 		if (queries == 0) {
 			book.authors = authors;
@@ -170,12 +171,14 @@ router.get('/:id', function(req, res) {
 	// book
 	req.db.get('SELECT books.id AS id,title,timestamp,pubdate,series_index,isbn,lccn,path,uuid,has_cover,' +
 		'languages.lang_code,format,uncompressed_size,data.name AS data_name,series.name AS series_name,' +
-		'series.id AS series_id FROM books ' +
+		'series.id AS series_id, publishers.name AS pubname FROM books ' +
 		' LEFT OUTER JOIN books_languages_link ON books_languages_link.book = books.id ' +
 		' LEFT OUTER JOIN languages ON languages.id = books_languages_link.lang_code ' +
 		' LEFT OUTER JOIN data ON data.book = books.id ' +
 		' LEFT OUTER JOIN books_series_link ON books.id = books_series_link.book ' +
 		' LEFT OUTER JOIN series ON series.id = books_series_link.series ' +
+		' LEFT OUTER JOIN books_publishers_link ON books.id = books_publishers_link.book ' +
+		' LEFT OUTER JOIN publishers ON publishers.id = books_publishers_link.publisher ' +
 		' WHERE books.id = ?', req.params.id,
 		function(err, row) {
 			if (err) console.log('ERR book: '+err);
@@ -210,9 +213,13 @@ router.get('/:id', function(req, res) {
 	);
 	// details
 	custom = new Array();
-	req.db.each('SELECT publishers.name as pubname FROM publishers, books_publishers_link WHERE books_publishers_link.publisher = publishers.id AND books_publishers_link.book = ?', req.params.id,
+	if (req.locals.book_details_custom) {
-		function(err, publisher) {
+		_.each(req.locals.book_details_custom, function(detail) {
-			custom.push({name:'pubname',value:publisher.pubname});
+			req.db.each(detail.query, req.params.id,
+				function(err, row) {
+					row.name = detail.name;
+					row.type = detail.type;
+					custom.push(row);
 				},
 				function(err) {
 					if (err) console.log('ERR tags: '+err);
@@ -220,6 +227,8 @@ router.get('/:id', function(req, res) {
 					callback();
 				}
 			);
+		});
+	}
 });
 
 

views/book.jade

@@ -48,10 +48,24 @@ block content
             li
               strong #{t('bouquins.book.detail.pubdate')}
               |  #{new Date(pubdate).getMonth()}-#{new Date(pubdate).getFullYear()}
+          if pubname
+            li
+              strong #{t('bouquins.book.detail.pubname')}
+              |  #{pubname}
           if custom
             each item in custom
               li
+                if item.label
+                  strong #{item.label}
+                else
                   strong #{t('bouquins.book.detail.'+item.name)}
+                case item.type
+                  when "bool"
+                    if item.value == 1
+                      span.glyphicon.glyphicon-ok
+                    else
+                      span.glyphicon.glyphicon-remove
+                  default
                     |  #{item.value}
       div.col-md-7
         if has_cover == 1