#4365 Fixes & refactor

debian/changelog

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

forms/account/address-list/address-list.js

@@ -1,6 +1,5 @@
 
-Hedera.AddressList = new Class
+Hedera.AddressList = new Class({
-({
 	Extends: Hedera.Form
 
 	,activate: function() {

forms/account/address-list/ui.xml

@@ -2,20 +2,16 @@
 <vn-group>
 	<db-form id="user-form">
 		<db-model property="model" id="user-model" updatable="true">
-			<custom>
+			SELECT id, defaultAddressFk
-				SELECT id, defaultAddressFk
+				FROM myClient c
-					FROM myClient c
-			</custom>
 		</db-model>
 	</db-form>
 	<db-model id="addresses" updatable="true">
-		<custom>
+		SELECT a.id, a.nickname, p.name province, a.postalCode,
-			SELECT a.id, a.nickname, p.name province, a.postalCode,
+		a.city, a.street, a.isActive
-			a.city, a.street, a.isActive
+			FROM myAddress a
-				FROM myAddress a
+			LEFT JOIN vn.province p ON p.id = a.provinceFk
-				LEFT JOIN vn.province p ON p.id = a.provinceFk
+				WHERE a.isActive
-					WHERE a.isActive
-		</custom>
 	</db-model>
 </vn-group>
 <div id="title">

forms/ecomerce/checkout/ui.xml

@@ -118,9 +118,9 @@
 							<db-model property="model" id="addresses">
 								SELECT a.id, a.nickname, p.name province, a.city, a.street, a.isActive, c.country
 									FROM myAddress a
-									LEFT JOIN vn.province p ON p.id = a.provinceFk
+										LEFT JOIN vn.province p ON p.id = a.provinceFk
-									JOIN vn.country c ON c.id = p.countryFk
+										JOIN vn.country c ON c.id = p.countryFk
-										WHERE a.isActive
+									WHERE a.isActive
 							</db-model>
 							<custom>
 								<div

package.json

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

web/json-service.php

@@ -3,6 +3,7 @@
 namespace Vn\Web;
 
 use Vn\Lib;
+use Vn\Lib\Locale;
 
 /**
  * Base class for JSON application.
@@ -50,7 +51,7 @@ class JsonService extends RestService {
 		if (_ENABLE_DEBUG || $errno & $eUser)
 			$json->message = $message;
 		else
-			$json->message = \s('Something went wrong');
+			$json->message = s('Something went wrong');
 		
 		if (_ENABLE_DEBUG) {
 			$json->code = $errno;
@@ -80,7 +81,7 @@ class JsonService extends RestService {
 			$json->message = $e->getMessage();
 		} else {
 			$json->exception = 'Exception';
-			$json->message = \s('Something went wrong');
+			$json->message = s('Something went wrong');
 		}
 
 		if (_ENABLE_DEBUG) {

web/rest-service.php

@@ -34,7 +34,7 @@ class RestService extends Service {
 		$isAuthorized = $db->getValue('SELECT myUser_checkRestPriv(#)',
 			[$_REQUEST['method']]);
 		if (!$isAuthorized)
-			throw new ForbiddenException(\s('You don\'t have enough privileges'));
+			throw new ForbiddenException(s('You don\'t have enough privileges'));
 
 		if ($method::SECURITY == Security::DEFINER) {
 			$methodDb = $db;
@@ -42,7 +42,7 @@ class RestService extends Service {
 			$methodDb = $this->getUserDb($_SESSION['user']);
 
 		if ($method::PARAMS !== NULL && !$method->checkParams($_REQUEST, $method::PARAMS))
-			throw new UserException (\s('Missing parameters'));
+			throw new UserException (s('Missing parameters'));
 		
 		Locale::addPath('rest/'. dirname($_REQUEST['method']));
 
@@ -52,7 +52,7 @@ class RestService extends Service {
 			$res = $method->run($methodDb);
 		} catch (Db\Exception $e) {
 			if ($e->getCode() == 1644)
-				throw new UserException(\s($e->getMessage()));
+				throw new UserException(s($e->getMessage()));
 		}
 
 		if ($method::SECURITY == Security::DEFINER)