Abfrage der größe in Aktionen

[is-werbung]

Hey leute,

hab soeben ein Modul programmiert indem man via Redaxo-Modul Bilder aus einen Slider ändern kann (Slider hat eine größe von 1024x300 Pixel)

Soweit läuft alles bestens,

jedoch will ich jetzt mit einer Aktion für dieses Modul noch das Absenden verhindern sollte das Bild kleiner sein als 1024x300.

Erst dachte ich kein problem - aber irgndwie meldet er überhaupt nichts - wisst ihr vielleicht was mein fehler ist?

Aktion:

Code: Alles auswählen

<?php
$img = $REX[HTDOCS_PATH].'files/'.$REX_ACTION['MEDIA']['1'].'';
$size = getimagesize($img);
$w = $size[0];
$h = $size[1];
if($w < '1024' OR $h < '300')
{
$REX_ACTION['SAVE'] = false;
$REX_ACTION['MSG'] = 'Bitte wählen Sie ein Bild mit min. einer Breite von 1024 und einer Höhe von 300!';
}
?>

Wenn ich den Code vom Modul noch Online stellen soll einfach posten.


Danke

[Koala]

Hast du mal die Inhalte der Variablen ausgeben lassen?
Also z.B: var_dump($img);

[is-werbung]

Ja, habs jetzt mal gemacht, der code oben ist so gesehen schon sauber, nur sollte ich beim nächsten mal den richtigen pfad zum bild angeben

trotzdem danke.

hier nochmal der überarbeitete code:

Code: Alles auswählen

<?php
$img = $REX[HTDOCS_PATH].'files/'.$REX_ACTION['MEDIA']['1'].'';
$size = @getimagesize($REX[HTDOCS_PATH].'files/'.$img);
$w = $size[0];
$h = $size[1];
if($w < '1024' OR $h < '300')
{
$REX_ACTION['SAVE'] = false;
$REX_ACTION['MSG'] = 'Bitte wählen Sie ein Bild mit min. einer Breite von 1024 und einer Höhe von 300!';
}
?>

Dies ist der Teil der vorhin falsch war:

Code: Alles auswählen

$size = @getimagesize($REX[HTDOCS_PATH].'files/'.$img);

wie man sieht hab ich nun davor mit einer Redaxo-Variable den richtigen Pfad angegeben.

ganz wichtig noch, das "@" symbol bei getimagesize sollte man keinesfalls auf "öffntlichen" seiten benutzen, es unterdrückt die fehlermeldung seitens php.

in meinen fall ist das vertretbar da es ja im backend des cms'es stattfindet und somit keine potentielle sicherheitslücke ist.

(ohne @-symbol gibt es eine php-fehlermeldung sollte generell kein bild gewählt sein, die redaxo-fehlermeldung kommt jedoch auch mit @-symbol)

[Koala]

hier nochmal der überarbeitete code:

Code: Alles auswählen

<?php
$img = $REX[HTDOCS_PATH].'files/'.$REX_ACTION['MEDIA']['1'].'';
$size = @getimagesize($REX[HTDOCS_PATH].'files/'.$img);
...

Die Pfadangabe HTDOCS_PATH doppelt drin zu haben, macht aber nicht wirklich sinn. Was steht denn da, wenn du nach der ersten Zeile ein var_dump($img); reinsetzt?
Da sollte dann schon der richtige Pfad im $img erscheinen.
BTW: Setze mal die Hochkommas in $REX: $REX['HTDOCS_PATH']

ganz wichtig noch, das "@" symbol bei getimagesize sollte man keinesfalls auf "öffntlichen" seiten benutzen, es unterdrückt die fehlermeldung seitens php.

Was du damit abfangen möchtest, kannst du auch für den Benutzer etwas informativer gestallten (ungetestet!):

Code: Alles auswählen

<?php
if (isset ($REX_ACTION['MEDIA']['1']) and $REX_ACTION['MEDIA']['1'] != '') {
  $img = $REX['HTDOCS_PATH'].'files/'.$REX_ACTION['MEDIA']['1'].'';
  $size = @getimagesize($REX['HTDOCS_PATH'].'files/'.$img);
  if ($size !== FALSE) {
    $w = $size[0];
    $h = $size[1];
    if ($w < '1024' OR $h < '300')
    {
      $REX_ACTION['SAVE'] = false;
      $REX_ACTION['MSG'] = 'Bitte wählen Sie ein Bild mit min. einer Breite von 1024 und einer Höhe von 300!';
    }
  } else {
    $REX_ACTION['SAVE'] = false;
    $REX_ACTION['MSG'] = '"'.$REX_ACTION['MEDIA']['1'].'" ist kein Bild.';
  }
} else {
  $REX_ACTION['SAVE'] = false;
  $REX_ACTION['MSG'] = 'Kein Bild gefunden.';
}?>

Geht aber bestimmt noch eleganter
Eine umfangreichere Funktion zu dem Thema findet sich auf php.net

[is-werbung]

stimmt danke, hab das mit den hochkommas und dem doppelten Pfad noch ausgebessert.

Ja, benutzerfreundlich oder nicht ist ja in dem sinne eher weniger intressant, ist ja schließlich nur 1x Person die an der seite arbeitet.

und zeit ist geld

danke trotzdem, echt toller support hier könnte sich manch ein anderes cms ne scheibe abschneiden