Add control of parameters errors

2 files changed, 80 insertions, 5 deletions

diff --git a/server/Test/imageTests.php b/server/Test/imageTests.php
index 39e83e1..47a69e2 100644
--- a/server/Test/imageTests.php
+++ b/server/Test/imageTests.php
@@ -49,7 +49,7 @@ else{
 // Détails Image
 //$details = $image->imageDetails($id_image);
 
-//$image->deleteImage('123456');
+$image->deleteImage('123456');
 
 //$image->desactivateImage($id_image);
 //$image->reactivateImage($id_image);
diff --git a/server/core/Image.php b/server/core/Image.php
index 637966c..15facc9 100644
--- a/server/core/Image.php
+++ b/server/core/Image.php
@@ -40,6 +40,9 @@ class image {
 	* @return Image
 	*/
 	public function __construct($app){
+		if(!isset($app)){
+			// Renvoyer erreur 
+		}
 		$this->app = $app;
 		$this->libClass = $app->getLibClass("Image");
 	}
@@ -54,7 +57,10 @@ class image {
 	*	options for the image creation
 	*
 	**/
-	public function createImage(array $opt){	
+	public function createImage(array $opt){
+		if(!isset($opt)){
+			// Renvoyer erreur 
+		}	
 		// VOIR SI MAUVAIS TYPE
 		$options = Array();
 		if(isset($opt['name'])){ // string, rendre le nom obligatoire, vérifier nom pas déjà pris
@@ -114,6 +120,9 @@ class image {
 	*
 	**/
 	public function detailsImage($id){
+		if(!isset($id)){
+			// Renvoyer erreur 
+		}
 		//vérifier existence image
 		$service = $this->libClass;
 		$image = $service->getImage($id);
@@ -130,6 +139,12 @@ class image {
 	*	options for the image creation
 	**/
 	public function updateImage($id, array $opt){
+		if(!isset($id)){
+			// Renvoyer erreur 
+		}
+		if(!isset($opt)){
+			// Renvoyer erreur 
+		}
 		//vérifier existence image
 		$service = $this->libClass;
 		$image = $service->getImage($id);
@@ -165,11 +180,26 @@ class image {
 	* @param string $id
 	*	identifier of the image
 	**/
-	public function imageDelete($id){
+	public function deleteImage($id){
 		// si protected = true, demander de le mettre a false
 		// vérifier existence image
-		$service = $this->libClass;
-		$service->getImage($id)->delete();
+		if(!isset($id)){
+			// Renvoyer erreur 
+		}
+		try{
+			$service = $this->libClass;
+			$service->getImage($id)->delete();
+		}catch(BadResponseError $e){
+			echo "YOLO";
+			$this->app->getErrorInstance->BadResponseHandler($e);
+        }catch(UserInputError $e){
+			$this->app->getErrorInstance->UserInputHandler($e);
+        }catch(BaseError $e){
+			$this->app->getErrorInstance->BaseErrorHandler($e);
+        }catch(NotImplementedError $e){
+			$this->app->getErrorInstance->NotImplementedHandler($e);
+        }
+		
 	}
 
 	/**
@@ -179,6 +209,9 @@ class image {
 	*	identifier of the image
 	**/
 	public function reactivateImage($id){
+		if(!isset($id)){
+			// Renvoyer erreur 
+		}
 		// vérifier existence image
 		$service = $this->libClass;
 		$image = $service->getImage($id);
@@ -192,6 +225,9 @@ class image {
 	*	identifier of the image
 	**/
 	public function desactivateImage($id){
+		if(!isset($id)){
+			// Renvoyer erreur 
+		}
 		// vérifier existence image
 		$service = $this->libClass;
 		$image = $service->getImage($id);
@@ -208,6 +244,12 @@ class image {
 	*	path of the image
 	**/
 	public function uploadImage($id, $file_name){
+		if(!isset($id)){
+			// Renvoyer erreur 
+		}
+		if(!isset($file_name)){
+			// Renvoyer erreur 
+		}
 		// vérifier existence image
 		$service = $this->libClass;
 		$image  = $service->getImage($id);
@@ -222,6 +264,9 @@ class image {
 	*	identifier of the image
 	**/
 	public function downloadImage($id){
+		if(!isset($id)){
+			// Renvoyer erreur 
+		}
 		// vérifier existence image 
 		$service = $this->libClass;
 		$image  = $service->getImage($id);
@@ -239,6 +284,12 @@ class image {
 	*	identifier of the member
 	**/
 	public function addMemberImage($image_id, $member_id){
+		if(!isset($image_id)){
+			// Renvoyer erreur 
+		}
+		if(!isset($member_id)){
+			// Renvoyer erreur 
+		}
 		// vérifier existence image
 		// on doit être le proprio de l'image
 		// vérifier membre existe
@@ -254,6 +305,12 @@ class image {
 	*	identifier of the image
 	**/
 	public function listMemberImage($image_id, $member_id){
+		if(!isset($image_id)){
+			// Renvoyer erreur 
+		}
+		if(!isset($member_id)){
+			// Renvoyer erreur 
+		}
 		// vérifier existence image
 		$service = $this->libClass;
 		$image = $service->getImage($image_id);
@@ -271,6 +328,12 @@ class image {
 	*	identifier of the member
 	**/
 	public function detailMemberImage($image_id, $member_id){
+		if(!isset($image_id)){
+			// Renvoyer erreur 
+		}
+		if(!isset($member_id)){
+			// Renvoyer erreur 
+		}
 		// vérifier existence image
 		// on doit être le proprio de l'image
 		// vérifier membre existe
@@ -289,6 +352,12 @@ class image {
 	*	identifier of the member
 	**/
 	public function removeMemberImage($image_id, $member_id){
+		if(!isset($image_id)){
+			// Renvoyer erreur 
+		}
+		if(!isset($member_id)){
+			// Renvoyer erreur 
+		}
 		// vérifier existence image
 		// on doit être le proprio de l'image
 		// vérifier membre existe
@@ -309,6 +378,12 @@ class image {
 	*	new status for the member
 	**/
 	public function updateMemberImage($image_id, $member_id, $status){
+		if(!isset($image_id)){
+			// Renvoyer erreur 
+		}
+		if(!isset($member_id)){
+			// Renvoyer erreur 
+		}
 		// vérifier existence image
 		// on doit être le proprio de l'image
 		// vérifier membre existe