Allow specifying a search as a parameter to the top-level URL

This commit is contained in:
Kovid Goyal 2015-11-26 13:11:21 +05:30
parent 3a873628da
commit 1adebcc700
6 changed files with 55 additions and 9 deletions

View File

@ -90,6 +90,7 @@ def interface_data(ctx, rd):
Return the data needed to create the server main UI
Optional: ?num=50&sort=timestamp.desc&library_id=<default library>
&search=''&extra_books=''
'''
ans = {'username':rd.username}
ans['library_map'], ans['default_library'] = ctx.library_map
@ -111,7 +112,7 @@ def interface_data(ctx, rd):
except Exception:
raise HTTPNotFound('Invalid number of books: %r' % rd.query.get('num'))
with db.safe_read_lock:
ans['search_result'] = search_result(ctx, rd, db, '', num, 0, ','.join(sorts), ','.join(orders))
ans['search_result'] = search_result(ctx, rd, db, rd.query.get('search', ''), num, 0, ','.join(sorts), ','.join(orders))
sf = db.field_metadata.ui_sortable_field_keys()
sf.pop('ondevice', None)
ans['sortable_fields'] = sorted(((
@ -120,9 +121,16 @@ def interface_data(ctx, rd):
ans['field_metadata'] = db.field_metadata.all_metadata()
ans['icon_map'] = icon_map()
mdata = ans['metadata'] = {}
for book_id in ans['search_result']['book_ids']:
data = book_as_json(db, book_id)
mdata[book_id] = data
try:
extra_books = set(int(x) for x in rd.query.get('extra_books', '').split(','))
except Exception:
extra_books = ()
for coll in (ans['search_result']['book_ids'], extra_books):
for book_id in coll:
if book_id not in mdata:
data = book_as_json(db, book_id)
if data is not None:
mdata[book_id] = data
return ans
@ -153,7 +161,8 @@ def more_books(ctx, rd):
mdata = ans['metadata'] = {}
for book_id in ans['search_result']['book_ids']:
data = book_as_json(db, book_id)
mdata[book_id] = data
if data is not None:
mdata[book_id] = data
return ans
@ -197,7 +206,8 @@ def get_books(ctx, rd):
raise HTTPBadRequest('Invalid search expression: %s' % as_unicode(err))
for book_id in ans['search_result']['book_ids']:
data = book_as_json(db, book_id)
mdata[book_id] = data
if data is not None:
mdata[book_id] = data
return ans
@endpoint('/interface-data/tag-browser', postprocess=json)

View File

@ -33,11 +33,13 @@ def encode_datetime(dateval):
return None
return isoformat(dateval)
empty_val = ((), '', {})
def add_field(field, db, book_id, ans, field_metadata):
datatype = field_metadata.get('datatype')
if datatype is not None:
val = db._field_for(field, book_id)
if val is not None and val != ():
if val is not None and val not in empty_val:
if datatype == 'datetime':
val = encode_datetime(val)
if val is None:
@ -48,6 +50,8 @@ def book_as_json(db, book_id):
db = db.new_api
with db.safe_read_lock:
ans = {'formats':db._formats(book_id)}
if not ans['formats'] and not db.has_id(book_id):
return None
fm = db.field_metadata
for field in fm.all_field_keys():
if field not in IGNORED_FIELDS:

View File

@ -57,6 +57,9 @@ class TopBar:
if not tooltip:
tooltip = _('Donate to support calibre development')
if callable(title):
title = title()
for bar in self.bar, self.dummy_bar:
left = bar.firstChild
clear(left)

View File

@ -88,7 +88,10 @@ class UI:
self.books_view = BooksView(interface_data, book_list_container)
self.items_view = ItemsView(interface_data, book_list_container)
self.search_panel = SearchPanel(interface_data, book_list_container)
ibs = BarState(run_animation=True)
ibs = BarState(run_animation=True, title=def():
q = self.books_view.interface_data['search_result']['query']
return (_('Books matching:') + ' ' + q) if q else 'calibre'
)
ibs.add_button(icon_name='sort-amount-desc', tooltip=_('Sort books'), action=show_panel_action('booklist-sort-menu'))
ibs.add_button(icon_name='search', tooltip=_('Search for books'), action=show_panel_action('booklist-search'))
ibs.add_button(icon_name='ellipsis-v', tooltip=_('More actions'), action=show_panel_action('more-actions-menu'))

View File

@ -3,10 +3,12 @@
from ajax import ajax
from elementmaker import E
from gettext import gettext as _
from session import UserSessionData
from utils import parse_url_params
from book_list.boss import Boss
from book_list.globals import set_boss, set_session_data
from gettext import gettext as _
def on_library_loaded(end_type, xhr, ev):
p = document.getElementById('page_load_progress')
@ -35,6 +37,9 @@ def load_book_list():
query = {k:temp.get(k) for k in str.split(
'library_id sort partition_method collapse_at dont_collapse sort_tags_by'
)}
url_query = parse_url_params()
for key in url_query:
query[key] = url_query[key]
ajax('interface-data/init', on_library_loaded, on_library_load_progress, query=query).send()
def on_load():

View File

@ -33,3 +33,24 @@ def to_utf8(string):
ua[i] = ch.charCodeAt(0)
)
return ua
def parse_url_params(url=None, allow_multiple=False):
url = url or window.location.href
qs = url.indexOf('?')
ans = {}
if qs < 0:
return ans
q = url.slice(qs + 1, ((url.indexOf('#') + 1) or (url.length + 1)))
if not q:
return ans
pairs = q.replace(/\+/g, " ").split("&")
for pair in pairs:
key, val = str.partition(pair, '=')[::2]
key, val = decodeURIComponent(key), decodeURIComponent(val)
if allow_multiple:
if not Object.prototype.hasOwnProperty.call(ans, key):
ans[key] = []
ans[key].append(val)
else:
ans[key] = val
return ans