[Allegro] phpac jetzt auch unter svn

[Klaus Lehmann]

On Fri, 08 Apr 2011 14:51:06 +0200 Bernhard Eversberg wrote:

guten tag herr eversberg
einige kleinigkeiten zum letzten stand des phpac (27.1.2011/8.4.2011)


1. in zahlreichen php's taucht ein schreibfehler im kommentar auf:
die "Vatianten" sind es wohl nicht....
include_once("css.php");        /* style Vatianten */
nennen wir sie "Varianten"?!



2. zur css.php:
<>Wir haben die PHP-Skripte von PHPAC neu ins svn eingebracht, im 
Repositorium "phpac".
<>Dort ist also nun der aktuelle Stand. Letzte Verbesserung: in 
grec.php und grex.php
<>war irgendwie eine Kleinigkeit herausgefallen, die für das Vormerken 
am OPAC
<>wichtig ist - das klappte deshalb nicht mehr.

evtl etwas übersehen?
// es wurde auf  Vormerken  geklickt
function vorm(recn)  // vbox fuellen und sichtbar machen, recn=TitelID
{   // vbox wird in grec.php/grex.php gesetzt, in av_ini.php ist CSS 
dafuer
aus der av_ini.php wurde alles, was nach "css" riecht, in die css.php 
verlagert, wa sich persönlich als gar nicht nötig finde. aber egal.
man müsste also in der css.php nachschauen, was zur vbox notiert ist. 
nur: da steht nichts!

grec-php sagt: 
<div id="vbox" style="visibility:hidden;"> </div>
da fehlt m.E. in der css.php etwas! 
oder doch nicht?



3. weiter in der css.php:
was sich mir in der css.php gar nicht erschliesst, ist diese zeile:
#box1 { position:absolute; top:200px; left:60%; width:35%; 
height:500px; z-index:1; }



4. in der av_mail.php ist von $addr die rede.
korrekt müsste es $adR sein.


5. weiter in der av_mail.php
da ist die rede von $output
$k=sizeof($result);  // Anzahl Zeilen
if($k<1 || strstr($output[$k-1],"Sorry, no results")) echo "Sorry\n";
ich bin ziemlich sicher, daß es $result heissen muß



6. zum allregister: genauer zur datei av_all.php
ich habe lange gebraucht, um das fehlverhalten beim all-register 
genauer zu beschreiben. da ich in der letzten woche nun mit dem 
"entflöhen" des phpac's vom 27.1.2011 beschäftigt war, konnte ich es 
endlich am schopfe packen.
man kann mit dem all-register mit vorkommenden umlauten suchen. es gibt 
nur ein ergebniss für die "erste suchseite". die zweite seite wird 
verweigert.
nachzuspielen hier:

http://tst.allegronet.de/ruck-zfl/
man suche oben, beim "universellen Wortregister" nach den beiden 
worten:
müller geschichte .
prima. die erste suchseite mit 40 treffern kommt!
klicken man auf weiter: niente!

die suchzeile ist: 
http://tst.allegron
et.de/ruck-zfl/rset.php?ufC=ALL%20m%EF%BF%BDller%20and%20ALL%20geschicht
e&uzA=40&uzS=41&urO=d57
der emailclient spielt jetzt leider nicht mit!
man sieht: ufC=ALL mzzzller and ALLgeschichte
da, wo ich "zzz" geschrieben habe, ist ein schwarzes karo zu sehen, mit 
einem ? innen drinen.
die auflösung von "ü" nach "ue" klappt nicht!

der fehler sitzt m.E. in av_all.php:
/* $ufC muss mit URL-Codierung weitergegeben werden! */
$job = array(
/* Inhalt der PHP-Variablen in die entspr. avanti-Variablen kopieren: 
*/

"switch coding 0",  // keine Umcodierung ASCII<->ANSI

"var '$UTF'",
"ins #uTf",
"var '$uzA'",
"ins #uzA",
"ins #uzB",
"var '$uzS'",
"ins #uzS",
"var '$uzW'",
"ins #uzW",

das macht er! 
deshalb[?] sehen wir die erste suchseite bestens!
aber die zweite suchseite, mit "Weiter" geklickt, die wir sehen können, 
bringt uns keine ergebnisse. 
hier fehlt "etwas", was die suchwörter an den index "aufgelöst" 
weitergibt.

das muss vielleicht bei :lop eingebaut werden?



punkte 1-5 habe ich für mich herausgefunden. das sind nur hinweise zur 
korrektur des phpac-pakets.
punkt 6 ist wichtig!


vielen dank. 
viele grüße, ihr klaus lehmann






Klaus Lehmann
eMail: allegronet at t-online.de
phone: 03528-452 807(fax 809)
mobil: 0171-953 7843
adress: allegronet.de / Klaus Lehmann
D-01454 Radeberg; Kleinwolmsdorfer Str. 37
http://allegronet.de
** Internetkataloge & WebHosting für AllegroC **
** Software für zufriedene Bibliothekare: 1000x bewaehrt und ergiebig **
** Yes we can. Only with allegro. Yes we do. **
** In 2010 bereits die 4 erfolgreiche Installation von allegro-utf8! cjk für alle! **
** Buchen Sie die allegro-Roadshow: Neues aus der allegroWerkstatt **