View Issue Details

IDProjectCategoryView StatusLast Update
0003749OXID eShop (all versions)4.11. Image handlingpublic2012-12-12 16:29
Reporterfrank Assigned To 
PriorityhighSeveritycrashReproducibilityalways
Status resolvedResolutionunable to reproduce 
Product Version4.5.8 revision 42471 
Summary0003749: oxpicturehandler::getPicUrl incorrect
DescriptionHaving more than one sub-shop it is important not to save all sub-shop pictures in the same picture directory out/pictures, but in picture directories under the custom theme directories, e.g. out/emazure/1/pictures, out/emazure/2/pictures, etc.

This did work until shop version 4.4.x, and also in version 4.5 several source is supporting this setup, however unfortunately not all !


The picture preview links in the shop admin (articles, tab pictures) do not work.


Reason is the function oxpicturehandler::getPicUrl. This function contains following call:

$aPicInfo = $this->_getPictureInfo( "master/" . ( $sAltPath ? $sAltPath : $sPath ), $sFile, $this->isAdmin() );


The parameter $this->isAdmin() is messing up the correct path selection. Things work fine, if this line is changed to:

$aPicInfo = $this->_getPictureInfo( "master/" . ( $sAltPath ? $sAltPath : $sPath ), $sFile, false );



oxpicturehandler::getPicUrl is always supposed to deal with frontend pictures. So I don't see any possible side-effects of this correction.

TagsImage Conversion, Subshops
ThemeBoth
BrowserAll
PHP Versionany
Database Versionany

Relationships

related to 0003751 resolvedLinas Kukulskis .htaccess improvement or documentation necessary for getimg.php 
related to 0003750 resolvedLinas Kukulskis dynimggenerator::_getImageUri incorrect 

Activities

There are no notes attached to this issue.