From 1aad7590330f7e4c6c9e4295ea2171e28eae4a73 Mon Sep 17 00:00:00 2001
From: Freddy Daniel Rojas Valda <daniel.rojas@colosa.com>
Date: Fri, 14 Feb 2014 17:00:03 -0400
Subject: [PATCH 1/9] Se adicionan validaciones a PMUSER

---
 workflow/engine/src/BusinessModel/User.php    | 55 +++++++++++++------
 .../src/Services/Api/ProcessMaker/User.php    |  3 +
 2 files changed, 40 insertions(+), 18 deletions(-)

diff --git a/workflow/engine/src/BusinessModel/User.php b/workflow/engine/src/BusinessModel/User.php
index e5582d15c..38b0eb9af 100644
--- a/workflow/engine/src/BusinessModel/User.php
+++ b/workflow/engine/src/BusinessModel/User.php
@@ -207,10 +207,16 @@ class User
     public function create($arrayData)
     {
         try {
-            global $RBAC;
+            global $RBAC;            
             require_once (PATH_TRUNK . "workflow" . PATH_SEP . "engine" . PATH_SEP . "classes" . PATH_SEP . "model" . PATH_SEP . "Users.php");
             $arrayData = array_change_key_case($arrayData, CASE_UPPER);
             $form = $arrayData;
+            if ($form['USR_REPLACED_BY'] != '') {
+                $oReplacedBy = \UsersPeer::retrieveByPK($form['USR_REPLACED_BY']);
+                if (is_null($oReplacedBy)) {
+                    throw new \Exception('`usr_replaced_by`:'.$form['USR_REPLACED_BY'].' '.\G::LoadTranslation('ID_AUTHENTICATION_SOURCE_INVALID'));
+                }
+            }
             if (isset($arrayData['USR_UID'])) {
                 $form['USR_UID'] = $arrayData['USR_UID'];
             } else {
@@ -255,10 +261,14 @@ class User
             } else {
                 $aData['USR_LASTNAME'] = $form['USR_LASTNAME'];
             }
-            if (!filter_var($form['USR_EMAIL'], FILTER_VALIDATE_EMAIL)) {
-                throw new \Exception('`usr_email`. '.\G::LoadTranslation('ID_INCORRECT_EMAIL'));
+            if ($form['USR_EMAIL'] == '') {
+                throw new \Exception('`usr_email`. E-mail is required');
             } else {
-                $aData['USR_EMAIL'] = $form['USR_EMAIL'];
+                if (!filter_var($form['USR_EMAIL'], FILTER_VALIDATE_EMAIL)) {
+                    throw new \Exception('`usr_email`. '.\G::LoadTranslation('ID_INCORRECT_EMAIL'));
+                } else {
+                    $aData['USR_EMAIL'] = $form['USR_EMAIL'];
+                }
             }
             if ($form['USR_DUE_DATE'] == '') {
                 throw new \Exception('`usr_due_date`. '.\G::LoadTranslation('ID_MSG_ERROR_DUE_DATE'));
@@ -280,24 +290,28 @@ class User
             $aData['USR_AUTH_USER_DN'] = $form['USR_AUTH_USER_DN'];
             $statusWF = $form['USR_STATUS'];
             if ($form['USR_STATUS'] == '') {
-                throw new \Exception('`usr_status`. '.\G::LoadTranslation('ID_SOME_FIELDS_REQUIRED'));
+                throw new \Exception('`usr_status`. User status is required');
             } else {
                 if ($form['USR_STATUS'] == 'ACTIVE' || $form['USR_STATUS'] == 'INACTIVE' || $form['USR_STATUS'] == 'VACATION') {
                     $aData['USR_STATUS'] = $form['USR_STATUS'];
                 } else {
-                    throw new \Exception('`usr_status`. Invalid value for field.');
+                    throw new \Exception('`usr_status`. Invalid value for status field.');
                 }
             }
-            $oCriteria = new \Criteria('rbac');
-            $oCriteria->add(\RolesPeer::ROL_CODE, $form['USR_ROLE']);
-            $oDataset = \RolesPeer::doSelectRS($oCriteria);
-            $oDataset->setFetchmode(\ResultSet::FETCHMODE_ASSOC);
-            $oDataset->next();
-            $aRow = $oDataset->getRow();
-            if ($oDataset->getRow()) {
-                $aData['USR_ROLE'] = $form['USR_ROLE'];
+            if ($form['USR_ROLE'] == '') {
+                throw new \Exception('`usr_role`. User role is required');
             } else {
-                throw new \Exception('`usr_role`. Invalid value for field.');
+                $oCriteria = new \Criteria('rbac');
+                $oCriteria->add(\RolesPeer::ROL_CODE, $form['USR_ROLE']);
+                $oDataset = \RolesPeer::doSelectRS($oCriteria);
+                $oDataset->setFetchmode(\ResultSet::FETCHMODE_ASSOC);
+                $oDataset->next();
+                $aRow = $oDataset->getRow();
+                if ($oDataset->getRow()) {
+                    $aData['USR_ROLE'] = $form['USR_ROLE'];
+                } else {
+                    throw new \Exception('`usr_role`. Invalid value for role field.');
+                }
             }
             try {
                 if ($aData['USR_STATUS'] == 'ACTIVE') {
@@ -322,7 +336,7 @@ class User
             $aData['USR_PHONE'] = $form['USR_PHONE'];
             $aData['USR_ZIP_CODE'] = $form['USR_ZIP_CODE'];
             $aData['USR_POSITION'] = $form['USR_POSITION'];
-            $aData['USR_REPLACED_BY'] = $form['USR_REPLACED_BY'];
+            $aData['USR_REPLACED_BY'] = $form['USR_REPLACED_BY'];           
             $oUser = new \Users();
             $oUser -> create( $aData );
             // comment photos files
@@ -530,8 +544,13 @@ class User
             if ($form['USR_ROLE'] != '') {
                 $aData['USR_ROLE'] = $form['USR_ROLE'];
             }
-            if (isset($form['USR_REPLACED_BY'])) {
-                $aData['USR_REPLACED_BY'] = $form['USR_REPLACED_BY'];
+            if ($form['USR_REPLACED_BY'] != '') {
+                $oReplacedBy = \UsersPeer::retrieveByPK($form['USR_REPLACED_BY']);
+                if (is_null($oReplacedBy)) {
+                    throw new \Exception('`usr_replaced_by`:'.$form['USR_REPLACED_BY'].' '.\G::LoadTranslation('ID_AUTHENTICATION_SOURCE_INVALID'));
+                } else {
+                    $aData['USR_REPLACED_BY'] = $form['USR_REPLACED_BY'];
+                }
             }
             if (isset($form['USR_AUTH_USER_DN'])) {
                 $aData['USR_AUTH_USER_DN'] = $form['USR_AUTH_USER_DN'];
diff --git a/workflow/engine/src/Services/Api/ProcessMaker/User.php b/workflow/engine/src/Services/Api/ProcessMaker/User.php
index 90f578d8f..f87ddca43 100644
--- a/workflow/engine/src/Services/Api/ProcessMaker/User.php
+++ b/workflow/engine/src/Services/Api/ProcessMaker/User.php
@@ -13,6 +13,9 @@ class User extends Api
 {
     /**
      * @url GET
+     * @param string $filter
+     * @param int $start
+     * @param int $limit
      */
     public function doGetUsers($filter = '', $start = null, $limit = null)
     {

From 862696096764dded5147d7889b3970b1dd068d21 Mon Sep 17 00:00:00 2001
From: Freddy Daniel Rojas Valda <daniel.rojas@colosa.com>
Date: Fri, 14 Feb 2014 17:04:12 -0400
Subject: [PATCH 2/9] Se adicionan validaciones a PMUSER

---
 workflow/engine/src/BusinessModel/User.php | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/workflow/engine/src/BusinessModel/User.php b/workflow/engine/src/BusinessModel/User.php
index 38b0eb9af..e0b80e880 100644
--- a/workflow/engine/src/BusinessModel/User.php
+++ b/workflow/engine/src/BusinessModel/User.php
@@ -207,7 +207,7 @@ class User
     public function create($arrayData)
     {
         try {
-            global $RBAC;            
+            global $RBAC;
             require_once (PATH_TRUNK . "workflow" . PATH_SEP . "engine" . PATH_SEP . "classes" . PATH_SEP . "model" . PATH_SEP . "Users.php");
             $arrayData = array_change_key_case($arrayData, CASE_UPPER);
             $form = $arrayData;
@@ -336,7 +336,7 @@ class User
             $aData['USR_PHONE'] = $form['USR_PHONE'];
             $aData['USR_ZIP_CODE'] = $form['USR_ZIP_CODE'];
             $aData['USR_POSITION'] = $form['USR_POSITION'];
-            $aData['USR_REPLACED_BY'] = $form['USR_REPLACED_BY'];           
+            $aData['USR_REPLACED_BY'] = $form['USR_REPLACED_BY'];
             $oUser = new \Users();
             $oUser -> create( $aData );
             // comment photos files

From 69bd91f99617bc2ec7e5beb7aeabaf29633fe6de Mon Sep 17 00:00:00 2001
From: Freddy Daniel Rojas Valda <daniel.rojas@colosa.com>
Date: Mon, 17 Feb 2014 11:29:38 -0400
Subject: [PATCH 3/9] Se adicionan validaciones a PMUSER y PROCESS SUPERVISOR

---
 .../src/BusinessModel/ProcessSupervisor.php   |  4 +-
 workflow/engine/src/BusinessModel/User.php    | 63 +++++++++++++++----
 2 files changed, 55 insertions(+), 12 deletions(-)

diff --git a/workflow/engine/src/BusinessModel/ProcessSupervisor.php b/workflow/engine/src/BusinessModel/ProcessSupervisor.php
index 283a5d71d..e233fd6a9 100644
--- a/workflow/engine/src/BusinessModel/ProcessSupervisor.php
+++ b/workflow/engine/src/BusinessModel/ProcessSupervisor.php
@@ -50,6 +50,7 @@ class ProcessSupervisor
             $oCriteria->addJoin(\ProcessUserPeer::USR_UID, \UsersPeer::USR_UID, \Criteria::LEFT_JOIN);
             $oCriteria->add(\ProcessUserPeer::PU_TYPE, 'SUPERVISOR');
             $oCriteria->add(\ProcessUserPeer::PRO_UID, $sProcessUID);
+            $oCriteria->add(\UsersPeer::USR_STATUS, 'ACTIVE');
             $oCriteria->addAscendingOrderByColumn(\UsersPeer::USR_FIRSTNAME);
             $oDataset = \ProcessUserPeer::doSelectRS($oCriteria);
             $oDataset->setFetchmode(\ResultSet::FETCHMODE_ASSOC);
@@ -61,7 +62,7 @@ class ProcessSupervisor
                                  'usr_firstname' => $aRow['USR_FIRSTNAME'],
                                  'usr_lastname' => $aRow['USR_LASTNAME'],
                                  'usr_username' => $aRow['USR_USERNAME'],
-                                 'usr_email' => $aRow['USR_EMAIL'] );
+                                 'usr_email' => $aRow['USR_EMAIL']);
                 $oDataset->next();
             }
             if ($aResp == null) {
@@ -218,6 +219,7 @@ class ProcessSupervisor
             $oCriteria->add(\UsersPeer::USR_UID, $aUIDS, \Criteria::IN);
             $oCriteria->addAscendingOrderByColumn(\UsersPeer::USR_FIRSTNAME);
             $oCriteria->add(\UsersPeer::USR_ROLE, 'PROCESSMAKER_ADMIN', \Criteria::EQUAL);
+            $oCriteria->add(\UsersPeer::USR_STATUS, 'ACTIVE');
             $oDataset = \UsersPeer::doSelectRS($oCriteria);
             $oDataset->setFetchmode(\ResultSet::FETCHMODE_ASSOC);
             $oDataset->next();
diff --git a/workflow/engine/src/BusinessModel/User.php b/workflow/engine/src/BusinessModel/User.php
index e0b80e880..7910e8cbb 100644
--- a/workflow/engine/src/BusinessModel/User.php
+++ b/workflow/engine/src/BusinessModel/User.php
@@ -217,6 +217,30 @@ class User
                     throw new \Exception('`usr_replaced_by`:'.$form['USR_REPLACED_BY'].' '.\G::LoadTranslation('ID_AUTHENTICATION_SOURCE_INVALID'));
                 }
             }
+            if ($form['USR_COUNTRY'] != '') {
+                $oCountry = \IsoCountryPeer::retrieveByPK($form['USR_COUNTRY']);
+                if (is_null($oCountry)) {
+                        throw new \Exception('invalid value for `usr_country`: '.$form['USR_COUNTRY']);
+                    }
+            }
+            if ($form['USR_CITY'] != '') {
+                $oCity = \IsoSubdivisionPeer::retrieveByPK($form['USR_COUNTRY'], $form['USR_CITY']);
+                if (is_null($oCity)) {
+                        throw new \Exception('invalid value for `usr_city`: '.$form['USR_CITY']);
+                    }
+            }
+            if ($form['USR_LOCATION'] != '') {
+                $oLocation = \IsoLocationPeer::retrieveByPK($form['USR_COUNTRY'], $form['USR_LOCATION']);
+                if (is_null($oLocation)) {
+                        throw new \Exception('invalid value for `usr_location`: '.$form['USR_LOCATION']);
+                    }
+            }
+            if ($form['USR_COUNTRY'] != '') {
+                $oReplacedBy = \IsoCountryPeer::retrieveByPK($form['USR_COUNTRY']);
+                if (is_null($oReplacedBy)) {
+                        throw new \Exception('invalid value for `usr_country`: '.$form['USR_COUNTRY']);
+                    }
+            }
             if (isset($arrayData['USR_UID'])) {
                 $form['USR_UID'] = $arrayData['USR_UID'];
             } else {
@@ -322,7 +346,7 @@ class User
                 }
                 $sUserUID = $this->createUser($aData);
                 if ($form['USR_ROLE'] != '') {
-                   $this->assignRoleToUser($sUserUID, $form['USR_ROLE']);
+                    $this->assignRoleToUser($sUserUID, $form['USR_ROLE']);
                 }
             } catch(Exception $oError) {
                 throw new \Exception($oError->getMessage());
@@ -381,7 +405,7 @@ class User
             $form = $arrayData;
             $countPermission = 0;
             $permission = $this->loadUserRolePermission($RBAC->sSystem, $usrLoggedUid);
-            foreach($permission as $key => $value) {
+            foreach ($permission as $key => $value) {
                 if ($value["PER_CODE"] == 'PM_USERS') {
                     $countPermission+=1;
                 }
@@ -400,12 +424,6 @@ class User
             if ($form['USR_NEW_PASS'] != '') {
                 $form['USR_PASSWORD'] = md5($form['USR_NEW_PASS']);
             }
-            if (!isset($form['USR_CITY'])) {
-                $form['USR_CITY'] = '';
-            }
-            if (!isset($form['USR_LOCATION'])) {
-                $form['USR_LOCATION'] = '';
-            }
             if (!isset($form['USR_AUTH_USER_DN'])) {
                 $form['USR_AUTH_USER_DN'] = '';
             }
@@ -534,9 +552,32 @@ class User
             } else {
                 $this->updateUser($aData);
             }
-            $aData['USR_COUNTRY'] = $form['USR_COUNTRY'];
-            $aData['USR_CITY'] = $form['USR_CITY'];
-            $aData['USR_LOCATION'] = $form['USR_LOCATION'];
+            if ($form['USR_COUNTRY'] != '') {
+                $oReplacedBy = \IsoCountryPeer::retrieveByPK($form['USR_COUNTRY']);
+                if (is_null($oReplacedBy)) {
+                        throw new \Exception('invalid value for `usr_country`: '.$form['USR_COUNTRY']);
+                } else {
+                    $aData['USR_COUNTRY'] = $form['USR_COUNTRY'];
+                    $aData['USR_CITY'] = '';
+                    $aData['USR_LOCATION'] = '';
+                }
+            }
+            if ($form['USR_CITY'] != '') {
+                $oCity = \IsoSubdivisionPeer::retrieveByPK($form['USR_COUNTRY'], $form['USR_CITY']);
+                if (is_null($oCity)) {
+                    throw new \Exception('invalid value for `usr_city`: '.$form['USR_CITY']);
+                } else {
+                    $aData['USR_CITY'] = $form['USR_CITY'];
+                }
+            }
+            if ($form['USR_LOCATION'] != '') {
+                $oLocation = \IsoLocationPeer::retrieveByPK($form['USR_COUNTRY'], $form['USR_LOCATION']);
+                if (is_null($oLocation)) {
+                        throw new \Exception('invalid value for `usr_location`: '.$form['USR_LOCATION']);
+                } else {
+                    $aData['USR_LOCATION'] = $form['USR_LOCATION'];
+                }
+            }
             $aData['USR_ADDRESS'] = $form['USR_ADDRESS'];
             $aData['USR_PHONE'] = $form['USR_PHONE'];
             $aData['USR_ZIP_CODE'] = $form['USR_ZIP_CODE'];

From c13b6163d3ae3a8a8ecca7bf21abe72ee6b7cbf5 Mon Sep 17 00:00:00 2001
From: Freddy Daniel Rojas Valda <daniel.rojas@colosa.com>
Date: Mon, 17 Feb 2014 11:39:20 -0400
Subject: [PATCH 4/9] Se adicionan validaciones a PMUSER

---
 workflow/engine/src/BusinessModel/User.php | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/workflow/engine/src/BusinessModel/User.php b/workflow/engine/src/BusinessModel/User.php
index 7910e8cbb..a163c8441 100644
--- a/workflow/engine/src/BusinessModel/User.php
+++ b/workflow/engine/src/BusinessModel/User.php
@@ -734,6 +734,7 @@ class User
                     }
                 }
             }
+            $oCriteria->add(\UsersPeer::USR_STATUS, 'CLOSED', \Criteria::ALT_NOT_EQUAL);
             $oDataset = \UsersPeer::doSelectRS($oCriteria);
             $oDataset->setFetchmode(\ResultSet::FETCHMODE_ASSOC);
             while ($oDataset->next()) {
@@ -770,6 +771,7 @@ class User
                 $oCriteria->add( $oCriteria->getNewCriterion( \UsersPeer::USR_USERNAME, "%$filter%", \Criteria::LIKE )->addOr( $oCriteria->getNewCriterion( \UsersPeer::USR_FIRSTNAME, "%$filter%", \Criteria::LIKE ) )->addOr( $oCriteria->getNewCriterion( \UsersPeer::USR_LASTNAME, "%$filter%", \Criteria::LIKE ) ) );
             }
             $oCriteria->add(\UsersPeer::USR_UID, $userUid);
+            $oCriteria->add(\UsersPeer::USR_STATUS, 'CLOSED', \Criteria::ALT_NOT_EQUAL);
             $oDataset = \UsersPeer::doSelectRS($oCriteria);
             $oDataset->setFetchmode(\ResultSet::FETCHMODE_ASSOC);
             while ($oDataset->next()) {

From 1568154e9d214b3b0adf62df7d0d1cca614f9ee0 Mon Sep 17 00:00:00 2001
From: Brayan Osmar Pereyra Suxo <brayan@colosa.com>
Date: Mon, 17 Feb 2014 16:08:06 -0400
Subject: [PATCH 5/9] correcion de label en DATABASE CONNECTION

---
 .../engine/src/BusinessModel/DataBaseConnection.php    | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/workflow/engine/src/BusinessModel/DataBaseConnection.php b/workflow/engine/src/BusinessModel/DataBaseConnection.php
index cd95698a0..8f0a2be67 100644
--- a/workflow/engine/src/BusinessModel/DataBaseConnection.php
+++ b/workflow/engine/src/BusinessModel/DataBaseConnection.php
@@ -105,21 +105,21 @@ class DataBaseConnection
                 $typesExists[] = $value['id'];
             }
             if (!in_array($dataDBConnection['DBS_TYPE'], $typesExists)) {
-                throw (new \Exception("This 'dbs_type' is invalid"));
+                throw (new \Exception("The dababase connection with dbs_type: '" . $dataDBConnection['DBS_TYPE'] . " is invalid"));
             }
         }
 
         if (isset($dataDBConnection['DBS_SERVER']) && $dataDBConnection['DBS_SERVER'] == '') {
-            throw (new \Exception("This 'dbs_server' is invalid"));
+            throw (new \Exception("The dababase connection with dbs_server: '" . $dataDBConnection['DBS_SERVER'] . "', is invalid"));
         }
 
         if (isset($dataDBConnection['DBS_DATABASE_NAME']) && $dataDBConnection['DBS_DATABASE_NAME'] == '') {
-            throw (new \Exception("This 'dbs_database_name' is invalid"));
+            throw (new \Exception("The dababase connection with dbs_database_name: '" . $dataDBConnection['DBS_DATABASE_NAME'] . "', is invalid"));
         }
 
         if (isset($dataDBConnection['DBS_PORT']) &&
             ($dataDBConnection['DBS_PORT'] == ''|| $dataDBConnection['DBS_PORT'] == 0)) {
-            throw (new \Exception("This 'dbs_port' is invalid"));
+            throw (new \Exception("The dababase connection with dbs_port: '" . $dataDBConnection['DBS_PORT'] . "', is invalid"));
         }
 
         if (isset($dataDBConnection['DBS_ENCODE'])) {
@@ -130,7 +130,7 @@ class DataBaseConnection
                 $encodesExists[] = $value['0'];
             }
             if (!in_array($dataDBConnection['DBS_ENCODE'], $encodesExists)) {
-                throw (new \Exception( "This 'dbs_encode' is invalid for '" . $dataDBConnection['DBS_TYPE'] . "'" ));
+                throw (new \Exception( "The dababase connection with dbs_encode: '" . $dataDBConnection['DBS_ENCODE'] . "', is invalid" ));
             }
         }
 

From 47e77ef1c1bbe6ffa76d57facee13820518a5417 Mon Sep 17 00:00:00 2001
From: Freddy Daniel Rojas Valda <daniel.rojas@colosa.com>
Date: Mon, 17 Feb 2014 16:13:17 -0400
Subject: [PATCH 6/9] Se adicionan validaciones a PMUSER. Se modifica download
 y upload (incompleto) para FILESMANAGER

---
 .../engine/src/BusinessModel/FilesManager.php    | 16 +++++-----------
 workflow/engine/src/BusinessModel/User.php       | 10 +++++-----
 .../Api/ProcessMaker/Project/FilesManager.php    | 11 +++++++++--
 3 files changed, 19 insertions(+), 18 deletions(-)

diff --git a/workflow/engine/src/BusinessModel/FilesManager.php b/workflow/engine/src/BusinessModel/FilesManager.php
index 5a656030a..f5900b1a1 100644
--- a/workflow/engine/src/BusinessModel/FilesManager.php
+++ b/workflow/engine/src/BusinessModel/FilesManager.php
@@ -391,18 +391,12 @@ class FilesManager
                     $sDirectory = PATH_DATA_MAILTEMPLATES . $sProcessUID . PATH_SEP . $sfile;
                     break;
             }
-            $arrayTaskUid = $this->getFileManagerUid($sDirectory);
-            if (!$arrayTaskUid) {
-                throw (new \Exception( 'invalid value specified for `path`.'));
-            }
-            /*
-            This is usefull when you are downloading big files, as it
-            will prevent time out of the script :
-            */
+            $url = $sDirectory;
+            $sPath = '/home/daniel/downloadTest/'.$sfile;//change the path to
             set_time_limit(0);
-            ini_set('display_errors',true);//Just in case we get some errors, let us know....
-            $fp = fopen ($sDirectory, 'w+');//This is the file where we save the information
-            $ch = curl_init($sDirectory);//Here is the file we are downloading
+            ini_set('display_errors',true);//Just in case we get some errors, let us know
+            $fp = fopen ($sPath, 'w');//This is the file where we save the information
+            $ch = curl_init(str_replace(" ","%20",$url));
             curl_setopt($ch, CURLOPT_TIMEOUT, 50);
             curl_setopt($ch, CURLOPT_FILE, $fp);
             curl_setopt($ch, CURLOPT_FOLLOWLOCATION, true);
diff --git a/workflow/engine/src/BusinessModel/User.php b/workflow/engine/src/BusinessModel/User.php
index a163c8441..85aa0a2a3 100644
--- a/workflow/engine/src/BusinessModel/User.php
+++ b/workflow/engine/src/BusinessModel/User.php
@@ -286,7 +286,7 @@ class User
                 $aData['USR_LASTNAME'] = $form['USR_LASTNAME'];
             }
             if ($form['USR_EMAIL'] == '') {
-                throw new \Exception('`usr_email`. E-mail is required');
+                throw new \Exception('invalid value specified for `usr_email`, can`t be null.');
             } else {
                 if (!filter_var($form['USR_EMAIL'], FILTER_VALIDATE_EMAIL)) {
                     throw new \Exception('`usr_email`. '.\G::LoadTranslation('ID_INCORRECT_EMAIL'));
@@ -313,8 +313,8 @@ class User
             $aData['USR_BIRTHDAY'] = date('Y-m-d');
             $aData['USR_AUTH_USER_DN'] = $form['USR_AUTH_USER_DN'];
             $statusWF = $form['USR_STATUS'];
-            if ($form['USR_STATUS'] == '') {
-                throw new \Exception('`usr_status`. User status is required');
+            if ($form['USR_STATUS'] == '') {                                
+                throw new \Exception('invalid value specified for `usr_status`, can`t be null');
             } else {
                 if ($form['USR_STATUS'] == 'ACTIVE' || $form['USR_STATUS'] == 'INACTIVE' || $form['USR_STATUS'] == 'VACATION') {
                     $aData['USR_STATUS'] = $form['USR_STATUS'];
@@ -322,8 +322,8 @@ class User
                     throw new \Exception('`usr_status`. Invalid value for status field.');
                 }
             }
-            if ($form['USR_ROLE'] == '') {
-                throw new \Exception('`usr_role`. User role is required');
+            if ($form['USR_ROLE'] == '') {                
+                throw new \Exception('invalid value specified for `usr_role`, can`t be null');
             } else {
                 $oCriteria = new \Criteria('rbac');
                 $oCriteria->add(\RolesPeer::ROL_CODE, $form['USR_ROLE']);
diff --git a/workflow/engine/src/Services/Api/ProcessMaker/Project/FilesManager.php b/workflow/engine/src/Services/Api/ProcessMaker/Project/FilesManager.php
index 05d342dff..7b4bb82b0 100644
--- a/workflow/engine/src/Services/Api/ProcessMaker/Project/FilesManager.php
+++ b/workflow/engine/src/Services/Api/ProcessMaker/Project/FilesManager.php
@@ -67,10 +67,17 @@ class FilesManager extends Api
     public function doPostProcessFilesManagerUpload($prjUid)
     {
         try {
-            //echo $request_data; die();
+            require_once (PATH_TRUNK . "workflow" . PATH_SEP . "engine" . PATH_SEP . "src" . PATH_SEP . "Extension" . PATH_SEP . "Restler" . PATH_SEP . "UploadFormat.php");
+//            $objCurlFileUploader = new CurlFileUploader("D:\\Program Files\\Apache\\htdocs\\test.txt", "http://localhost/fileUploaderTest.php",'file1'); 
+//            $objCurlFileUploader->UploadFile();
+
             $userUid = $this->getUserId();
+//            $objCurlFileUploader = new CurlFileUploader("D:\\Program Files\\Apache\\htdocs\\test.txt", "http://localhost/fileUploaderTest.php",'file1'); 
+            $filesManager = new \Extension\Restler\UploadFormat("/home/daniel/info.txt", "http://daniel.pmos.colosa.net/api/1.0/daniel/project/1265557095225ff5c688f46031700471/process-file-manager/upload",'file1');
+            $filesManager->UploadFormat();
+/*
             $filesManager = new \BusinessModel\FilesManager();
-            $arrayData = $filesManager->uploadProcessFilesManager($prjUid, $userUid);
+            $arrayData = $filesManager->uploadProcessFilesManager($prjUid, $userUid);*/
             //Response
             $response = $arrayData;
         } catch (\Exception $e) {

From 0a4aff7ea07e3408e1ce6c294797190af4842859 Mon Sep 17 00:00:00 2001
From: Freddy Daniel Rojas Valda <daniel.rojas@colosa.com>
Date: Mon, 17 Feb 2014 16:18:29 -0400
Subject: [PATCH 7/9] Se modifica download y upload (incompleto) para
 FILESMANAGER

---
 workflow/engine/src/BusinessModel/FilesManager.php       | 2 +-
 .../Services/Api/ProcessMaker/Project/FilesManager.php   | 9 ---------
 2 files changed, 1 insertion(+), 10 deletions(-)

diff --git a/workflow/engine/src/BusinessModel/FilesManager.php b/workflow/engine/src/BusinessModel/FilesManager.php
index f5900b1a1..5f6b65d34 100644
--- a/workflow/engine/src/BusinessModel/FilesManager.php
+++ b/workflow/engine/src/BusinessModel/FilesManager.php
@@ -392,7 +392,7 @@ class FilesManager
                     break;
             }
             $url = $sDirectory;
-            $sPath = '/home/daniel/downloadTest/'.$sfile;//change the path to
+            $sPath = $sfile;//path to
             set_time_limit(0);
             ini_set('display_errors',true);//Just in case we get some errors, let us know
             $fp = fopen ($sPath, 'w');//This is the file where we save the information
diff --git a/workflow/engine/src/Services/Api/ProcessMaker/Project/FilesManager.php b/workflow/engine/src/Services/Api/ProcessMaker/Project/FilesManager.php
index 7b4bb82b0..f04850374 100644
--- a/workflow/engine/src/Services/Api/ProcessMaker/Project/FilesManager.php
+++ b/workflow/engine/src/Services/Api/ProcessMaker/Project/FilesManager.php
@@ -68,16 +68,7 @@ class FilesManager extends Api
     {
         try {
             require_once (PATH_TRUNK . "workflow" . PATH_SEP . "engine" . PATH_SEP . "src" . PATH_SEP . "Extension" . PATH_SEP . "Restler" . PATH_SEP . "UploadFormat.php");
-//            $objCurlFileUploader = new CurlFileUploader("D:\\Program Files\\Apache\\htdocs\\test.txt", "http://localhost/fileUploaderTest.php",'file1'); 
-//            $objCurlFileUploader->UploadFile();
-
             $userUid = $this->getUserId();
-//            $objCurlFileUploader = new CurlFileUploader("D:\\Program Files\\Apache\\htdocs\\test.txt", "http://localhost/fileUploaderTest.php",'file1'); 
-            $filesManager = new \Extension\Restler\UploadFormat("/home/daniel/info.txt", "http://daniel.pmos.colosa.net/api/1.0/daniel/project/1265557095225ff5c688f46031700471/process-file-manager/upload",'file1');
-            $filesManager->UploadFormat();
-/*
-            $filesManager = new \BusinessModel\FilesManager();
-            $arrayData = $filesManager->uploadProcessFilesManager($prjUid, $userUid);*/
             //Response
             $response = $arrayData;
         } catch (\Exception $e) {

From 161674e3f5fc45f9fb0e76107c8d1675b0d872cf Mon Sep 17 00:00:00 2001
From: Brayan Osmar Pereyra Suxo <brayan@colosa.com>
Date: Mon, 17 Feb 2014 16:37:38 -0400
Subject: [PATCH 8/9] Correcion de update en REPORT TABLES

---
 workflow/engine/src/BusinessModel/Table.php | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/workflow/engine/src/BusinessModel/Table.php b/workflow/engine/src/BusinessModel/Table.php
index 9645813c4..e77294d60 100644
--- a/workflow/engine/src/BusinessModel/Table.php
+++ b/workflow/engine/src/BusinessModel/Table.php
@@ -526,7 +526,8 @@ class Table
                 $dataValidate['rep_tab_dsc']  = $tab_data['rep_tab_dsc'];
                 $dataValidate['rep_tab_connection'] = $row['DBS_UID'];
                 $dataValidate['rep_tab_type'] = $row['ADD_TAB_TYPE'];
-                $dataValidate['rep_tab_grid'] = $row['ADD_TAB_GRID'];
+                list($gridName, $gridId) = explode( '-', $row['ADD_TAB_GRID'] );
+                $dataValidate['rep_tab_grid'] = $gridId;
             } else {
                 $dataValidate['pmt_uid']  = $tab_uid;
                 $dataValidate['pmt_tab_name'] = $row['ADD_TAB_NAME'];

From 65a747b60c862e5be5c568dcf499d5d98d88ddea Mon Sep 17 00:00:00 2001
From: Brayan Osmar Pereyra Suxo <brayan@colosa.com>
Date: Mon, 17 Feb 2014 16:48:44 -0400
Subject: [PATCH 9/9] Arreglo en update

---
 workflow/engine/src/BusinessModel/Table.php | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/workflow/engine/src/BusinessModel/Table.php b/workflow/engine/src/BusinessModel/Table.php
index e77294d60..c58e3541e 100644
--- a/workflow/engine/src/BusinessModel/Table.php
+++ b/workflow/engine/src/BusinessModel/Table.php
@@ -526,8 +526,11 @@ class Table
                 $dataValidate['rep_tab_dsc']  = $tab_data['rep_tab_dsc'];
                 $dataValidate['rep_tab_connection'] = $row['DBS_UID'];
                 $dataValidate['rep_tab_type'] = $row['ADD_TAB_TYPE'];
-                list($gridName, $gridId) = explode( '-', $row['ADD_TAB_GRID'] );
-                $dataValidate['rep_tab_grid'] = $gridId;
+                $dataValidate['rep_tab_grid'] = '';
+                if (strpos($row['ADD_TAB_GRID'], '-')) {
+                    list($gridName, $gridId) = explode( '-', $row['ADD_TAB_GRID'] );
+                    $dataValidate['rep_tab_grid'] = $gridId;
+                }
             } else {
                 $dataValidate['pmt_uid']  = $tab_uid;
                 $dataValidate['pmt_tab_name'] = $row['ADD_TAB_NAME'];