Bug tracking

Feed subscription: RSS or Atom.

 

#569 – Compatibilité avec pluggin modifiant url high
Date added: 2009-03-23 16:03:02 Type: Feature request
Status: Resolved Priority: Normal
Affected version: 0.9-beta4 Milestone: n/a
Reported by: Assigned to: flipflip
Bonjour,

Serait-il possible de faire en sorte que ce pluggin soit compatible avec le pluggin que j'utilise pour utiliser des fichiers haute définition avec un préfixe et des fois sous forme de .zip ?

ligne 663 de class.inc.php à modifier :

0 1 2 3 4 5 6
 
include_once(PHPWG_ROOT_PATH.'include/functions_picture.inc.php');
            $fichiers[$i]['nom'] = get_high_url($element_info);
            $fichiers[$i]['path'] = get_high_url($element_info);
            $fichiers[$i]['poids'] = $row['filesize'];
            $i++;
 

Attachements:
Re: Compatibilité avec pluggin modifiant url high - 2009-03-23 16:04:08
DidouDidou
Adeline LETESSIER--TRUCHON
De plus en relisant n'y a t'il pas une erreur puisque filesize correspond à la taille de l'image normal et non de l'image haute définition non ?
Re: Compatibilité avec pluggin modifiant url high - 2009-03-23 16:18:21
DidouDidou
Adeline LETESSIER--TRUCHON
J'ai validé trop vite...

Modification à faire :

1-Test sur has_high==FALSE ligne 544

0 1 2
 
if(get_high_location($row) == '')
 


2-Ajout has_high dans la requete ligne 608 :
0 1 2 3 4 5 6 7 8 9
 
$query = '
          SELECT '.$prefixeTable.'download_multi.id_image, '.$prefixeTable.'download_multi.filesize, '.$prefixeTable.'download_multi.type,
          '.IMAGES_TABLE.'.file, '.IMAGES_TABLE.'.path, '.IMAGES_TABLE.'.tn_ext, '.IMAGES_TABLE.'.has_high
          FROM '.$prefixeTable.'download_multi
          INNER JOIN '.IMAGES_TABLE.'
          ON '.IMAGES_TABLE.'.id = '.$prefixeTable.'download_multi.id_image
          WHERE id_user = '.$user['id'].'
      ;';
 


3-Modification du calcul des name et path ligne 663
0 1 2 3 4 5 6 7 8 9 10 11 12 13 14
 
if($row['type'] == 'h') {
                        $element_info=array(
                                'id'=>$row['id_image'],
                                'path'=>$row['path'],
                                'has_high'=>$row['has_high']
                        );
                        
                        include_once(PHPWG_ROOT_PATH.'include/functions_picture.inc.php');
            $fichiers[$i]['nom'] = get_high_url($element_info);
            $fichiers[$i]['path'] = get_high_url($element_info);
            $fichiers[$i]['poids'] = $row['filesize'];
            $i++;          
          } 
 


Cette fois j'espère que je n'ai rien oublié ;)

Adeline
flipflip edited this bug - 2009-03-24 14:22:47
  • Status changed to Working
  • Version changed to 0.9-beta4
  • Assign to flipflip
Re: Compatibilité avec pluggin modifiant url high - 2009-03-24 14:38:50
flipflipflipflip
Philippe MALADJIAN
Alors pour le filesize sur les pwg_high non c'est ok.

Pour les get_thumbnail_url et get_high_url c'est ok sauf que j'ai utilisé get_thumbnail_path et get_high_path et pour les versions normal je suis passé get_image_path($row);

Au contraire if(get_high_location($row) == '') ne sert à rien puisque il renvoie le chemin alors que if($row['has_high'] == false) contrôle si oui ou non il y a une pwg_high.
flipflip edited this bug - 2009-03-24 14:44:26
  • Status changed to Need tests
Re: Compatibilité avec pluggin modifiant url high - 2009-03-31 15:47:45
DidouDidou
Adeline LETESSIER--TRUCHON
Il manque juste l'id de l'image dans $query donc il faudrait rajouter IMAGES_TABLE.id soit renommer download_multi.id_image AS id...

flipflip edited this bug - 2009-04-02 11:43:54
  • Status changed to Resolved
This bug has been resolved.
Post to this thread

This bug report has been closed by a developer.
Being resolved or rejected, this report has been closed by a developer, thus you may not need to add an answer (even if you can do it anyway).

Title:


Message:

Authorized (X)HTML tags: a, img, abbr, blockquote, code, ul, ol, li, h3, h4, p, br, hr, strong, em, pre, sub, sup, table, caption, thead, tr, th, tfoot, tbody, td.

Captcha security:
In order to check you are not a robot, please answer to that simple question:
Please write the 3rd character of Quincy.        

Nickname: