SMS bugs solved, refactor

debian/changelog

@@ -1,4 +1,4 @@
-hedera-web (1.405.46) stable; urgency=low
+hedera-web (1.405.47) stable; urgency=low
 
   * Initial Release.
 

forms/ecomerce/catalog/catalog.js

@@ -303,7 +303,7 @@ Hedera.Catalog = new Class
 	{
 		var sql = '';
 		var batch = new Sql.Batch ();
-		var query = new Sql.String ({query: 'CALL basket_item_add (#warehouse, #item, #amount);'});
+		var query = new Sql.String ({query: 'CALL basketAddItem (#warehouse, #item, #amount);'});
 		var amountSum = 0;
 
 		for (var warehouse in this.items)

forms/ecomerce/confirm/ui.xml

@@ -25,7 +25,7 @@
 	</db-form>
 	<db-query id="confirm-query" on-ready="onConfirm">
 		<custom>
-			CALL basket_confirm ()
+			CALL basketConfirm
 		</custom>
 	</db-query>
 </vn-group>

package.json

@@ -1,6 +1,6 @@
 {
   "name": "hedera-web",
-  "version": "1.405.46",
+  "version": "1.405.47",
   "description": "Verdnatura web page",
   "license": "GPL-3.0",
   "repository": {

rest/misc/sms.php

@@ -11,39 +11,34 @@ class Sms extends Vn\Web\JsonRequest
 
 	function run ($db)
 	{
-		$smsConfig = $db->getRow ('SELECT uri, user, password, title FROM sms_config');
+		$smsConfig = $db->getObject ('SELECT uri, user, password, title FROM sms_config');
 
-		$sClient = new SoapClient ($smsConfig['uri']);
+		$sClient = new SoapClient ($smsConfig->uri);
 		$xmlString = $sClient->sendSMS (
-			 $smsConfig['user']
+			 $smsConfig->user
-			,$smsConfig['password']
+			,$smsConfig->password
-			,$smsConfig['title']
+			,$smsConfig->title
 			,$_REQUEST['to']
 			,$_REQUEST['text']
 		);
 		$xmlResponse = new SimpleXMLElement ($xmlString);
+		$res = $xmlResponse->sms;
 
-		$customer = empty ($_REQUEST['customer']) ? NULL : $_REQUEST['customer'];
+		$db->query (
+			'INSERT INTO vn2008.sms (Id_trabajador, `text`, `to`, `from`, sent, response, Id_Cliente)
+				VALUES (account.userGetId(), #, #, #, #, #, #)',
+			[
+				 $_REQUEST['text']
+				,$_REQUEST['to']
+				,$_REQUEST['to']
+				,$res->codigo
+				,$res->descripcion
+				,empty ($_REQUEST['customer']) ? NULL : $_REQUEST['customer']
+			]
+		);
 
-		try {
+		if ($sms->codigo != 0)
-			$sms = $xmlResponse->sms;
+			throw new Lib\UserException ($res->descripcion);
-			$db->query (
-				'INSERT INTO vn2008.sms (Id_trabajador, `text`, `to`, `from`, sent, response, Id_Cliente)
-					VALUES (account.userGetId(), #, #, #, #, #, #)',
-				[
-					 $_REQUEST['text']
-					,$_REQUEST['to']
-					,$_REQUEST['to']
-					,$sms->codigo
-					,$sms->descripcion
-					,$customer
-				]
-			);
-		}
-		catch (Exception $e)
-		{
-			trigger_error ($e->getMessage (), E_USER_WARNING);
-		}
 
 		return TRUE;
 	}

web/service.php

@@ -87,7 +87,8 @@ abstract class Service
 
 		if (isset ($_COOKIE['PHPSESSID'])
 		|| isset ($_SESSION['access'])
-		|| isset ($_SESSION['skipVisit']))
+		|| isset ($_SESSION['skipVisit'])
+		|| !isset ($_SERVER['HTTP_USER_AGENT']))
 			return;
 
 		$agent = $_SERVER['HTTP_USER_AGENT'];