Skip to content

IABookreader OCR search has exceptions #195

@DiegoPino

Description

@DiegoPino

What?

Well there is an issue here..

var searchInsideResultsScale = {};
searchInsideResults.matches.forEach(function(match,index,array) {
match.par[0].boxes.forEach(function(box,index,array) {
var pageindex = self.leafNumToIndex(array[index].page);
array[index].l = Math.round(box.l * self.getPageWidth(pageindex));
array[index].t = Math.round(box.t * self.getPageHeight(pageindex));
array[index].r = Math.round(box.r * self.getPageWidth(pageindex));
array[index].b = Math.round(box.b * self.getPageHeight(pageindex));
})
});

      // ZOOM IN .. HERE!
          match.par[0].boxes.forEach(function(box,index,array) {
          });

We assume here that match.par will always have first entry with boxes and other data. Happens that sometimes Solr will return this!
When that does happen the search box gets stuck (JS error!)

Array (50) = $1
0 {par: Array, text: "aus einer malerisehen, steilen Felsschlncht herabb…n hin, links und vor uns hatten wir prichtige Fer"}
1 [] (0)
2 [] (0)
3 {par: Array, text: "Lycaena escheri ist ein langsaner Flieger, der die…orquiui flicgt sehr langsam und gemichlich und zw"}
4 {par: Array, text: "spiter, nachdem er bereits die Tiere aus dem Banat…ten G'¢f' messen kaum 80 mm Linge | vorgezeigt. '"}
5 [] (0)
6 {par: Array, text: "“bung ist variabel: Hs gibt hellbraune Exemplare, …nan, wenn Im Trebitscher Bezirke ist sie schou au"}
7 {par: Array, text: "streifens zwei oder mehrere, so lassen sich an zwe… mit zwei an einer Stelle zusammenhingenden Wohn-"}
8 {par: Array, text: "kaun. aufhebt. Unser Cortijo schien der Sammelpunk… hifilich; die wenig saubere Kleidung, die lider-"}
9 {par: Array, text: "mehr als 11/, cm Linge geklebt werden, wenn sie nu…n sind doch wobl allgemein mit Torf ausgelegt, in"}
10 [] (0)
11 [] (0)
12 {par: Array, text: "schemuny und Wirkung, nwr als Landwind, d. L. vou …nertriglicher; ich habe Tage und Nichte in Girana"}
13 {par: Array, text: "es se1 die betreffende Form einer Spezies oder ein…jektiv ,typisch 1t aber nicht erkennen, ob es von"}
14 {par: Array, text: "gepaart mit Leichtigkeit und Kraft. Die Einlenkung…uch stabformig (alsc nicht hohl) oder flach sind."}
15 {par: Array, text: "eten Banditen tiberfallen, und nachdem die Dieners…he Verhand- lungen dieserhalb eingelcitet worden."}
16 [] (0)
17 {par: Array, text: "und am néchsten Morgen den Muley-Hazen hitten erst…nuten. Hr meinte sie an ivgend einer Stelle des]'"}
18 {par: Array, text: "die als Bindematerial dienen. Auf den Bergtouren w… die Tmosaoke der Esel, Serong gevannt, gefertigh"}
19 {par: Array, text: ",rberechnet, Kornerfutter einkaufen und unsere Wei…afl verschiedene Leute da wiren, die schon 6iters"}
20 {par: Array, text: "dieselbe bleibt. Diese Lebensgewohnheit — wenn man…d der Periodizitit der Jahreszeiten, und die ver-"}
21 {par: Array, text: "Tine groferc Anzah! franzisischer Gelehrter, die P…gte Lamarck, wie es moglich ist, die Vorginge der"}
22 [] (0)
23 {par: Array, text: "| dem Cortijo an; doch ritten wir nicht, sondem zo…a, Eucosmia montivagata) aufgescheucht und wenige"}
24 {par: Array, text: "duktivitit der Kongresse* 2. ,,Der Besuch der Muse…utung verlieren. Diese Erscheinung hat verschiede"}
25 {par: Array, text: "| 1800 hat Lichtenstein eine DMMiickenlarve als Ch…gen-Kongrefl von 1905 das Gesetz aufgestellt, daf"}
26 {par: Array, text: "eignet gehalten. Erst heute, wo ich diese Zeilen s… laBt sich wohl auch die Misteilung des Herrn Sle"}
27 [] (0)
28 [] (0)
29 [] (0)
30 {par: Array, text: "Schon begann das Tagesgrauen, dennoch safien noch …verlassen sollten. Ks war inzwischen Tag geworden"}
31 {par: Array, text: "nicht ausgesaugten im Laufe des niichsten Tages. A…allene Kameradin dem Ameisenldwen zn _entreiflen,"}
32 [] (0)
33 [] (0)
34 [] (0)
35 {par: Array, text: "lwirkenden Faktor bei der Taschenbildung eines mem…cudder (1892) weitere Aufmerksamkeit durch Untes-"}
36 [] (0)
37 [] (0)
38 [] (0)
39 {par: Array, text: "dieser nordamerikanischen Lokalrasse eine Form vor…konomierat Valentin Wiist begeht Ende August sein"}
40 [] (0)
41 {par: Array, text: "hat und mit Hilfe des Pzimitéitspliluip% ihre . Mi…, wird’s auch weiter tun. Aber wir andern, sollen"}
42 [] (0)
43 [] (0)
44 {par: Array, text: "Meine (enossen brachen schon kurz nach 5 Uhr auf, …e Tiere gesattelt und beladen und ging es nun auf"}
45 [] (0)
46 {par: Array, text: "faltern, fand aber nur einige tote und zerfetzte, … man daraus, daBl die ganze Stadt Krasnowodsk von"}
47 [] (0)
48 Array (0)
No Properties

Array Prototype
49 {par: Array, text: "Kokons, Larven, Eiemn hineinschiebt. Das geplinder…ildeite Fangmethode eignet sich fitr den Frihling"}

See the empties?

For sanity we should check if match.par[0].boxes exist

@patdunlavey @giancarlobi I might also clean up the OCR controller to remove empties...

Metadata

Metadata

Assignees

Labels

IIIFSpecs/Manifests/ImplementationsJavascriptFavourite language of a PHP developerUXLike UI but with an XbugSomething isn't working

Type

No type

Projects

No projects

Relationships

None yet

Development

No branches or pull requests

Issue actions