JMCrop - Image Crop-Addon

[iceman-fx]

Und ich habe leider noch einen Fehler entdeckt.

Wenn ich ein Preset nutze, um ein Bild zuzuschneiden, dann erhalten ich immer diese Fehlermeldung:

Code: Alles auswählen

Warning: imagecreatetruecolor() [function.imagecreatetruecolor]: Invalid image dimensions in /home/www/web69/html/redaxo/include/addons/imagecrop/classes/croppie.class.php on line 191

Warning: imagejpeg(): supplied argument is not a valid Image resource in /home/www/web69/html/redaxo/include/addons/imagecrop/classes/croppie.class.php on line 307

Wenn ich den Ausschnitt aber von Hand definiere klappt alles einwandfrei.

Es wird bei beiden Varianten der DB-Eintrag im Medienpool angelegt, aber im Presetmodus fehlt halt die physische Datei.

EDIT1: Mir ist auch aufgefallen, dass die Konfiguration nicht unter Rex 4.1 verfügbar ist.
EDIT2: Der obige Fehler betrifft alle Presetbuttons. Die Vorgaben werden nicht an die Hiddenfelder übergeben. Damit sind diese immer leer und es gibt keine Abfrage die das prüft.

[iceman-fx]

So Leute,

auch den Fehler mit den Presets konnte ich nun auch noch beheben.
Es fehlt(e) ein Eventhandler.

Um dies zu beheben erweitert den Code wie folgt in der Datei "config.inc.php:

