ofdb hack

[Butzemann]

Klappt. Herzlichen Dank.

[Butzemann]

Einen kleinen Fehler bei der Suche nach "bestimmten" Titeln aus der OFDB. Wenn es Titel gibt, die auch Titel aus dem "Hardcorebereich" sind, so erscheint ein Hinweis (Als Beispiel: Predator, oder Hostel):

"» Es gibt x weitere(n) Treffer aus dem Hardcore-Bereich. (Weitere Informationen)"

Das zersprengt dann regelrecht die komplette Seite.

Es müssten also in der import Datei Veränderungen zu diesem Segment durchgeführt werden:

Code: Alles auswählen

    function ofdbsearch() {
        
        if ( !strlen($_GET["searchText"]) )
            return -1;
        
        // url to get search result
        $this->url = "http://www.ofdb.de/view.php?page=suchergebnis&SText=" . urlencode($_GET["searchText"]) . "&Kat=Titel";
        
        if ( loadFile( $this->url, $this->buf1, "r" ) >= 0 ) {
            
            $this->buf2 = explode( "<b>Titel:</b><br><br>", $this->buf1 );
            $this->buf1 = $this->buf2[1];
            $this->buf2 = explode( "<br><br><br></font></p><br>", $this->buf1 );
            $this->buf1 = $this->buf2[0];
            
            $this->buf2 = explode( "<br>", $this->buf1 );
            
            for ( $i=0; $i<count($this->buf2) && $i<25; ++$i ) {
                
                $pattern = "/[0-9]*\.\s<a href=\"view.php\?page=film&fid=|\">|<font size=\"1|\s\/\s|<\/font>\s\(|\)<\/a>/";
                $this->buf1 = preg_split ($pattern, trim ($this->buf2[$i]), -1, PREG_SPLIT_NO_EMPTY );
                
                $this->searchResult[$i] = array();
                $this->searchResult[$i]['title'] =  $this->buf1[1];
                $this->searchResult[$i]['id'] =  $this->buf1[0];
                $this->searchResult[$i]['year'] =  $this->buf1[3];
                
                if ( $this->buf1[1] != $this->buf1[2] )
                    $this->searchResult[$i]['alt'] =  $this->buf1[2];
                else
                    $this->searchResult[$i]['alt'] =  "";
            }
            //print_r($this->searchResult);
        }
    }

[Butzemann]

Auch das Problem wurde nun durch einen "simplen" Eingriff gelöst:

Code: Alles auswählen

    function ofdbsearch() {
        
        if ( !strlen($_GET["searchText"]) )
            return -1;
        
        // url to get search result
        $this->url = "http://www.ofdb.de/view.php?page=suchergebnis&SText=" . urlencode($_GET["searchText"]) . "&Kat=Titel";
        
        if ( loadFile( $this->url, $this->buf1, "r" ) >= 0 ) {
            
            $this->buf2 = explode( "<b>Titel:</b><br><br>", $this->buf1 );
            $this->buf1 = $this->buf2[1];
            $this->buf2 = explode( "<br><br><br>", $this->buf1 );
            $this->buf1 = $this->buf2[0];
            
            $this->buf2 = explode( "<br>", $this->buf1 );
            
            for ( $i=0; $i<count($this->buf2) && $i<25; ++$i ) {
                
                $pattern = "/[0-9]*\.\s<a href=\"view.php\?page=film&fid=|\">|<font size=\"1|\s\/\s|<\/font>\s\(|\)<\/a>/";
                $this->buf1 = preg_split ($pattern, trim ($this->buf2[$i]), -1, PREG_SPLIT_NO_EMPTY );
                
                $this->searchResult[$i] = array();
                $this->searchResult[$i]['title'] =  $this->buf1[1];
                $this->searchResult[$i]['id'] =  $this->buf1[0];
                $this->searchResult[$i]['year'] =  $this->buf1[3];
                
                if ( $this->buf1[1] != $this->buf1[2] )
                    $this->searchResult[$i]['alt'] =  $this->buf1[2];
                else
                    $this->searchResult[$i]['alt'] =  "";
            }
            //print_r($this->searchResult);
        }
    }

[der-domi]

Ich will mal nicht so sein, ...

Danke für die Änderungen. Wenn man voraussetzt, dass alle Kategorien vorhanden sind, ist es gar nicht so viel
Bei deiner Umsetzung kann es vorkommen, dass mehrmals das checked-Attribut ausgegeben wird. Das werde ich in meiner Änderung verbessern.

Warum hast du denn nicht erwähnt, dass du nur einen Wegweiser benötigst, damit du loslegen kannst. So sah es aus, dass du erstmal Andere schaffen lassen wollen.

[der-domi]

Ich habe die Änderungen von Butzemann übernommen und stellenweise verbessert. Ich bitte um Testen.

class.import.php

[der-domi]

Ich habe die Änderungen von Butzemann übernommen und stellenweise verbessert.

@Butzemann:
Es wäre gut, wenn du das Modul/Klasse synchronisieren könntest. Dadurch ist eine weitere Pflege einfacher. Deine Designänderungen in den print-Funktionen kannst du ja unverändert lassen. Ansonsten alles übernehmen...

[der-domi]

Hi,

mich hat es gestört, dass die Inhaltsangabe vom IMDB-Import nicht ganz so ordentlich aussieht. Deshalb habe ich den ganzen Klumb entfernt. Ich wußte nicht, dass es sich dabei um mehrere, verschiedene Inhaltsangaben handelt. Butzemann hatte mich darauf hingewiesen. Ich habe es für die neue Version vergessen. Blabla...

Hier die neue Version: class.import.zip

[der-domi]

Hallo,

anbei ein Update für den OFDB-Import

[itshorty]

Hallo, da nach einigen Umstellungen auf der ofdb.de Hompage die Importfunktion nicht mehr funktioniert hat, habe ich die class.import.php upgedated. Dies hat Änderungen an den URLs und an der Such RegEx erforderlich gemacht.

Diff:
http://nopaste.ch/89cfecb0d8761b3.html

class.import.php:
http://nopaste.ch/afde56a4add63bb.html

[Butzemann]

Hi,
hab mir mal auf die Schnelle deine Datei angeguckt.

Erstmal danke für die Veränderungen. Hab sogar noch Fehler in meiner Datei gefunden, die du gleich mit berichtigt hast

Finde den Schritt eine zusätzliche Variable einzuführen auch sehr sinnvoll. Lifestyle hatte dieselbe Idee. Wir haben es aber erstmal nicht in Angriff genommen. Gut, dass du das gemacht hast.

Und auch super finde ich, dass das endlich mit dem Schrägstrich in der URL gelöst worden ist.

2 Punkte sind aber noch m.M. offen.

1. Beim Cover-Import der OFDB hast du einen zusätzlichen Link auf imageshack angegeben. Der auf die ofdb würde auch gehen -> http://www.ofdb.de:81/film/na.gif
2. Ich weiß, dass es hier etwas an der Übersichtlichkeit fehlt, deswegen habe ich auch den domi gebeten, einen "fixierten" Thread im Forum "FilmV: Installation, Update & Fehlerbehebung" einzurichten, der immer nur die aktuellste class.import Datei enthält. Ich habe nämlich in diesen Forum Ende Febr. jeweils die Design -und Originalversion der class.import Datei gepostet. Ich weiß nicht, ob du diese zur Grundlage deiner Veränderungen genommen hast, aber anscheinend schon. Trotzdem hast du die IMDB Veränderung(en) in deiner Datei nicht berücksichtigt...

Naja, das wars erstmal. Wenn ich wieder etwas Zeit habe, rüste ich die Dateien mal nach und poste Sie, mit deinem Einverständnis

Ach ja, eine Sache noch.

Ich find's ja gut, dass du dich an diesem Projekt beteiligts, nur extra in die Datei noch

Code: Alles auswählen

//Updated auf neue ofdb.de Hompage by itshorty

zu schreiben? Wenns danach gehen würde, würden ziemlich viele User Erwähnung in den Dateien finden

Der domi vermerkt dann die Namen in der History der filmdb

[itshorty]

1. Beim Cover-Import der OFDB hast du einen zusätzlichen Link auf imageshack angegeben. Der auf die ofdb würde auch gehen -> http://www.ofdb.de:81/film/na.gif

Habe wie du villeicht bemerkt hast ein anderes Bild verwendet, da ich dieses zu meinem Design passt.
//geändert

Naja, das wars erstmal. Wenn ich wieder etwas Zeit habe, rüste ich die Dateien mal nach und poste Sie, mit deinem Einverständnis

Kein Problem, deswegen habe ichs geuppt!

Ach ja, eine Sache noch.

Ich find's ja gut, dass du dich an diesem Projekt beteiligts, nur extra in die Datei noch

Code: Alles auswählen

//Updated auf neue ofdb.de Hompage by itshorty

zu schreiben? Wenns danach gehen würde, würden ziemlich viele User Erwähnung in den Dateien finden

Der domi vermerkt dann die Namen in der History der filmdb

Sorry, gewohnheit aus Xtreme Programming unter Java
//geändert

class.import.php:
http://nopaste.ch/af9f32131c4895a.html

[der-domi]

Das Thema ist geschlossen. Bitte siehe in http://www.der-domi.de/board/viewtopic.php?f=1&t=272 nach!