Addon Installer [REX 4.2, 4.3 | JETZT FULL AJAX]

[Hirbod]

ACHTUNG: NEUE VERSION UNTER http://www.redaxo.org/de/forum/post88705.html

[Hirbod]

Konnte nicht schlafen und habe bis jetzt noch am Addon gearbeitet und eine Echtzeitliste aller verfügbaren Addons hinzugefügt. Mit 2 Klicks lässt sich ein Addon automatisch installieren, ohne auch nur an FTP zu denken

Version 0.8 changed to 1.0 stable

LG
Hirbod

BITTE BITTE FEEDBACK

[Richard.Laing]

hmmm
komme leider nicht weit beim install:

Code: Alles auswählen

Fatal error: Cannot redeclare file_get_html() (previously declared in /var/www/xxx/redaxo/include/addons/xsearch/plugins/plaintext/classes/class.simple_html_dom.inc.php:40) in /var/www/xxx/redaxo/include/addons/addoninstaller/classes/dom.class.php on line 41

und unter hilfe wird die headmap angezeigt
gruß richy

[Hirbod]

Vielen Dank Richard, das Problem trifft auf, da xSearch und mein Addon anscheinend die selbe Klasse verwenden - oder zufälligerweise den gleichen Namen. Falls der Fehler jetzt noch mal auftritt, bitte ich dich das xSearch-Addon kurz zu deaktivieren, oder ich ändere einfach meine Funktionsnamen.

Version 1.0 changed to 1.4

- Bug von Richard _endgültig_ gefixed
- Beschreibung hinzugefügt
- Optik und Tabllen verbessert
- Performance optimiert

Download: http://www.redaxo.de/180-0-addon-detail ... don_id=823

Bitte weiter testen und alle möglichen Bugs bitte melde. Danke

[jdlx]

oder ich ändere einfach meine Funktionsnamen.

Ein gängiger Weg sowas zu vermeiden ist ein Addon-präfix im Funktionsnamen.. ala

Code: Alles auswählen

a[RXID]_meine_funktion()

lg,
j.

[Hirbod]

Genau so ist es jetzt auch. Hab ich gerade in der Version 1.3 eingebaut.
Was sagst du zum Addon?

[Jan.Kristinus]

Hallo,

ohne es getestet zu haben, cool .. du nimmst uns damit viel arbeit ab .. kannst du mich wegen api & co anrufen ?

lg

jn

[Hirbod]

Hallo Jan,

ich habe gleich ein paar Termine, kann dich aber gerne heute Nachmittag anrufen.


@ Ansonsten:

Update auf Version 1.5
- doppelte declaration von class behoben

EDIT:

Weitere 5 Bugs entfernt. Version geändert auf 1.5

[jdlx]

Was sagst du zum Addon? :)

Sehr fein das!
.. jetzt noch Module, und externe URLs.. ;->

lg,
j.

[Hirbod]

Was sagst du zum Addon?

Sehr fein das!
.. jetzt noch Module, und externe URLs.. ;->

lg,
j.

Externe URLs sind kein Problem. Das sind ein paar Zeilen Code zu ändern. Ich habe bereits daran gedacht, ebenfalls einen Modulinstaller zu implementieren, das geht aber wirklich nur, wenn mir Jan ne API stellt, sonst wird das ein regExp Chaos

[Hirbod]

Update von 1.5 auf 1.6

Neu:
- Es wird jetzt die zum Addon kompatible Redaxo-Version angezeigt.
- 2 kleine Bugfixes, welche mit simple_php_dom aufgetreten sind

Download über die Addonseite!

[Hirbod]

Update von 1.6 auf 1.7

Neu in dieser Version:

- Kompletter Rewrite des Addon.
- Manueller Installer vorerst entfernt (wird in anderer Form wieder eingesetzt)
- Backend neu gestaltet
- Beschreibung, Hinweise und Version werden jetzt angezeigt
- Full Ajaxified