Code: Alles auswählen

            jQuery(document).ready(function(){
                var api = jQuery.Jcrop(\'#testImage\',{
                    onSelect: showCoords,
                    aspectRatio: ' . (int)$REX['ADDON']['imagecrop']['proportionen'] . ',
                    boxWidth: 670, boxHeight: 600

ersetzen mit

Code: Alles auswählen

            jQuery(document).ready(function(){
                var api = jQuery.Jcrop(\'#testImage\',{
                    onSelect: showCoords,
					onChange: showCoords,
                    aspectRatio: ' . (int)$REX['ADDON']['imagecrop']['proportionen'] . ',
                    boxWidth: 670, boxHeight: 600

[iceman-fx]

Mein vorerst letzter Beitrag hierzu: auch das fehlen der Konfig habe ich gefunden und beseitigt.

Dazu muss in der Datei "index.inc.php" im Ordner pages in Zeile 20 der folgende Code angepasst werden:

Code: Alles auswählen

if ((!isset($_GET['bild'])) && (!isset($_POST['speichern']))){ 
    // kein Bild ausgewählt und auch keines gespeichert...
    $REX["PAGE_NO_NAVI"] = 0;
    require $REX['INCLUDE_PATH'].'/layout/top.php';
    rex_title('ImageCrop');    

ersetzen mit

Code: Alles auswählen

if ((!isset($_GET['bild'])) && (!isset($_POST['speichern']))){ 
    // kein Bild ausgewählt und auch keines gespeichert...
    $REX["PAGE_NO_NAVI"] = 0;
    require $REX['INCLUDE_PATH'].'/layout/top.php';
    rex_title('ImageCrop', $REX['ADDON']["imagecrop"]['SUBPAGES']);    

Jetzt funktioniert es bei mir unter Rex 4.1 est einmal richtig. Lediglich an der benutzeroberfläche würde ich noch etwas feilen und auch die Bestätigungsmeldungen anders präsentieren.

VG
iceman


PS: Kann eigentlich jemand unter Rex 4.1 meine gefunden Fehler nachvollziehen?

[alex2911]

oha - hier ist ja einiges passiert....
ich glaube, ich hatte sowas wie if $_post[bla] != "" get_bla = post_bla drinn.

naja, zuerst muss ich mich da einlesen

danke für die meldungen und die fixes !


//EDIT 1
onChange: showCoords flog absichtlich raus.

[iceman-fx]

//EDIT 1
onChange: showCoords flog absichtlich raus.

Hi Alex,

ohne das funzt es bei mir aber sonst nicht.
Dies habe ich auf der Herstellerseite nachgelesen und entsprechend eingefügt.

[alex2911]

onChange wurde nutzlos da die aktuelle markierungsdarstellung auf wunsch rausflog.

aber ich schau mir alles nochmal an dieses wochenende.

[alex2911]

Muss der User wirklich die x,y,x2,y2 Koordinaten sehen, das werde ich auf jedenfall rausnehmen.

Hier die Details warum onchange rausflog

[iceman-fx]

Wie schon geschrieben, ich hatte das im JS gegengeprüft und da waren die Hiddenfelder mit den Coords immer leer.

Aber kannst ja nochmal nachschauen.

[iceman-fx]

Hi Alex,

konntest Du schon die Fehler nachvollziehen?
Andere Frage noch: ist eigentlich eine Anpassung an PHP4 angedacht?

[alex2911]

Servus

Nein, ich hatte leider noch keine Zeit

PHP4-kompatibel wird es wohl nicht mehr.
Dazu fehlt einfach die Zeit. Aber um die von Dir angesprochenen Bugs kümmere ich mich noch sobald ich irgendwie Luft habe.

Uns hat das Hochwasser leider wieder erwischt diese Woche ....

[alex2911]

aloha

schnelles update da ich es gerade für etwas gebraucht hab...
JMCROP auf eine neue version gebracht.
wurde nur die JS-datei getauscht auf die 0.9.8.

have fun

[iceman-fx]

Hi alex,

behebt das gleich meine gefundenen Bugs mit?

Gruß
iceman

[alex2911]

Man kann natürlich hierbei auch einen anderen Zusatz vorsehen, aber ich weiss jetzt nicht, ob bei der Bildspeicherung auch die Überschreibroutine von Rex genutzt wird (vorhandene Dateinamen um "_1", "_2", etc. erweitern).

wird genutzt.

auch den Fehler mit den Presets konnte ich nun auch noch beheben

ja naja - das passiert wenn du ein preset aufrufst und das bild gleich so speicherst.
wenn du aber nach dem aufruf des preset das kästchen etwas bewegst, funktioniert es wieder.

das problem bei onChange ist dass immer ein event abgefeuert wird sobald etwas passiert (also auch wenn du gerade einen ausschnitt erstellst).
Und dies kostet Ressourcen beim Client weshalb das rausflog.

[alex2911]

änderungen hochgeladen. hoffe, ich habe nichts vergessen

danke für dein feedback iceman

[alex2911]

mal eine rückfrage nach 10 Tagen ...
gibt es feedback ?

[Pio]

Hi alex,

die 10 Tage sind zwar schon etwas her , aber ich habe unter 4.2.1 zwei Bugs gefunden.

1. ich gebe in den Metadaten (Metafile/Bild) eines Artikels ein Bild an um eine Navigation mit Imagelinks zu erstellen. Wenn ich das Bild ändern will kommt folgende Meldung:
   Fatal error: Call to a member function getFileName() on a non-object in /redaxo/include/addons/imagecrop/config.inc.php on line 77
   Deaktiviere ich das Imagecrop Addon, ist wieder alles OK.
2. ich habe Dateien in denen ein Pluszeichen vorkommt (z.B. kind+kegel.jpg), klicke ich nun im Medienpool auf "Bild bearbeiten" bekomme ich kein Bild angezeigt, weil Imagecrop das Pluszeichen entfernt (?), denke ich.

Gruß Pio

[simonsinus]

Hallo

Ich habe gestern das ImageCrop
Addon auf eine Redaxo 4.2.1 Installation installiert.

Gestern liefs perfekt. Heute gibt er eine Fehlermeldung aus:

Fatal error: Call to a member function getFileName() on a non-object in .../redaxo/include/addons/imagecrop/config.inc.php on line 77

Könnte das mit dem kurz darauf installierten multifile uploader zu tun haben?

Grüße Simon

[maximaexchen]

Hallo in die Runde.

Ich krame mal den alten Thread wieder aus...
das Addon ist ja echt ziemlich klasse.
Vor allem die Zusatzmöglichkeit mit dem Skalieren.

Habe mal versucht das in 4.3 zum Laufen zu bekommen... doch leider ohne Erfolg.

Ich weiß leider auch nicht, wo ich da genau ansetzetn soll...

Hat sich daran schon mal jemand versucht?

Vielen Dank im Voraus und
Grüße Marcus

[simonsinus]

Hallo Maxi

Ich glaube die Zeile ist schuld

Code: Alles auswählen

if ($REX['VERSION'] == 4 && $REX['SUBVERSION'] == 2){
            // Version 4.2
            require dirname( __FILE__) . '/top.php';
            $_REQUEST['page'] = 'mediapool';

Da muss man die Abfrage der Subversion natürlich auf 3 setzen.
Dann funktioniert Image Crop zwar
aber wenn man das Bild speichern möchte geht Redaxo auf die falsche Zielseite und speichert das Bild nicht.

Hat jemand schon ne idee?

Hallo in die Runde.

Ich krame mal den alten Thread wieder aus...
das Addon ist ja echt ziemlich klasse.
Vor allem die Zusatzmöglichkeit mit dem Skalieren.

Habe mal versucht das in 4.3 zum Laufen zu bekommen... doch leider ohne Erfolg.

Ich weiß leider auch nicht, wo ich da genau ansetzetn soll...

Hat sich daran schon mal jemand versucht?

Vielen Dank im Voraus und
Grüße Marcus

[iceman-fx]

Für alle die nach einer Funktion für Rex 4.3 suchen...

Ich habe auf Basis des bestehenden jCrop ein neues geschrieben, welches mit Rex 4.2 - 4.31 getestet wurde.

Den ausführlichen Beitrag dazu findet Ihr hier: http://forum.redaxo.de/ftopic15288.html.

Eurer iceman