Ajout d'un second champ image sur les marques

Version imprimable

Bonjour,

Je suis sur Prestashop 1.6

J'ai tenté d'ajouter une seconde image dans la rubrique les marques

J'ai Overridé le controller AdminManufacturersController.php comme ceci, en appelant "drapeau" le nouveau champ :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
<?php
 
class AdminManufacturersControllerCore extends AdminController
{
    public $bootstrap = true ;
    /** @var array countries list */
    protected $countries_array = array();
 
    public function __construct()
    {
        $this->table = 'manufacturer';
        $this->className = 'Manufacturer';
        $this->lang = false;
        $this->deleted = false;
        $this->allow_export = true;
        $this->list_id = 'manufacturer';
        $this->identifier = 'id_manufacturer';
        $this->_defaultOrderBy = 'name';
        $this->_defaultOrderWay = 'ASC';
 
        $this->bulk_actions = array(
            'delete' => array(
                'text' => $this->l('Delete selected'),
                'icon' => 'icon-trash',
                'confirm' => $this->l('Delete selected items?')
            )
        );
 
        $this->context = Context::getContext();
 
        $this->fieldImageSettings = array(
            'name' => 'logo',
            'dir' => 'm'
        );
 
        $this->fieldImageSettings_v2 = array(
            'name' => 'drapeau',
            'dir' => 'd'
        );
 
        $this->fields_list = array(
            'id_manufacturer' => array(
                'title' => $this->l('ID'),
                'align' => 'center',
                'class' => 'fixed-width-xs'
            ),
            'logo' => array(
                'title' => $this->l('Logo'),
                'image' => 'm',
                'orderby' => false,
                'search' => false,
                'align' => 'center',
            ),
            'drapeau' => array(
                'title' => $this->l('drapeau'),
                'image' => 'd',
                'orderby' => false,
                'search' => false,
                'align' => 'center',
            ),
            'name' => array(
                'title' => $this->l('Name'),
                'width' => 'auto'
            ),
            'addresses' => array(
                'title' => $this->l('Addresses'),
                'search' => false,
                'align' => 'center'
            ),
            'products' => array(
                'title' => $this->l('Products'),
                'search' => false,
                'align' => 'center',
            ),
            'active' => array(
                'title' => $this->l('Enabled'),
                'active' => 'status',
                'type' => 'bool',
                'align' => 'center',
                'class' => 'fixed-width-xs',
                'orderby' => false
            )
        );
 
        parent::__construct();
    }
 
