Parse error ? GGr

Version imprimable

Parse error ? GGr

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:

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...

10/03/2008, 15h32
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 ;)

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:

try {

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
<?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 ;)

10/03/2008, 16h07
Celira
Je suppose que tu fermes les { ouvrantes quelque part :
Code:

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à ;)
10/03/2008, 16h54
raoulchatigre
Citation:

Envoyé par Celira

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:

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
10/03/2008, 17h08
Celira

Citation:

Envoyé par raoulchatigre

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 :twisted:
10/03/2008, 17h40
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 :lol: