[UPD DE] Filmstarts

[yeti]

v2.2.17
-fixed actors and comments.

I've fixed it Saturday, too. But because of testing not uploaded yet

Greets,
yeti

[J]

lief jetzt leider ein wenig parallel, hatte mich gestern spontan entschieden mich mal wieder damit zu beschäftigen.

ja, testen macht immer am meisten Spaß, ich habe allein vier verschiedene Fälle für die Kommentarformatierung bei aktuellen Filmen gefunden. Ich habe aber auch keine sehr ausführlichen Tests gemacht, wahrscheinlich gibt es weitere Ausnahmen.

Du kannst dir ja mal meine Lösung anschaun (es sind nicht sehr viele Änderungen), wenn Deine robuster ist und noch mehr Varianten abdeckt, können wir natürlich auch diese nehmen.

Aber danke dass du noch dran bist, werde versuchen, mich zukünftig wieder rauszuhalten

[yeti]

Parallel ist kein Problem. Mir fallen Fehler im Bereich Kommentare sowieso nur selten auf, da ich die normalerweise nicht importiere. Merke ich dann nur bei Tests.

Deine Version ist ok. Du hast ja bei der Titelsuche auch noch etwas verändert, so dass mehr, aber auch nicht alles, gefunden wird. Ich hab auch noch nicht herausgefunden, wieso die Suche Mal funktioniert und Mal nicht. Aber % vor und hinter dem Suchstring findet den Titel meistens doch. Leider kann man die nicht standardmäßig dazu setzen, denn das funktioniert auch nicht immer. Ich habe das Gefühl, dass die Suche auf Seiten von Filmstarts manchmal anders verarbeitet wird.

Deine Actor-Extras sind ja auch nice, aber als ich die auf 0 gesetzt habe, war ich erstmal überrascht, dass nicht mehr nur die 8 mit Rolle, sondern alle Darsteller importiert werden

[J]

Das mit der Suche ist allerdings anstrengend, die funktioniert auch im Browser eigentlich nur online bei der Eingabe im Suchfeld (via ajax?) richtig.
Ich hänge an den Titel hier manchmal per Hand ein Leerzeichen an, oder muß Wörter im Titel ergänzen oder löschen, alles in allem sehr unbefriedigend. Leider habe ich auch noch keine allgemein brauchbare Lösung gefunden, daher probiere ich ab und an etwas aus, habe dann aber vergessen es wieder zurückzusetzen.

Die Actor-Extras sind eher eine optische Spielerei für den HTML-Viewer und sollten eigentlich per default auf 0 stehen (war wohl etwas spät am Abend) und ja, die Liste der Schauspieler habe ich auch mal ad hoc erweitert, da bei einigen Filmen auch bekanntere erst weiter hinten in einer Nebenrolle auftauchen. War mir bei beiden Sachen allerdings auch nicht recht sicher, ob es so nun besser ist und ist dann leider ein wenig eine Mischung aus fixing und ausprobieren geworden. Aber läßt sich ja beides recht einfach wieder ändern.

[yeti]

V2.2.18 uploaded

fixed description

Greets,
yeti

[antp]

V2.2.18 uploaded

Where?

[yeti]

Where?

http://ks3.antp.be/upload.php

[antp]

Thanks, I'll get the file and update it on the server
That is the link for the old server, for future uploads please use https://ks3c.antp.be/upload.php (of course you couldn't guess; I'll put a message on the old page)

[J]

damn, I totally missed the update from yeti and invented the wheel again

@antp
the version I send you per mail is also labeled as v2.2.18. You can delete it, I guess both scripts do the same.

Anyway, I uploaded my version to https://www75.zippyshare.com/v/mNDB5KeK/file.html and labeled it v2.2.18a, just in case somebody is interested. I added some more changes to this script, so you can test if this version more suitable for you.

sorry for the confusing, looks like I need to be faster next time

cheers
J.

[antp]

ok I restored it; Indeed when multiple people work on the same script it may be safer to download the latest version before working on it

[UPollaehne]

I have uploaded a new version (2.2.19, to https://ks3c.antp.be/upload.php) where I removed the "/1" from the search URL and changed the check for successful search from

Code: Select all

Pos('Filme</h2>', PageText) = 0

to

Code: Select all

Pos('Resultate', PageText) = 0

since the page seems to have line breaks between Filme and the closing h2 tag a lot of times.

[antp]

Thanks, it has been put with the other scripts available

[yeti]

New Version v2.2.20 uploaded (ks3c.antp.be...)

fixed: actors import

[antp]

Thanks, it is now on the update server

[yeti]

New Version v2.2.21 uploaded (ks3c.antp.be...)

fix: additional actors sometimes added twice
fix: special chars in the role-field not decoded
new: role from additional actors will be added too

greets

[antp]

Thanks

[yeti]

New Version v2.2.22 uploaded (ks3c.antp.be...)

fix: rating
fix: user review
fix (partially): workaround for a Delphi-Bug that breaks UTF8Decode() if the string contains a 4-Byte UTF-8 Code.

[antp]

Thanks

[yeti]

New Version v2.2.23 uploaded (ks3c.antp.be...)

chg: code cleaned up
- Indentation optimized (2 spaces per indent)
- All tabs removed
- Upper / lowercase corrected
- Comments rearranged / deleted
Most lines were changed because of this customization.

chg: workaround for UTF8Decode() optimized

[antp]

Thanks