    public function renderForm()
    {
        if (!($manufacturer = $this->loadObject(true))) {
            return;
        }
 
        $image = _PS_MANU_IMG_DIR_.$manufacturer->id.'.jpg';
        $image_url = ImageManager::thumbnail($image, $this->table.'_'.(int)$manufacturer->id.'.'.$this->imageType, 350,
            $this->imageType, true, true);
        $image_size = file_exists($image) ? filesize($image) / 1000 : false;
 
        $image_v2 = _PS_MANU_IMG_DIR_.$manufacturer->id.'_v2.jpg';
        $image_url_v2 = ImageManager::thumbnail($image_v2, $this->table.'_'.(int)$manufacturer->id.'.'.$this->imageType, 350,
            $this->imageType, true, true);
        $image_size_v2 = file_exists($image_v2) ? filesize($image_v2) / 1000 : false;
 
        $this->fields_form = array(
            'tinymce' => true,
            'legend' => array(
                'title' => $this->l('Manufacturers'),
                'icon' => 'icon-certificate'
            ),
            'input' => array(
                array(
                    'type' => 'text',
                    'label' => $this->l('Name'),
                    'name' => 'name',
                    'col' => 4,
                    'required' => true,
                    'hint' => $this->l('Invalid characters:').' &lt;&gt;;=#{}'
                ),
                array(
                    'type' => 'textarea',
                    'label' => $this->l('Short description'),
                    'name' => 'short_description',
                    'lang' => true,
                    'cols' => 60,
                    'rows' => 10,
                    'autoload_rte' => 'rte', //Enable TinyMCE editor for short description
                    'col' => 6,
                    'hint' => $this->l('Invalid characters:').' &lt;&gt;;=#{}'
                ),
                array(
                    'type' => 'textarea',
                    'label' => $this->l('Description'),
                    'name' => 'description',
                    'lang' => true,
                    'cols' => 60,
                    'rows' => 10,
                    'col' => 6,
                    'autoload_rte' => 'rte', //Enable TinyMCE editor for description
                    'hint' => $this->l('Invalid characters:').' &lt;&gt;;=#{}'
                ),
                array(
                    'type' => 'file',
                    'label' => $this->l('Logo'),
                    'name' => 'logo',
                    'image' => $image_url ? $image_url : false,
                    'size' => $image_size,
                    'display_image' => true,
                    'col' => 6,
                    'hint' => $this->l('Upload a manufacturer logo from your computer.')
                ),
                array(
                    'type' => 'file',
                    'label' => $this->l('Drapeau'),
                    'name' => 'drapeau',
                    'image' => $image_url_v2 ? $image_url_v2 : false,
                    'size' => $image_size_v2,
                    'display_image' => true,
                    'col' => 6,
                    'hint' => $this->l('Upload a manufacturer drapeau from your computer.')
                ),
                array(
                    'type' => 'text',
                    'label' => $this->l('Meta title'),
                    'name' => 'meta_title',
                    'lang' => true,
                    'col' => 4,
                    'hint' => $this->l('Forbidden characters:').' &lt;&gt;;=#{}'
                ),
                array(
                    'type' => 'text',
                    'label' => $this->l('Meta description'),
                    'name' => 'meta_description',
                    'lang' => true,
                    'col' => 6,
                    'hint' => $this->l('Forbidden characters:').' &lt;&gt;;=#{}'
                ),
                array(
                    'type' => 'tags',
                    'label' => $this->l('Meta keywords'),
                    'name' => 'meta_keywords',
                    'lang' => true,
                    'col' => 6,
                    'hint' => array(
                        $this->l('Forbidden characters:').' &lt;&gt;;=#{}',
                        $this->l('To add "tags," click inside the field, write something, and then press "Enter."')
                    )
                ),
                array(
                    'type' => 'switch',
                    'label' => $this->l('Enable'),
                    'name' => 'active',
                    'required' => false,
                    'class' => 't',
                    'is_bool' => true,
                    'values' => array(
                        array(
                            'id' => 'active_on',
                            'value' => 1,
                            'label' => $this->l('Enabled')
                        ),
                        array(
                            'id' => 'active_off',
                            'value' => 0,
                            'label' => $this->l('Disabled')
                        )
                    )
                )
            )
        );
 
        if (!($manufacturer = $this->loadObject(true))) {
            return;
        }
 
        if (Shop::isFeatureActive()) {
            $this->fields_form['input'][] = array(
                'type' => 'shop',
                'label' => $this->l('Shop association'),
                'name' => 'checkBoxShopAsso',
            );
        }
 
        $this->fields_form['submit'] = array(
            'title' => $this->l('Save')
        );
 
        foreach ($this->_languages as $language) {
            $this->fields_value['short_description_'.$language['id_lang']] = htmlentities(stripslashes($this->getFieldValue(
                $manufacturer,
                'short_description',
                $language['id_lang']
            )), ENT_COMPAT, 'UTF-8');
 
            $this->fields_value['description_'.$language['id_lang']] = htmlentities(stripslashes($this->getFieldValue(
                $manufacturer,
                'description',
                $language['id_lang']
            )), ENT_COMPAT, 'UTF-8');
        }
 
        return parent::renderForm();
    }
 
 
    protected function afterImageUpload()
    {
        $res = true;
        $generate_hight_dpi_images = (bool)Configuration::get('PS_HIGHT_DPI');
 
        /* Generate image with differents size */
        if (($id_manufacturer = (int)Tools::getValue('id_manufacturer')) &&
            isset($_FILES) &&
            count($_FILES) &&
            file_exists(_PS_MANU_IMG_DIR_.$id_manufacturer.'.jpg')) {
            $images_types = ImageType::getImagesTypes('manufacturers');
            foreach ($images_types as $k => $image_type) {
                $res &= ImageManager::resize(
                    _PS_MANU_IMG_DIR_.$id_manufacturer.'.jpg',
                    _PS_MANU_IMG_DIR_.$id_manufacturer.'-'.stripslashes($image_type['name']).'.jpg',
                    (int)$image_type['width'],
                    (int)$image_type['height']
                );
 
                if ($generate_hight_dpi_images) {
                    $res &= ImageManager::resize(
                        _PS_MANU_IMG_DIR_.$id_manufacturer.'.jpg',
                        _PS_MANU_IMG_DIR_.$id_manufacturer.'-'.stripslashes($image_type['name']).'2x.jpg',
                        (int)$image_type['width']*2,
                        (int)$image_type['height']*2
                    );
                }
            }
 
            $current_logo_file = _PS_TMP_IMG_DIR_.'manufacturer_mini_'.$id_manufacturer.'_'.$this->context->shop->id.'.jpg';
 
            if ($res && file_exists($current_logo_file)) {
                unlink($current_logo_file);
            }
        }
 
        /* Generate image with differents size */
        if (($id_manufacturer = (int)Tools::getValue('id_manufacturer')) &&
            isset($_FILES) &&
            count($_FILES) &&
            file_exists(_PS_MANU_IMG_DIR_.$id_manufacturer.'_v2.jpg')) {
            $images_types = ImageType::getImagesTypes('manufacturers');
            foreach ($images_types as $k => $image_type) {
                $res &= ImageManager::resize(
                    _PS_MANU_IMG_DIR_.$id_manufacturer.'_v2.jpg',
                    _PS_MANU_IMG_DIR_.$id_manufacturer.'-'.stripslashes($image_type['name']).'_v2.jpg',
                    (int)$image_type['width'],
                    (int)$image_type['height']
                );
 
                if ($generate_hight_dpi_images) {
                    $res &= ImageManager::resize(
                        _PS_MANU_IMG_DIR_.$id_manufacturer.'_v2.jpg',
                        _PS_MANU_IMG_DIR_.$id_manufacturer.'-'.stripslashes($image_type['name']).'_v22x.jpg',
                        (int)$image_type['width']*2,
                        (int)$image_type['height']*2
                    );
                }
            }
 
            $current_logo_file_v2 = _PS_TMP_IMG_DIR_.'manufacturer_mini_'.$id_manufacturer.'_'.$this->context->shop->id.'_v2.jpg';
 
            if ($res && file_exists($current_logo_file_v2)) {
                unlink($current_logo_file_v2);
            }
        }
 
        if (!$res) {
            $this->errors[] = Tools::displayError('Unable to resize one or more of your pictures.');
        }
 
        return $res;
    }
}

Le champ apparait bien sur l'admin dans les marques mais lorsque je sélectionne mon image et que je clique sur "enregistrer", l'image n'est pas enregistrée.

Je ne vois pas où faire d'autres modifications, je n'ai pas l'impression que l'image doit être enregistrée en base de données.

Merci pour votre aide

06/10/2017, 16h10
v1cent
Alors deux choses :
- Il faudrait que tu crée une classe dans le dossier override au lieu d'éditer directement celle là. Tu appelle ta classe
  
  Code:
  
  class AdminManufacturersController extends AdminManufacturersControllerCore
  
  , Prestashop va utiliser automatiquement celle-là (tu peux facilement trouver la doc sur le sujet)
  De cette manière tu pourra faire des mises à jour sans que ton code soit impacté (tu ne met que les fonctions que tu modifie dans la classe fille)
  Ce n'est pas indispensable pour que ça fonctionne ceci dit.
- Il faut ajouter ton champ à la classe Manufacturer pour que l'enregistrement (pareil il faudrait idéalement le faire dans le dossier override). Il y a plusieurs modifications à apporter le plus simple est de te baser sur l'autre champ et de dupliquer son code dans la classe fille en changeant le nom. N'hésite pas si tu as un souci à l'implémentation !
06/10/2017, 16h21
taka10

1) Oui oui je sais pour l'override, c'est juste que lorsque je l'ai fait les modifications n'étaient pas prises en compte mais OK je modifierai pour cela.

2) Justement dans la classe Manufacturer, il n'y a pas de trace du champ de base "logo", je suis étonné
06/10/2017, 17h14
v1cent

Ha j'ai parlé trop vite pardon

Oui en effet les images pour cette fonctionnalité ne sont pas du tout stockées en base, elles sont simplement ajoutées dans un dossier avec l'id du fabricant. Si l'image qui porte le bon nom existe prestashop l'affiche, sinon non.

