Diese Board wurde aus Spamschutz-Gründen auf minimale Funktionen beschränkt.
Nutze für neue Anfragen/Bug-Reports bitte meine Gitlab-Instanz.

ofdb hack

Was könnte man an der Filmverwaltung verbessern? Fragen und Anregungen von Entwickler an Entwickler.

Moderator: Lifestyle

Butzemann
Beiträge: 139
Registriert: Mi 20. Dez 2006, 15:22
Kontaktdaten:

Beitrag von Butzemann »

Klappt. Herzlichen Dank.
Butzemann
Beiträge: 139
Registriert: Mi 20. Dez 2006, 15:22
Kontaktdaten:

Beitrag 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);
        }
    }

Butzemann
Beiträge: 139
Registriert: Mi 20. Dez 2006, 15:22
Kontaktdaten:

Beitrag 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);
        }
    }
Benutzeravatar
der-domi
Site Admin
Beiträge: 673
Registriert: Di 4. Apr 2006, 19:22
Kontaktdaten:

Beitrag 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.
Nicht verzagen, Domi fragen

Viele Grüße
Dominik
Benutzeravatar
der-domi
Site Admin
Beiträge: 673
Registriert: Di 4. Apr 2006, 19:22
Kontaktdaten:

Beitrag von der-domi »

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

class.import.php
Nicht verzagen, Domi fragen

Viele Grüße
Dominik
Benutzeravatar
der-domi
Site Admin
Beiträge: 673
Registriert: Di 4. Apr 2006, 19:22
Kontaktdaten:

Beitrag 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...
Nicht verzagen, Domi fragen

Viele Grüße
Dominik
Benutzeravatar
der-domi
Site Admin
Beiträge: 673
Registriert: Di 4. Apr 2006, 19:22
Kontaktdaten:

Beitrag 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
Nicht verzagen, Domi fragen

Viele Grüße
Dominik
Benutzeravatar
der-domi
Site Admin
Beiträge: 673
Registriert: Di 4. Apr 2006, 19:22
Kontaktdaten:

Re:

Beitrag von der-domi »

Hallo,

anbei ein Update für den OFDB-Import
Dateianhänge
class.import.zip
(6.22 KiB) 954-mal heruntergeladen
Nicht verzagen, Domi fragen

Viele Grüße
Dominik
itshorty
Beiträge: 2
Registriert: Mo 3. Mär 2008, 21:18
Wohnort: Wels|Esternberg - Austria
Kontaktdaten:

Update

Beitrag 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
Zuletzt geändert von itshorty am Di 4. Mär 2008, 09:37, insgesamt 1-mal geändert.
//MFG|Best Regards itshorty
Butzemann
Beiträge: 139
Registriert: Mi 20. Dez 2006, 15:22
Kontaktdaten:

Re: ofdb hack

Beitrag 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 :!:
itshorty
Beiträge: 2
Registriert: Mo 3. Mär 2008, 21:18
Wohnort: Wels|Esternberg - Austria
Kontaktdaten:

Re: ofdb hack

Beitrag 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
//MFG|Best Regards itshorty
Benutzeravatar
der-domi
Site Admin
Beiträge: 673
Registriert: Di 4. Apr 2006, 19:22
Kontaktdaten:

Re: ofdb hack

Beitrag von der-domi »

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

Viele Grüße
Dominik
Gesperrt

Wer ist online?

Mitglieder in diesem Forum: 0 Mitglieder und 1 Gast