Discussion :

[Tonio_17000]

Bonjour,

voila j'ai le message d'erreur suivant:


Parse error: syntax error, unexpected '{' in /mnt/114/sdc/b/3/lopezcommerce/admin/newsdesk.php on line 16

et voici le code correspondant:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
 // delete file (if exists)
    $filename = tep_get_local_path(DIR_FS_CATALOG_IMAGES . $HTTP_GET_VARS['newsdesk_image']);
 
    try {
        if (file_exists($filename)) unlink($filename);
    }
    catch(Exception $e)
    {
        $filename='';
    }
 
 
    // update the table

Je ne trouve pas l'erreur. Merci d'avance a celui qui peut m'aider...

[Celira]

Bonjour
Vu le message d'erreur, tu as une { en trop ou une } manquante. Donc il te reste plus qu'à vérifier tes enchainements... Bon courage

[Tonio_17000]

Merci Celira,

je ne vois tjrs pas bien ou il manque un { ou } peut etre peux tu regarderr de plus pres si je te donne le code depuis le debut...?

Voici le code:
la ligne 16 etant

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

try {

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
<?php
 
require('includes/application_top.php');
require('includes/functions/newsdesk_general.php');
 
if ($HTTP_GET_VARS['action']) {
	switch ($HTTP_GET_VARS['action']) {
 
// -------------------------------------------------------------------------------------------------------------------------------------------------------------
case 'unlink_image':
// -------------------------------------------------------------------------------------------------------------------------------------------------------------
 
    // delete file (if exists)
    $filename = tep_get_local_path(DIR_FS_CATALOG_IMAGES . $HTTP_GET_VARS['newsdesk_image']);
 
    try {
        if (file_exists($filename)) unlink($filename);
    }
    catch(Exception $e)
    {
        $filename='';
    }
 
 
    // update the table
    $image = $HTTP_GET_VARS['newsdesk_image_number'];
    switch ($image){
        case '1':
        $image_to_delete="newsdesk_image";
        break;
        case '2':
        $image_to_delete="newsdesk_image_two";
        break;
        case '3':
        $image_to_delete="newsdesk_image_three";
        break;
    }
 
    $id_to_update = $HTTP_GET_VARS['newsdesk_update_id'];
 
    tep_db_query("update " . TABLE_NEWSDESK . " set " . $image_to_delete . "=NULL where newsdesk_id = '" . tep_db_input($id_to_update) . "'");
 
    switch ($image){
        case '1':
        $image_to_delete="newsdesk_image_text";
        break;
        case '2':
        $image_to_delete="newsdesk_image_text_two";
        break;
        case '3':
        $image_to_delete="newsdesk_image_text_three";
        break;
    }

Merci d'avance

[Celira]

Je suppose que tu fermes les { ouvrantes quelque part :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
if ($HTTP_GET_VARS['action']) {
	switch ($HTTP_GET_VARS['action']) {

En dehors de ça, je crois que le case fonctionne comme le if : si tu veux y mettre plus d'une instruction, il faut encadrer le bloc pas des {}. Essaye donc d'ajouter celles-là

[raoulchatigre]

En dehors de ça, je crois que le case fonctionne comme le if : si tu veux y mettre plus d'une instruction, il faut encadrer le bloc pas des {}. Essaye donc d'ajouter celles-là

FOUTAISES !!!
N'écoutes pas cette sorcière, elle veut détourner ton esprit de la lumière salvatrice que je détiens !

Il n'est nullement indispensable d'encadrer tes instructions 'case' par des accolades. Certes tu peux toujours quand même le faire afin de

• réduire la portée des variables déclarées dans un switch
• enjoliver ton code pour y voir plus clair

mais ce n'est en aucun cas une obligation.

Chaque case fonctionne un peu comme un GOTO, c'est pourqui tu dois mettre une instruction 'break' avant la fin sous peine d'exécuter le code du case suivant.


Sinon pour tes histoires d'accolades non refermées, cela semble en effet être la cause des instructions

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
if ($HTTP_GET_VARS['action']) {
	switch ($HTTP_GET_VARS['action']) {

qui ne sont pas refermées.

Celira avait raison sur ce point et c'est pourquoi je ne le conduirait pas au bûcher.


... zut

[Celira]

FOUTAISES !!!
N'écoutes pas cette sorcière, elle veut détourner ton esprit de la lumière salvatrice que je détiens !

Il n'est nullement indispensable d'encadrer tes instructions 'case' par des accolades.

Force m'est d'admettre, après test, que c'est exact. De là à me traiter de sorcière... J'en connais un qui va finir coassant dans une mare s'il ne fait pas attention

[raoulchatigre]

Tu réchappes à peine d'une tentative de combustion spontanée et tu veux t'en prendre à celui qui te sauves d'une mort certaine ? Y'a plus d'respect ma parole