Du coup soit toi tu fait un système pour enregistrer les images en base, soit tu duplique le système (et au lieu d'appeler tes images 1, 2, 3 etc. tu les appelle par exemple drapeau1, drapeau2, etc)

Jette un oeil dans ManufacturerController (la variable s'appelle image).

Ca va être un peu chiant de gérer les différentes tailles de miniature je te préviens !
06/10/2017, 19h49
taka10

Dans mon premier post, j'explique justement que je modifie ManufacturerController et la variable ne s'appelle pas image mais logo où alors j'ai zappé un truc.

Je n'ai pas besoin des différentes tailles justement, c'est juste pour télécharger le drapeau de la marque.

EN fait, je ne trouve pas la fonction qui télécharge l'image "logo" sur le serveur
07/10/2017, 15h27
v1cent

oui ça s'appelle logo en backoffice mais image en front office j'ai l'impression (je me base uniquement sur le code, je n'ai utilisé cette fonctionnalité sur aucune boutique)

tu as le code dans la fonction protected function postImage($id) dans le fichier classes/controller/AdminController.php

du coup je pense que tu as déclaré comme il faut, tes images devraient être classées par id fabricant dans le dossier /img/d/ (c'est bien plus difficile vu le code de mettre un nom personnalisé dans /img/m/)

Il y a quoi dans le dossier img/d/ ? elles y sont ?
07/10/2017, 15h30
taka10

Non je n'ai aucun message d'erreur et mes images ne sont pas téléchargées dans /img/d/ . Je ne vois absolument pas quoi faire de plus

Justement j'ai overridé AdminController.php et j'ai rajouté :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
 
/** @var array Name and directory where class image are located */
    public $fieldImageSettings = array();
 
    /** @var array Name and directory where class image are located */
    public $fieldImageSettings_v2 = array();
 
protected function postImage($id)
    {
        if (isset($this->fieldImageSettings['name']) && isset($this->fieldImageSettings['dir'])) {
            return $this->uploadImage($id, $this->fieldImageSettings['name'], $this->fieldImageSettings['dir'].'/');
        } elseif (!empty($this->fieldImageSettings)) {
            foreach ($this->fieldImageSettings as $image) {
                if (isset($image['name']) && isset($image['dir'])) {
                    $this->uploadImage($id, $image['name'], $image['dir'].'/');
                }
            }
        }
        if (isset($this->fieldImageSettings_v2['name']) && isset($this->fieldImageSettings_v2['dir'])) {
            return $this->uploadImage($id, $this->fieldImageSettings_v2['name'], $this->fieldImageSettings_v2['dir'].'/');
        } elseif (!empty($this->fieldImageSettings_v2)) {
            foreach ($this->fieldImageSettings_v2 as $image) {
                if (isset($image['name']) && isset($image['dir'])) {
                    $this->uploadImage($id, $image['name'], $image['dir'].'/');
                }
            }
        }
        return !count($this->errors) ? true : false;
    }

Regarde dans AdminScencesControllers, il y a deux champs !

Le bon code serait donc

Code:

1
2
3
4
5
6
 
           $this->fieldImageSettings = 
                                array(
                                    array('name' => 'logo', 'dir' => 'm'), 
                                    array('name' => 'drapeau','dir' => 'd')
                                );

07/10/2017, 15h49
taka10

D'accord, je vais tester
07/10/2017, 15h53
taka10

Alléluia !!!

ça fonctionne!!! Plus d'une semaine que je suis dessus.

Un grand merci pour ton implication et tes conseils
07/10/2017, 15h57
v1cent

je t'en prie, j'ai aussi souvent perdu du temps sur les subtilités du code de Presta :D
07/10/2017, 16h10
taka10

Il reste un conflit entre les 2 images par contre car lorsque j'upload un drapeau (sans logo) , je drapeau est bien uploadé mais si un logo est déjà présent, il copie le logo à la place du drapeau et on n'a plus le logo
10/10/2017, 10h44
v1cent

Désolé je croyais le problème réglé je n'ai pas vérifié.

Je n'ai pas très bien compris il est sensé mettre les images du drapeau dans img/d/ et les images du logo dans img/m/, il fait quoi en vrai, il met les deux dans le même dossier ?
10/10/2017, 11h33
taka10

Pas de soucis, j'aurai du m'expliquer mieux que ça.

En fait, sur la liste des marques sur l'admin, le nouveau champ "drapeau" apparait bien mais c'est l'image qui remonte c'est l'image du logo et lorsque je vais voir sur modifier, l'image ne remonte pas non plus.

Par contre, l'image est bien téléchargé au bon endroit dans img/d/ et donc pas de soucis pour l'afficher en consultation