Todo:
- JavaScript-Fallback erstellen.
- Suche integrieren
- Modul Installation implementieren
- Template Installation implementieren
- Backend-Skin Installation implementieren

Bitte alle die neue Version laden. Alle die die alte Version installiert haben, sollten den Addon Installer eigentlich sogar mit dem Addon Installer updaten können, da alle files überschrieben werden.

Bitte Bugs, Change Request etc. durchgeben.

[Xong]

Hi Hbod!

Ich habe kein Curl installiert und habe deswegen die Curl-Funktionen umgeschrieben:

Code: Alles auswählen

<?php
	function download_file_from_redaxo($file, $includeDir)
	{
	    file_put_contents($path, file_get_contents($url));
	}
	
	function get_time_of_file($file)
	{
		foreach(get_headers('http://www.redaxo.de'.$file) as $h)
		{
      if(0 === strpos($h, 'Last-Modified: '))
        return date('d.m.Y H:i:s', strtotime(substr($h, 15)));
		}
		
		return 'Keine Angabe';
	}
?>

Desweiteren wäre es schön, bei Ajax-Requests ein Ladebild anzuzeigen.

Außerdem solltest du die temporäre zip-Datei im Ordner /files/addons/addoninstaller speichern.

Ansonsten ein tolles Addon.

Was passiert eigentlich, wenn das ausgewählte Addon schon installiert ist?

[Gregor.Harlan]

Hallo,

Hi Hbod!

Ich habe kein Curl installiert und habe deswegen die Curl-Funktionen umgeschrieben:

Code: Alles auswählen

<?php
	function download_file_from_redaxo($file, $includeDir)
	{
	    file_put_contents($path, file_get_contents($url));
	}
	
	function get_time_of_file($file)
	{
		foreach(get_headers('http://www.redaxo.de'.$file) as $h)
		{
      if(0 === strpos($h, 'Last-Modified: '))
        return date('d.m.Y H:i:s', strtotime(substr($h, 15)));
		}
		
		return 'Keine Angabe';
	}
?>

Für deine Variante muss allerdings dafür allow_url_fopen eingeschaltet sein. Optimal (aber auch am aufwändigsten) wäre es mit fsockopen zu arbeiten!

Gruß, Gregor

[Hirbod]

Hi Hbod!

Ich habe kein Curl installiert und habe deswegen die Curl-Funktionen umgeschrieben:

Code: Alles auswählen

<?php
	function download_file_from_redaxo($file, $includeDir)
	{
	    file_put_contents($path, file_get_contents($url));
	}
	
	function get_time_of_file($file)
	{
		foreach(get_headers('http://www.redaxo.de'.$file) as $h)
		{
      if(0 === strpos($h, 'Last-Modified: '))
        return date('d.m.Y H:i:s', strtotime(substr($h, 15)));
		}
		
		return 'Keine Angabe';
	}
?>

Desweiteren wäre es schön, bei Ajax-Requests ein Ladebild anzuzeigen.

Außerdem solltest du die temporäre zip-Datei im Ordner /files/addons/addoninstaller speichern.

Ansonsten ein tolles Addon.

Was passiert eigentlich, wenn das ausgewählte Addon schon installiert ist?

Hi Xong,

vielen Dank für dein Input. Zu deinen Vorschlägen:

Das Ladebild wird in der nächsten Version eingefügt. Wollte ich in dieser Version eigentlich, hab ich jedoch zum Schluss vergessen.

Thema cURL: Die meißten haben curl installiert, und bei den meißten wird allow_url_fopen() deaktiviert sein. Von daher werde ich wohl deine umgeschriebene Funktion nicht übernehmen... obwohl, ich könnte ja abfragen, ob curl zur Verfügung steht, wenn nicht, wird deine Funktion genutzt.

Temp: Ich speicher das file in /files/addons/addoninstaller/temp - was ist daran falsch? Meinst du es ist besser, auf den Ordner zu verzichten und einfach in den Hauptordner zu schreiben?

