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

Neue Funktionen #3

Closed
wants to merge 2 commits into from
Closed

Neue Funktionen #3

wants to merge 2 commits into from

Conversation

DarkHunter85
Copy link

Hallo,

ich habe einige Kleinigkeiten hinzugefügt, die mir an calibre2opds gefallen haben.

Buchreihen können nun angezeigt werden und die Liste "Bücher" auch zuerst nach Serien sortiert werden, einstellbar im Admin-Menü. In der Detailansicht eines Buchs wird nun auch die Serie angezeigt, sofern vorhanden.

In allen Buchlisten wird jetzt auch der Autor des jeweiligen Buches mit angezeigt.

Sofern die Änderungen gefallen, könnten diese ja in die nächste Version einfließen ;o)

@rvolz
Copy link
Owner

rvolz commented Jul 23, 2012

Vielen Dank für den Code, ich schaue mir die Änderungen demnächst genauer an. Eine Frage ist, ob das Hauptmenü nicht langsam zu groß wird, gerade bei Geräten mit kleinen Bildschirmen (à la iPhone etc).

Rainer

@DarkHunter85
Copy link
Author

Okay, an die kleineren Geräte habe ich so nicht gedacht, da ich das ganze bisher nur aus PC-Sicht betrachtet hatte... Aber ich habe mal einen Screenshot von meinem Galaxy S2 gemacht:

http://koenigsmoerder.net/dahu/bbs.jpeg

Auf einem 7-Zoll-Reader müsste das ganze aber noch gut aussehen, denke ich.

Vielleicht ließe sich ja im schlimmsten Falle die Schrift für das Hauptmenü verkleinern? Ansonsten bieten sich evtl. auch entsprechende Icons an.

Gruß
Benjamin

Series:
	- View series of books
	- sort books by series (see settings on admin page) - Bugfix
	- Show series on book details if book is part of series

Smaller changes:
	- Author is shown in list-view
@DarkHunter85
Copy link
Author

Mir ist da noch ein kleiner (Logik-)Fehler aufgefallen... Ich habe daher die Funktion "allSortedTitles" korrigiert ;)

@rvolz
Copy link
Owner

rvolz commented Aug 2, 2012

Ok, die Serienfunktionalität und die Erweiterung der Listenanzeige wurden in 0.9 übernommen.

@rvolz rvolz closed this Aug 2, 2012
@DarkHunter85
Copy link
Author

Hallo Herr Volz,

vielen Dank für die Übernahme der Funktion :o)

Mir ist allerdings noch etwas aufgefallen:

Die Funktion "seriesDetails" gibt momentan (0.9.1) die Bücher einer Reihe in der Reihenfolge wieder, wie sie als ID in der Datenbank gespeichert sind. (bicbucstriim.php - Zeilen 521 bis 531).

In meiner Variante werden die Bücher anhand des Indexes der Serie sortiert, was ich bei Buchreihen durchaus für sinnvoll halte.

Returns a series and the related books

function seriesDetails($id) {
$series = $this->findOne('Series', 'select * from series where id='.$id);
if (is_null($series)) return NULL;
$books = $this->find('Book', 'select BSL.book, Books.* from books_series_link BSL, books Books where Books.id=BSL.book and series='.$id.' order by series_index');
return array('series' => $series, 'books' => $books);
}

Ist dies so von Ihnen beabsichtigt oder habe ich hier etwas übersehen?

Ich hoffe, dass Sie mir hier nicht falsch verstehen. Die Frage ist rein interessehalber, ich will keineswegs auf meinem Code beharren ;)

Mit freundlichen Grüßen
Benjamin Schmidt

-----Ursprüngliche Nachricht-----
Von: Rainer Volz [mailto:reply@reply.github.com]
Gesendet: Donnerstag, 2. August 2012 13:17
An: DarkHunter85
Betreff: Re: [BicBucStriim] Neue Funktionen (#3)

Ok, die Serienfunktionalität und die Erweiterung der Listenanzeige wurden in 0.9 übernommen.


Reply to this email directly or view it on GitHub:
#3 (comment)

@rvolz
Copy link
Owner

rvolz commented Aug 8, 2012

Das war dann wohl ein Versehen meinerseits. Ich habe das Serien-SELECT in findSlice übernommen, in seriesDetails aber nicht. Wird demnächst behoben. Vielen Dank für den Tipp!

@rvolz
Copy link
Owner

rvolz commented Aug 8, 2012

Die Sortierung wurde in 0.9.2 behoben, zum Download bereit.

@cnyncrvr cnyncrvr mentioned this pull request May 5, 2014
jampot5000 pushed a commit to jampot5000/BicBucStriim that referenced this pull request Jul 17, 2014
"error loading page" if searching for strings with ' in it:
exception 'PDOException' with message 'SQLSTATE[HY000]: General error: 1 near "brian": syntax error' in C:\Apache24\htdocs\bbs122\lib\BicBucStriim\calibre.php:292
Stack trace:
#0 ..\lib\BicBucStriim\calibre.php(292): PDO->query('select count(*)...')
#1 ..\lib\BicBucStriim\calibre.php(275): Calibre->count('select count(*)...')
#2 ..\lib\BicBucStriim\calibre.php(556): Calibre->findSliceFiltered('Book', 0, 30, Object(CalibreFilter), 'o'brian')
rvolz#3 ..\index.php(911): Calibre->titlesSlice('de', 0, 30, Object(CalibreFilter), 'o'brian')
rvolz#4 [internal function]: globalSearch()
rvolz#5 ..\vendor\slim\slim\Slim\Route.php(436): call_user_func_array('globalSearch', Array)
rvolz#6 ..\vendor\slim\slim\Slim\Slim.php(1307): Slim\Route->dispatch()
rvolz#7 ..\vendor\slim\slim\Slim\Middleware\Flash.php(85): Slim\Slim->call()
rvolz#8 ..\vendor\slim\slim\Slim\Middleware\MethodOverride.php(92): Slim\Middleware\Flash->call()
rvolz#9 ..\lib\BicBucStriim\calibre_config_middleware.php(50): Slim\Middleware\MethodOverride->call()
rvolz#10 ..\lib\BicBucStriim\login_middleware.php(33): CalibreConfigMiddleware->call()
rvolz#11 ..\lib\BicBucStriim\own_config_middleware.php(33): LoginMiddleware->call()
rvolz#12 ..\lib\BicBucStriim\caching_middleware.php(41): OwnConfigMiddleware->call()
rvolz#13 ..\vendor\slim\slim\Slim\Middleware\PrettyExceptions.php(67): CachingMiddleware->call()
rvolz#14 ..\vendor\slim\slim\Slim\Slim.php(1254): Slim\Middleware\PrettyExceptions->call()
rvolz#15 ..\index.php(196): Slim\Slim->run()
rvolz#16 {main}
JuergenBY referenced this pull request Jan 29, 2015
MoonReader et al try to download resources first without proper
AUTH-Headers, only after a 401 response they try again with AUTH-info.

Added separate routes for OPDS that answer with 401.
@Binog Binog mentioned this pull request Oct 18, 2016
@Deanna4711 Deanna4711 mentioned this pull request Nov 22, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants