Seite 4 von 4

Verfasst: Fr 13. Jul 2007, 13:56
von Butzemann
Klappt. Herzlichen Dank.

Verfasst: Fr 13. Jul 2007, 15:16
von 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);
        }
    }


Verfasst: So 22. Jul 2007, 14:42
von 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);
        }
    }

Verfasst: Mi 25. Jul 2007, 21:59
von der-domi
COOLover hat geschrieben: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.

Verfasst: Mi 25. Jul 2007, 22:04
von der-domi
Ich habe die Änderungen von Butzemann übernommen und stellenweise verbessert. Ich bitte um Testen.

class.import.php

Verfasst: Mi 25. Jul 2007, 22:11
von der-domi
der-domi hat geschrieben: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...

Verfasst: Sa 18. Aug 2007, 09:01
von 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

Re:

Verfasst: Mo 14. Jan 2008, 22:33
von der-domi
Hallo,

anbei ein Update für den OFDB-Import

Update

Verfasst: Mo 3. Mär 2008, 21:37
von 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

Re: ofdb hack

Verfasst: Di 4. Mär 2008, 00:24
von 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 :lol:

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 :D

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 :shock: :lol:

Der domi vermerkt dann die Namen in der History der filmdb :D :!:

Re: ofdb hack

Verfasst: Di 4. Mär 2008, 09:17
von itshorty
Butzemann hat geschrieben: 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
Butzemann hat geschrieben:Naja, das wars erstmal. Wenn ich wieder etwas Zeit habe, rüste ich die Dateien mal nach und poste Sie, mit deinem Einverständnis :D
Kein Problem, deswegen habe ichs geuppt!
Butzemann hat geschrieben: 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 :shock: :lol:

Der domi vermerkt dann die Namen in der History der filmdb :D :!:
Sorry, gewohnheit aus Xtreme Programming unter Java
//geändert

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

Re: ofdb hack

Verfasst: Di 4. Mär 2008, 23:24
von der-domi
Das Thema ist geschlossen. Bitte siehe in http://www.der-domi.de/board/viewtopic.php?f=1&t=272 nach!