From 12b4324d3a9b9c75aa028ef6cb8377e9c8ae8ef6 Mon Sep 17 00:00:00 2001
From: Laurent Destailleur <eldy@users.sourceforge.net>
Date: Sat, 3 May 2008 23:05:40 +0000
Subject: [PATCH] Allow an external module to use the wrapper

---
 htdocs/viewimage.php | 55 +++++++++++++++++++++++++-------------------
 1 file changed, 31 insertions(+), 24 deletions(-)

diff --git a/htdocs/viewimage.php b/htdocs/viewimage.php
index 069bcfa0a37..0b730874e98 100644
--- a/htdocs/viewimage.php
+++ b/htdocs/viewimage.php
@@ -78,21 +78,21 @@ if ($modulepart)
     }
 
     // Wrapping pour les photos utilisateurs
-    if ($modulepart == 'userphoto')
+    elseif ($modulepart == 'userphoto')
     {
     	$accessallowed=1;
     	$original_file=$conf->users->dir_output.'/'.$original_file;
     }
 
     // Wrapping pour les photos adherents
-    if ($modulepart == 'memberphoto')
+    elseif ($modulepart == 'memberphoto')
     {
     	$accessallowed=1;
     	$original_file=$conf->adherent->dir_output.'/'.$original_file;
     }
     
     // Wrapping pour les apercu factures
-    if ($modulepart == 'apercufacture')
+    elseif ($modulepart == 'apercufacture')
     {
         $user->getrights('facture');
         if ($user->rights->facture->lire)
@@ -103,7 +103,7 @@ if ($modulepart)
     }
 
     // Wrapping pour les apercu propal
-    if ($modulepart == 'apercupropal')
+    elseif ($modulepart == 'apercupropal')
     {
         $user->getrights('propale');
         if ($user->rights->propale->lire)
@@ -114,7 +114,7 @@ if ($modulepart)
     }
     
     // Wrapping pour les apercu commande
-    if ($modulepart == 'apercucommande')
+    elseif ($modulepart == 'apercucommande')
     {
         $user->getrights('commande');
         if ($user->rights->commande->lire)
@@ -125,7 +125,7 @@ if ($modulepart)
     }
     
     // Wrapping pour les apercu intervention
-    if ($modulepart == 'apercufichinter')
+    elseif ($modulepart == 'apercufichinter')
     {
         $user->getrights('ficheinter');
         if ($user->rights->ficheinter->lire)
@@ -136,7 +136,7 @@ if ($modulepart)
     }
 
     // Wrapping pour les images des stats propales
-    if ($modulepart == 'propalstats')
+    elseif ($modulepart == 'propalstats')
     {
         $user->getrights('propale');
         if ($user->rights->propale->lire)
@@ -147,7 +147,7 @@ if ($modulepart)
     }
 
     // Wrapping pour les images des stats commandes
-    if ($modulepart == 'orderstats')
+    elseif ($modulepart == 'orderstats')
     {
         $user->getrights('commande');
         if ($user->rights->commande->lire)
@@ -158,7 +158,7 @@ if ($modulepart)
     }
 
     // Wrapping pour les images des stats factures
-    if ($modulepart == 'billstats')
+    elseif ($modulepart == 'billstats')
     {
         $user->getrights('facture');
         if ($user->rights->facture->lire)
@@ -169,7 +169,7 @@ if ($modulepart)
     }
 
     // Wrapping pour les images des stats expeditions
-    if ($modulepart == 'expeditionstats')
+    elseif ($modulepart == 'expeditionstats')
     {
         $user->getrights('expedition');
         if ($user->rights->expedition->lire)
@@ -180,7 +180,7 @@ if ($modulepart)
     }
 
     // Wrapping pour les images des stats produits
-    if (eregi('^productstats_',$modulepart))
+    elseif (eregi('^productstats_',$modulepart))
     {
         $user->getrights('produit');
         if ($user->rights->produit->lire)
@@ -191,7 +191,7 @@ if ($modulepart)
     }
 
     // Wrapping pour les produits
-    if ($modulepart == 'product')
+    elseif ($modulepart == 'product')
     {
         $user->getrights('produit');
         if ($user->rights->produit->lire)
@@ -202,7 +202,7 @@ if ($modulepart)
     }
 
     // Wrapping pour les prelevements
-    if ($modulepart == 'prelevement')
+    elseif ($modulepart == 'prelevement')
     {
         $user->getrights('prelevement');
         if ($user->rights->prelevement->bons->lire) $accessallowed=1;
@@ -211,7 +211,7 @@ if ($modulepart)
     }
 
     // Wrapping pour les graph telephonie
-    if ($modulepart == 'telephoniegraph')
+    elseif ($modulepart == 'telephoniegraph')
     {
         $user->getrights('telephonie');
         if ($user->rights->telephonie->lire)
@@ -222,61 +222,68 @@ if ($modulepart)
     }
 
     // Wrapping pour les graph energie
-    if ($modulepart == 'energie')
+    elseif ($modulepart == 'energie')
     {
       $accessallowed=1;
       $original_file=DOL_DATA_ROOT.'/energie/graph/'.$original_file;
     }
 
     // Wrapping pour les graph bank
-    if ($modulepart == 'bank')
+    elseif ($modulepart == 'bank')
     {
       $accessallowed=1;
       $original_file=$conf->banque->dir_temp.'/'.$original_file;
     }
 
     // Wrapping pour les images wysiwyg
-    if ($modulepart == 'fckeditor')
+    elseif ($modulepart == 'fckeditor')
     {
       $accessallowed=1;
       $original_file=$conf->fckeditor->dir_output.'/'.$original_file;
     }
 
     // Wrapping pour les images wysiwyg mailing
-    if ($modulepart == 'mailing')
+    elseif ($modulepart == 'mailing')
     {
       $accessallowed=1;
       $original_file=$conf->mailing->dir_output.'/'.$original_file;
     }
 
     // Wrapping pour les graph energie
-    if ($modulepart == 'graph_stock')
+    elseif ($modulepart == 'graph_stock')
     {
       $accessallowed=1;
       $original_file=DOL_DATA_ROOT.'/graph/entrepot/'.$original_file;
     }
 
     // Wrapping pour les graph fournisseurs
-    if ($modulepart == 'graph_fourn')
+    elseif ($modulepart == 'graph_fourn')
     {
       $accessallowed=1;
       $original_file=DOL_DATA_ROOT.'/graph/fournisseur/'.$original_file;
     }
 
     // Wrapping pour les graph des produits
-    if ($modulepart == 'graph_product')
+    elseif ($modulepart == 'graph_product')
     {
       $accessallowed=1;
       $original_file=DOL_DATA_ROOT.'/graph/product/'.$original_file;
     }
 
     // Wrapping pour les code barre
-    if ($modulepart == 'barcode')
+    elseif ($modulepart == 'barcode')
     {
-		$accessallowed=1;
-		$original_file='';
+    	$accessallowed=1;
+		$original_file='';	// No files are built on disk
     }
 
+    // Wrapping generique (allows any module to open a file if file is in directory
+    // called DOL_DATA_ROOT/modulepart.
+    else
+    {
+    	$accessallowed=1;
+      	$original_file=DOL_DATA_ROOT.'/'.$modulepart.'/'.$original_file;
+    }
 }
 
 // Security: