- Posts: 2696
- Thank you received: 74
addtocart_popup_image_size = 200
6 years 8 months ago - 6 years 8 months ago #314378
by erickb
addtocart_popup_image_size = 200 was created by erickb
Bonjour
j'ai cree une entree qui n'existait pas dans la config* : addtocart_popup_image_size = 200 pour éviter le $imageSize = (int)$config->get('addtocart_popup_image_size', 50); mais j'ai toujours des image générées automatiquement dans thumbnails/50x50 alors que nulle part je n'ai d'images en 50 ; j'ai aussi contrôlé dans media /emails
A quel autre endroit ces 50x50 peuvent etre appelés ?
Pourquoi a plusieurs endroits les $imageHelper->getThumbnail($file->file_path, array(100, 100) ... ont ils des valeurs en dur comme 50,50 ou 100,100 ? et non des valeurs lues dans la config
* INSERT INTO `#__hikashop_config` (`config_namekey`, `config_value`, `config_default`) VALUES ('addtocart_popup_image_size', '200', '200');
merci
j'ai cree une entree qui n'existait pas dans la config* : addtocart_popup_image_size = 200 pour éviter le $imageSize = (int)$config->get('addtocart_popup_image_size', 50); mais j'ai toujours des image générées automatiquement dans thumbnails/50x50 alors que nulle part je n'ai d'images en 50 ; j'ai aussi contrôlé dans media /emails
A quel autre endroit ces 50x50 peuvent etre appelés ?
Pourquoi a plusieurs endroits les $imageHelper->getThumbnail($file->file_path, array(100, 100) ... ont ils des valeurs en dur comme 50,50 ou 100,100 ? et non des valeurs lues dans la config
* INSERT INTO `#__hikashop_config` (`config_namekey`, `config_value`, `config_default`) VALUES ('addtocart_popup_image_size', '200', '200');
merci
Last edit: 6 years 8 months ago by erickb.
Please Log in or Create an account to join the conversation.
6 years 8 months ago - 6 years 8 months ago #314386
by erickb
Replied by erickb on topic addtocart_popup_image_size = 200
je viens de trouver que ces 50x50 correspondent a l'ajout a la liste des favoris alors que les icones sont en 200x200
Last edit: 6 years 8 months ago by erickb.
Please Log in or Create an account to join the conversation.
6 years 8 months ago #314395
by nicolas
Replied by nicolas on topic addtocart_popup_image_size = 200
Bonjour,
Pour le backend et les emails de notification, la taille des miniatures est forcée car elle est adaptée à la mise en page que nous avons à ces endroits.
Pour votre cas avec l'ajout à la liste des favoris, c'est le code:
dans la fonction getAjaxCartData du fichier components/controllers/product.php qui fait cela pour l'affichage de la boîte de notification d'ajout par le plugin HikaShop cart notification.
Pour le backend et les emails de notification, la taille des miniatures est forcée car elle est adaptée à la mise en page que nous avons à ces endroits.
Pour votre cas avec l'ajout à la liste des favoris, c'est le code:
Code:
$img = $imageHelper->getThumbnail($image_path, array(50,50), array('default' => true), true);
Please Log in or Create an account to join the conversation.
6 years 8 months ago - 6 years 8 months ago #314419
by erickb
Replied by erickb on topic addtocart_popup_image_size = 200
merci mais pourquoi 50 en dur alors qu'il devrait lire la valeur de addtocart_popup_image_size
si $imageSize > 0 pourquoi ne pas l'utiliser ?
Code:
$imageSize = (int)$config->get('addtocart_popup_image_size', 50);
.....
if($imageSize > 0) {
$img = $imageHelper->getThumbnail($image_path, array($imageSize,$imageSize), array('default' => true), true);
si $imageSize > 0 pourquoi ne pas l'utiliser ?
Last edit: 6 years 8 months ago by erickb.
Please Log in or Create an account to join the conversation.
6 years 8 months ago - 6 years 8 months ago #314423
by erickb
Replied by erickb on topic addtocart_popup_image_size = 200
j'ai change la function mais ce serait mieux en natif dans la 4.2.3
Code:
protected function getAjaxCartData($data, $cart, $status) {
$config = hikashop_config();
$imageSize = (int)$config->get('addtocart_popup_image_size', 50);
$ret = array(
'ret' => (int)$status
);
if(!empty($cart->messages))
$ret['messages'] = $cart->messages;
if(empty($cart->cart_products))
$ret['empty'] = true;
if(!empty($data['type']) && $data['type'] == 'product' && !empty($data['form']) && count($data['form'])) {
$added = array();
foreach($data['form'] as $prod => $qty) {
if($qty)
$added[] = array('product' => $prod, 'quantity' => $qty);
}
if(count($added) == 1) {
$data = array_merge($data,$added[0]);
}
}
if(!isset($data['product']))
return $ret;
$imageHelper = hikashop_get('helper.image');
if(!empty($cart->products)) {
foreach($cart->products as $product) {
if($product->product_id != $data['product'])
continue;
$ret['product_name'] = $product->product_name;
$ret['quantity'] = (int)$product->cart_product_quantity;
if($imageSize > 0) {
$image_path = (isset($product->images[0]->file_path) ? $product->images[0]->file_path : '');
$img = $imageHelper->getThumbnail($image_path, array($imageSize,$imageSize), array('default' => true), true);
if($img->success)
$ret['image'] = $img->url;
}
break;
}
}
if(!isset($ret['product_name'])) {
$productClass = hikashop_get('class.product');
$product = $productClass->getProduct($data['product']);
$ret['product_name'] = $product->product_name;
$ret['quantity'] = 0;
$image_path = ((isset($product->parent) && isset($product->parent->images[0]->file_path)) ? $product->parent->images[0]->file_path : '');
$image_path = (isset($product->images[0]->file_path) ? $product->images[0]->file_path : $image_path);
$img = $imageHelper->getThumbnail($image_path, array($imageSize,$imageSize), array('default' => true), true);
if($img->success)
$ret['image'] = $img->url;
}
if(!empty($cart->messages)) {
foreach($cart->messages as $msg) {
if(empty($msg['product_id']) || (int)$msg['product_id'] != $data['product'])
continue;
$ret['message'] = $msg['msg'];
break;
}
}
return $ret;
}
Last edit: 6 years 8 months ago by erickb.
Please Log in or Create an account to join the conversation.
6 years 8 months ago #314422
by nicolas
Replied by nicolas on topic addtocart_popup_image_size = 200
Bonjour,
C'est en effet une remarque pertinente. L'option cachée a été ajoutée à moitié sans remplacer les '50' dans les appels getThumbnail à cet endroit.
Nous allons changer cela pour la prochaine version. Merci d'avoir pointé cela, nous étions passé à coté.
C'est en effet une remarque pertinente. L'option cachée a été ajoutée à moitié sans remplacer les '50' dans les appels getThumbnail à cet endroit.
Nous allons changer cela pour la prochaine version. Merci d'avoir pointé cela, nous étions passé à coté.
The following user(s) said Thank You: erickb
Please Log in or Create an account to join the conversation.
6 years 8 months ago #314447
by erickb
Replied by erickb on topic addtocart_popup_image_size = 200
merci
Please Log in or Create an account to join the conversation.
Time to create page: 0.178 seconds