Hallo Babelfisch
Bin heute doch noch dazu gekommen deine Erweiterung etwas genauer an zu schauen. Das Problem mit der falschen Prüfung für den Modus 'box' und 'proportional' konnte ich mit einer erweiterten IF Abfrage lösen (nicht ausgiebig getestet):
PHP-Code:
if($mode == 'box' || $mode == 'proportional') {
if($arrSize[0] < $width*2 && $arrSize[1] < $height*2) {
return;
}
} elseif($arrSize[0] < $width*2 || $arrSize[1] < $height*2) {
return;
}
Gleichzeitig ist mir noch ein Bug beim zusammenstellen des <img> tags aufgefallen. Da wurde mir nämlich immer ein quadratischer Wert für Breite und Höhe ausgegeben, statt die effektiven Kantenwerte des neuen Bildes. Auf Zeile 159 im RetinaImage.php ist der falsche Array Eintrag gesetzt
PHP-Code:
$strTag = str_replace('<img', '<img width="'.$imageSize[1].'"', $strTag);
// should be
$strTag = str_replace('<img', '<img width="'.$imageSize[0].'"', $strTag);
Und zu guter Letzt ist mir beim Server Umzug noch aufgefallen, dass die die Entwicklungsanleitung die ich selber Verwendet habe, für die .htaccess Datei in den "assets" eine (inzwischen) falsche oder zumindest nicht unbedingt überall funktionstüchtige Vorlage angibt. Bei deinem Modul hatte ich auf dem neuen Server das gleiche Problem. Contao und andere Module verwenden folgenden .htaccess Inhalt für den Modul>assets Ordner:
HTML-Code:
<IfModule !mod_authz_core.c>
Order allow,deny
Allow from all
</IfModule>
<IfModule mod_authz_core.c>
Require all granted
</IfModule>
anstelle von
HTML-Code:
order deny,allow
allow from all
Hoffe das hilft und wäre cool, wenn du das mal noch einbauen könntest.
Herzlich
Patric
Zitieren
Zitat von Babelfisch
Lesezeichen