search

Scout24 / restapi-php-sdk

SDK für PHP-Entwickler um die Arbeit mit der API von ImmobilienScout24 zu vereinfachen. Über die API erhält man Ergebnislisten, Objektdaten, uvm. von ImmobilienScout und kann diese Daten inhherhalb seines Webservice integrieren. Die Dokumentation zum SDK findet man unter https://github.com/Immocaster/php-sdk/wiki.
64 stars 46 forks source link

Datenspeicherung #11

Closed Jako closed 9 years ago

Jako commented 10 years ago

Leider ist das Umschalten der Datenspeicherung nur halb implementiert: in Rest.php wird explizit Immocaster_Data_Mysql benutzt.

Pixelairport commented 10 years ago

Ich muss kurz nachfragen. Nutzen Sie eine andere DB als MySql? Wenn ja, welche?

Jako commented 10 years ago

Eigentlich nicht, aber ich hab den Code für PDO angepasst (MODX/XPDO in meinem Fall).

Pixelairport commented 10 years ago

Bin grad etwas unsicher wie ich helfen kann. Mit Modx/XPDO hab ich noch nichts gemacht.

Jako commented 10 years ago

Hilfe brauche ich keine. Ich hab in meinem Fall in der Rest.php einfach alle Immocaster_Data_Mysql ersetzt.

Es ist nur etwas ungünstig, dass man in setDataStorage zwar eine andere Klasse angeben kann, diese aber nicht überall benutzt wird.

floriant commented 9 years ago

Wird dieser Teil jemals angepasst? Stand jetzt wirkt die Datenspeicherung auch für mich unflexibel. Wenn man diese SDK in ein Framework integrieren möchte, stellt sich das als extrem unschöne fumelei herraus.

Was ich als Entwickler mit einem Framework bräuchte währe die Möglichkeit einfach den Token zu bekommen. Dann kann ich ihn selbstständig speichern und gegebenenfalls an geforderter Stelle mit übergeben.

So habe ich halt auch nur die Optionen einen "Hack" zu machen, oder das ganze Ding selbst zu schreiben. :\

smisch commented 9 years ago

Ich möchte mich floriant anschließen: Die Immocaster SDK sollte die Finger von Session und Datenbankverbindung lassen und das Speichern und Laden der Tokens über ein Interface veröffentlichen.

Eine gute Orientierung bietet meiner Meinung nach die Paypal PHP Api.

fmielke commented 9 years ago

Hallo Leute!

Mit commit 3d897168282accd280031181893a0faf618cca18 ist es nun möglich einen User ohne MySQL Datenbank zu authentifizieren. Dazu nutzt in der index bitte $oImmocaster->authenticateWithoutDB(true);. Das Feld Benutzername muss nicht mehr gefüllt werden. Es wird anstelle der DB in eine Session gespeichert. In die Session werden Request Token und Secret gespeichert, nicht aber Access Token und Secret. Letzteres werden nach erfolgreicher Authentifizierung einmalig in einer Codebox in index.php aus der Rest.php heraus ausgegeben. Genügt das?

In 16bcfa507d685d59b6ed64457e55bb5a276189a8 wurde nachträglich noch auf die Session.php verzichtet und die Session nach Benutzen gelöscht.

Außerdem wird die DB-Speicherung nun komplett losgelöst von einer Session durchgeführt.

Entspricht das Euren Vorstellungen und ist es so besser integrierbar?

MfG Fabian

fmielke commented 9 years ago

Nachtrag: Mit merge 1208856d6855b3de9b56548030ab97e466bd1921 wurde das Handling noch ein wenig besser. Man erhält in der index ein Array zurück, wenn man ohne Datenbank authentifiziert. Das sollte die Weitergabe der Tokens an das eigene System vereinfachen und das Einbinden in ein Fremdsystem ermöglichen.

Oder wenn man einfach mal schnell Access Token und Secret haben möchte, geht das jetzt mit 1 Klick.