Existierende Addons: Werden gnadenlos überschrieben... Soll ich prüfen, ob das Addon schon existiert, falls ja, unzip abbrechen? Oder einfach überschreiben? Was haltet ihr für besser?

Des Weiteren sind Jan und ich momentan in einer Art Vorplanung, das Addon wird sich noch gewaltig ändern und soll so einiges mehr können, als aktuell. Mehr dazu demnächst.

---

Nachtrag @Xong:

Das was du umgeschrieben hast, kann aber gar nicht funktionieren, oder?

function download_file_from_redaxo($file, $includeDir)
{
file_put_contents($path, file_get_contents($url));
}

du übergibst $file, $includeDir, nutzt bei file_put_contents jedoch $path und $url

Da stimmt was nicht, würd ich sagen

[zehbaeh]

Ansich eine praktische Sache. Ich würde allerdings NIE ein Modul/Addon aus nicht vertrauenswürdigen Quellen* ohne vorheriges Code-Review* einsetzen.

*Quellen: Redaxo ist in diesem Fall auch eine nicht vertrauenswürdige Quelle, da der zur Verfügung stehende Code i.d.R. ebenfalls keinem Code-Review unterzogen wurde.

*Code-Review: Auf "bösartigen"/fehlerhaften Code überprüfgen.
Zb. auf nicht beachtete Kontextwechsel, was ein sehr verbreiteter Fehler ist.

[jdlx]

Ansich eine praktische Sache. Ich würde allerdings NIE ein Modul/Addon aus nicht vertrauenswürdigen Quellen* ohne vorheriges Code-Review* einsetzen.

Was genau hat das mit dem Addon zu tun? Das man blind installiert? Wer bitte reviewt den Code der Addons/Module die er einsetzt.. du?

lg,
j.

[Richard.Laing]

hi@all
zur veranschaulichung

ein kurzes video dazu
http://www.rexvideo.de/tutorials/videot ... ation.html

gruß richy

[zehbaeh]

Was genau hat das mit dem Addon zu tun? Das man blind installiert? Wer bitte reviewt den Code der Addons/Module die er einsetzt.. du?

Ja. Das bin ich meinem Auftraggeber schuldig!
Sollte der Auftraggeber selbst die Möglichkeit erhalten Addons / Module zu installieren, müsste ich in jedem Fall auch auf die damit verbundenen Gefahren hinweisen. Mir von Ihm bestätigen lassen das alle damit verbundenen folgenden Aufwendungen in seiner Verantwortung liegen.

PS-Edit: Für das Code-Review hab ich schon an einer halbautomatischen Lösung "gebastelt", bisher aber nur mit für mich nicht befriedigenden Erfolg.

[jdlx]

Sollte der Auftraggeber selbst die Möglichkeit erhalten Addons / Module zu installieren, müsste ich in jedem Fall auch auf die damit verbundenen Gefahren hinweisen.

Mußt ihm ja nicht die Rechte geben.. wenn du sie im gibts, dann ists quasi so, als hätte er FTP Zugriff für ./addons. Das seh ich aber nicht als Problem dieses Addons.. genausowenig gibt man einem normalen User ja normalerweise Rechte für System, Imp./Exp., usw..

lg,
j.

[Hirbod]

hi@all
zur veranschaulichung

ein kurzes video dazu
http://www.rexvideo.de/tutorials/videot ... ation.html

gruß richy

Hi Richy,

vielen Dank
Super Video

[Richard.Laing]

hi hbod
wie verhällt es sich denn bei plugins?
hab ich noch nicht ausprobiert
gruß richy

[Hirbod]

Hi Richard,

für Plugins habe ich noch nichts programmiert. Habe mich bisher auch nicht damit beschäftigt. Muss ich mal testen.

LG
Hirbod

[elwood]

Moin,

der Addon-Installer kommt mit der neuen .org-Domain nicht klar. Die http://www.redaxo.de -URL ist hardgecodet.

Gruss, Elwood