Changeset 392 for server/trunk/web/lib/Sophie/Controller/Chat

Timestamp:

11/04/11 23:31:47 (13 years ago)

Author:

grenoya

Message:

• add test in set function to be sure that the user spell correctly the variable to set

File:

• server/trunk/web/lib/Sophie/Controller/Chat/Cmd.pm (modified) (4 diffs)

server/trunk/web/lib/Sophie/Controller/Chat/Cmd.pm

@@ -196 +196 @@
 sub set : XMLRPC {
     my ( $self, $c, $reqspec, $var, $val ) = @_;
-
+    
+    # if there is no variable to fix, Sophie ask and stop
     if (!$var) {
         return $c->stash->{xmlrpc} = {
@@ -206 +207 @@
     } 
     
+    # if the variable is not 'distribution', 'release' or 'arch', Sophie
+    # complains and stop
+    if ($var != "distribution" || $var != "release" || $var != "arch") {
+        return $c->stash->{xmlrpc} = {
+            private_reply => 1,
+            message => [
+                "'$var' is not valid ! possible parameters are : 'distribution',
+                'release' and 'arch'."
+            ]
+        }
+    }
+
+    # if there is no value to give to the variable, Sophie ask and stop
     if (!$val) {
         return $c->stash->{xmlrpc} = {
@@ -239 +253 @@
     my ( $self, $c, $reqspec, $var ) = @_;
 
-
+    # if there is no variable to fix, Sophie ask and stop
     if (!$var) {
         return $c->stash->{xmlrpc} = {
@@ -249 +263 @@
     } 
     
+    # if the variable is not 'distribution', 'release' or 'arch', Sophie
+    # complains and stop
+    if ($var != "distribution" || $var != "release" || $var != "arch") {
+        return $c->stash->{xmlrpc} = {
+            private_reply => 1,
+            message => [
+                "'$var' is not valid ! possible parameters are : 'distribution',
+                'release' and 'arch'."
+            ]
+        }
+    }
+
     $c->forward('/user/update_data', [ $reqspec->{from}, { $var => undef } ]);