Changeset 691 for trunk/phpbms/modules/api/include/apiclass.php

Timestamp:

12/30/09 14:17:21 (2 years ago)

Author:

nate

Message:

• Added additional data check for api get requests.
• The files's class getRecord no longer loads in the entire binary file into memory when the function is called. The function also returns a field named 'apifileurl' that gives the to get a file via an api login.
• Added a file that serves a files record's file field using api login credentials posted to it.

Files:

• trunk/phpbms/modules/api/include/apiclass.php (modified) (2 diffs)

trunk/phpbms/modules/api/include/apiclass.php

@@ -598 +598 @@
                             $processor->timeFormat =  $this->options->timeFormat;
                         }//end if
-
-                        if(!$this->options->useUuid){
+                        
+                        $errorMessage = "";
+                        if($this->options->useUuid){
+                            if(!isset($request["data"]["uuid"]))
+                                $errorMessage = "The `uuid` field must be set.";
+                        }else{
+                            if(!isset($request["data"]["id"]))
+                                $errorMessage = "The `id` field must be set.";
+                        }//end if
+                        
+                        if($errorMessage)
+                            $this->sendError("Update failed from request number ".$i, $errorMessage);
+                        elseif(!$this->options->useUuid){
                             $therecord = $processor->getRecord((int) $request["data"]["id"], $this->options->useUuid);
                             $thereturn = $therecord["id"];
@@ -608 +619 @@
                             $thevalue = $request["data"]["uuid"];
                         }
-
 
                         if($thereturn == $thevalue)