search

nightstomp / rex_multiupload

Implements a state of the art multiple ajax fileupload-ulility in redaxo with lot of functions. Does not require Flash. Works even on smartphones (min. iOS 6). Brought to you by nightstomp.com - web development frankfurt | redaxo | experts for mobile devices like iPhone/iPad and Android systems
http://nightstomp.com
13 stars 4 forks source link

Multiupload / Medienpool #6

Closed olien closed 11 years ago

olien commented 11 years ago

Hi Hirbod,

nach der Installation und Aktivierung des Addons wird der Medienpool nicht mehr angezeigt.

Fehler:

Fatal error: rexCallFunc: Function "rex_multiupload_menu_insert" not found! in /Applications/MAMP/htdocs/rex45b/redaxo/include/functions/function_rex_extension.inc.php on line 157

REDAXO: 4.5b6.0 PHP: 5.4.4

LG Oliver

ghost commented 11 years ago

kann @olien s meldung bestätigen. tritt bei mir auch auf.

REDAXO: 4.5.0 (eine der neusten Versionen aus GitHub) PHP: 5.4.7

hirbod commented 11 years ago

Wie kann das sein? :S - Bin total verwirrt... Hier und auf der CeBIT hat das sauber funktioniert...

ghost commented 11 years ago

hmmm, ich verstehe es auch noch nicht. habe gerade mal oberflächlich draufgeschaut. hier mal paar fragen um das prob vielleicht eingrenzen zu können: php version? existiert funktion rex_multiupload_menu_insert wirklich? neuste rex beta genommen? mal so probiert (nur halt umschreiben für die mp ep's): https://github.com/redaxo/redaxo4/issues/94#issuecomment-13180197

gharlan commented 11 years ago

Ich hab das Addon gerade mal geladen und kann den Fehler in der aktuellen 4.5 nicht nachvollziehen. Wenn man Redaxo von github verwendet, muss man nur dran denken, die class.compat.php einzukommentieren. Aber wenn man das vergisst, kommt eine andere Meldung als die oben.

PS: @Hirbod Hab nur kurz drüber geschaut, aber sieht schick aus :)

gharlan commented 11 years ago

@RexDude Kannst du mal schauen, was $include_files bei dir an dieser Stelle enthält?

ghost commented 11 years ago

witzig, wollt mir das auch grad mal genauer anschauen :) ...feedback kommt...

ghost commented 11 years ago

ok, habs schon gefunden. bei mir hieß das addon verzeichnis "redaxo_multiupload". es muss aber "rex_multiupload" heissen. dummerweise kam hier auch nicht die fehlermeldung, die eigentlich kommen sollte :/ @gharlan ???

@Hirbod sehr nice das ganze. insbesondere die neue blaue dropzone mit den abgerundeten ecken macht echt was her :) nur der grüne hover effekt passt irgendwie nicht finde ich, da grün eigentlich gar nicht aftaucht im agk_skin (ausser bei rex_info's) --> ein hell blau und ein dunkel blau aus dem agk_skin nehmen? ...oder halt so belassen :). aber sonst super geil!!! thx!

vielleicht ist damit auch ralfs prob ausm forum beseitigt?

gharlan commented 11 years ago

@Hirbod Bitte hier den Addonnamen fest setzen, und ihn nicht aus den Parametern holen. Denn dann liefert Redaxo eine Fehlermeldung, wenn der Addonordner falsch benannt ist, aber so kann Redaxo das nicht erkennen.

hirbod commented 11 years ago

Hab in der README extra betont, dass das Verzeichnis umbenannt werden muss. Mich nervt das mit den Ordnernamen. Ich werde das in Zukunft wohl so machen, das ich hier in GIthub die Hierarchie verändere und "redaxo_multiupload -> rex_multiupload" ablege. Dann sollte hoffentlich allen klar sein, das sie nur das innere Verzeichnis hochladen dürfen.

@gharlan Wollte bewusst darauf verzichten den String fest anzugeben. Kann ich aber ändern, wenn das hilft.

Oder ich werde schauen, ob ich irgendwo fest "rex_multiupload" eingebaut habe und ebenfalls durch den $mypage String ersetzen, dann sollte das AddOn funktionieren, egal wie das Verzeichnis lautet. Entspricht aber nicht der Update-AddOnKey norm... alles nervig! :D

gharlan commented 11 years ago

Ich würde mich schon auf einen Key festlegen, und den dann fest verwenden und Redaxo die Fehlermeldung ausgeben lassen, wenn der Ordner falsch heißt. So ist es jedenfalls gedacht. redaxo/redaxo4#78

ghost commented 11 years ago

@Hirbod auf jedenfall den key verwenden, so wie das andere addons auch machen. du wirst dann zumindest mit der R4.5 kein geschiess mehr haben was die addon verzeichnisnamen angeht :)

ghost commented 11 years ago

Ich werde das in Zukunft wohl so machen, das ich hier in GIthub die Hierarchie verändere und "redaxo_multiupload -> rex_multiupload" ablege.

momentan ist es halt ein mischmasch. entweder sollte die repo "rex_multiupload" lauten oder halt "redaxo_rex_multiupload".

übrigens habe ich mich gerfagt ob diese zeile hier nicht noch probleme machen wird: https://github.com/nightstomp/redaxo_multiupload/blob/master/config.inc.php#L32

(ich habe bei mir in der master.inc.php diese beta geschichten deshalb auch rausgenommen, bei mir steht nur 4.5.0)

gharlan commented 11 years ago

Da er mit version_compare() arbeitet, sollte das kein Problem sein. Die Funktion kommt auch mit Beta-Versionen etc. zurecht.

ghost commented 11 years ago

ok, wusst ich net...dachte ein nicht numerisches zeichen könnte da probleme machen.

hirbod commented 11 years ago

Added some Magic: https://github.com/nightstomp/redaxo_multiupload/blob/master/install.inc.php

Ab sofort ist es egal, wie der Ordner heißt. Die Install-Routine kümmert sich darum :)