PHP 7.3 upgrade, bugs fixed, improved JS log

debian/changelog

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

debian/control

@@ -9,7 +9,7 @@ Vcs-Git: https://git.verdnatura.es/hedera-web
 
 Package: hedera-web
 Architecture: all
-Depends: apache2 | httpd, nodejs, php-cli, php-vn-lib, php-apcu, php-ldap, php-ssh2, php-mcrypt, php-imap
+Depends: apache2 | httpd, nodejs, php-cli, php-vn-lib, php-apcu, php-ldap, php-ssh2, php-imap
 Suggests: php-text-captcha, php-soap, libphp-phpmailer, php-gd
 Section: misc
 Priority: optional

debian/postinst

@@ -10,6 +10,6 @@ then
     apache2_invoke enconf hedera-web.conf
 fi
 
-service php7.0-fpm restart
+#service php7.3-fpm restart
 service cron restart
 

forms/cms/home/home.js

@@ -1,11 +1,9 @@
 
-Hedera.Home = new Class
+Hedera.Home = new Class({
-({
 	Extends: Hedera.Form
 	
-	,onStartOrderClick: function ()
+	,onStartOrderClick: function() {
-	{
+		this.hash.set({'form': 'ecomerce/catalog'});
-		this.hash.set ({'form': 'ecomerce/catalog'});
 	}
 });
 

js/hedera/app.js

@@ -1,225 +1,199 @@
 
-var Login = require ('./login');
+var Login = require('./login');
-var Gui = require ('./gui');
+var Gui = require('./gui');
 
-module.exports = new Class
+module.exports = new Class({
-({
 	Extends: Vn.Object,
 	Properties:
 	{
 		conn:
 		{
 			type: Db.Connection
-			,get: function ()
+			,get: function() {
-			{
 				return this._conn;
 			}
 		}
 	}
 
-	,initialize: function ()
+	,initialize: function() {
-	{
+		window.onerror = this._onWindowError.bind(this);
-		window.onerror = this._onWindowError.bind (this);
+		window.onunload = this._onWindowUnload.bind(this);
-		window.onunload = this._onWindowUnload.bind (this);
+		Vn.Hash.initialize();
-		Vn.Hash.initialize ();
 
-		var conn = new Db.Connection ();
+		var conn = new Db.Connection();
-		this.link ({_conn: conn}, {'error': this._onConnError});
+		this.link({_conn: conn}, {'error': this._onConnError});
 
-		this.initAutoLogin ();
+		this.initAutoLogin();
 	}
 	
-	,run: function ()
+	,run: function() {
-	{
+		if (this.tryAutoLogin())
-		if (this.tryAutoLogin ())
 			return;
 	
-		var login = this._login = new Login ({conn: this._conn});
+		var login = this._login = new Login({conn: this._conn});
-		login.on ('login', this._onLogin, this);
+		login.on('login', this._onLogin, this);
-		login.show ();
+		login.show();
 	}
 	
-	,_onLogin: function ()
+	,_onLogin: function() {
-	{
+		this._freeLogin();
-		this._freeLogin ();
 
 		if (this._gui)
 			return;
 		
-		var gui = this._gui = new Gui ({conn: this._conn});
+		var gui = this._gui = new Gui({conn: this._conn});
-		gui.on ('logout', this._onLogout, this);
+		gui.on('logout', this._onLogout, this);
-		gui.show ();
+		gui.show();
 	}
 	
-	,_onLogout: function ()
+	,_onLogout: function() {
-	{
+		this.clearAutoLogin();
-		this.clearAutoLogin ();
+		this._freeGui();
-		this._freeGui ();
+		this.run();
-		this.run ();
 	}
 	
-	,_onWindowUnload: function ()
+	,_onWindowUnload: function() {
-	{
+		this.unref();
-		this.unref ();
 	}
 	
-	,_onWindowError: function (message, file, line, col, err)
+	,_onWindowError: function(message, file, line, col, err) {
-	{
+		var error = new Error(message);
-		var error = new Error (message);
 		error.fileName = file;
 		error.lineNumber = line;
-		this._notifyError (error);
+		this._notifyError(error);
 	}
 
-	,_onConnError: function (conn, error)
+	,_onConnError: function(conn, error) {
-	{
 		if (error instanceof Vn.JsonException)
-		switch (error.exception)
+		switch (error.exception) {
-		{
 			case 'BadLogin':
-				Htk.Toast.showError (_('Invalid login'));
+				Htk.Toast.showError(_('Invalid login'));
-				this._logout ();
+				this._logout();
 				break;
 			case 'SessionExpired':
-				Htk.Toast.showError (_('You\'ve been too idle'));
+				Htk.Toast.showError(_('You\'ve been too idle'));
-				this._logout ();
+				this._logout();
 				break;
 			case 'OutdatedVersion':
-				this._newVersion (error);
+				this._newVersion(error);
 				break;
 			default:
-				Htk.Toast.showError (error.message);
+				Htk.Toast.showError(error.message);
 		}
-		else
+		else {
-		{
+			console.error(error);
-			console.error (error);
+			this._notifyError(error);
-			this._notifyError (error);
 		}
 	}
 	
-	,_logout: function ()
+	,_logout: function() {
-	{
 		if (this._gui)
-			this._gui.logout ();
+			this._gui.logout();
 	}
 	
-	,_newVersion: function ()
+	,_newVersion: function() {
-	{
 		if (this.ignoreVersion)
 			return;
 
 		this.ignoreVersion = true;
 
-		var dialog = new Htk.Dialog ({
+		var dialog = new Htk.Dialog({
 			 message: _('New version available')
 			,buttons: Htk.Dialog.Button.ACCEPT
 			,icon: 'warning'
 		});
-		dialog.on ('response', this._onNewVersionResponse, this);
+		dialog.on('response', this._onNewVersionResponse, this);
-		dialog.open ();
+		dialog.open();
 	}
 	
-	,_onNewVersionResponse: function ()
+	,_onNewVersionResponse: function() {
-	{
+		location.reload();
-		location.reload ();
 	}
 	
-	,_notifyError: function (error)
+	,_notifyError: function(error) {
-	{
+		Htk.Toast.showError(_('Something went wrong'));
-		Htk.Toast.showError (_('Something went wrong'));
 
 		var params = {
 			 file: error.fileName
 			,line: error.lineNumber
 			,message: error.message
 			,stack: error.stack
+			,agent: navigator.userAgent
+			,location: location.href
 		};
-		this._conn.send ('core/log', params);
+		this._conn.send('core/log', params);
 	}
 	
-	,_freeLogin: function ()
+	,_freeLogin: function() {
-	{
+		if (this._login) {
-		if (this._login)
+			this._login.disconnectByInstance(this);
-		{
+			this._login.hide();
-			this._login.disconnectByInstance (this);
+			this._login.unref();
-			this._login.hide ();
-			this._login.unref ();
 			this._login = null;
 		}
 	}
 	
-	,_freeGui: function ()
+	,_freeGui: function() {
-	{
+		if (this._gui) {
-		if (this._gui)
+			this._gui.disconnectByInstance(this);
-		{
+			this._gui.hide();
-			this._gui.disconnectByInstance (this);
+			this._gui.unref();
-			this._gui.hide ();
-			this._gui.unref ();
 			this._gui = null;
 		}
 	}
 	
-	,_destroy: function ()
+	,_destroy: function() {
-	{
+		this._freeLogin();
-		this._freeLogin ();
+		this._freeGui();
-		this._freeGui ();
+		this.deinitAutoLogin();
-		this.deinitAutoLogin ();
+		this._conn.unref();
-		this._conn.unref ();
 	}
 	
 	// Auto login functionality
 	
 	,_firstLogin: true
 
-	,initAutoLogin: function ()
+	,initAutoLogin: function() {
-	{
+		var isGuest = new Vn.HashParam({
-		var isGuest = new Vn.HashParam
-		({
 			type: Boolean,
 			key: 'guest'
 		});
-		this.link ({_isGuest: isGuest}, {'changed': this._onGuestChange});
+		this.link({_isGuest: isGuest}, {'changed': this._onGuestChange});
 
-		var token = new Vn.HashParam
+		var token = new Vn.HashParam({
-		({
 			type: String,
 			key: 'token'
 		});
-		this.link ({_token: token}, {'changed': this._onTokenChange});
+		this.link({_token: token}, {'changed': this._onTokenChange});
 	}
 	
-	,_onGuestChange: function ()
+	,_onGuestChange: function() {
-	{
 		if (this._isGuest.value)
-			setTimeout (this.tryAutoLogin.bind (this));
+			setTimeout(this.tryAutoLogin.bind(this));
 	}
 	
-	,_onTokenChange: function ()
+	,_onTokenChange: function() {
-	{
 		if (this._token.value)
-			setTimeout (this.tryAutoLogin.bind (this));
+			setTimeout(this.tryAutoLogin.bind(this));
 	}
 	
-	,deinitAutoLogin: function ()
+	,deinitAutoLogin: function() {
-	{
+		this._isGuest.unref();
-		this._isGuest.unref ();
+		this._token.unref();
-		this._token.unref ();
 	}
 
-	,autoLogin: function ()
+	,autoLogin: function() {
-	{
+		var guest = localStorage.getItem('hederaGuest');
-		var guest = localStorage.getItem ('hederaGuest');
 
-		if (this._isGuest.value || guest)
+		if (this._isGuest.value || guest) {
-		{
+			localStorage.setItem('hederaGuest', true);
-			localStorage.setItem ('hederaGuest', true);
 			return true;
 		}
 
 		if (this._token.value)
 			this._conn.token = this._token.value;
 		else
-			this._conn.fetchToken ();
+			this._conn.fetchToken();
 		
 		if (this._conn.token)
 			return true;
@@ -227,9 +201,8 @@ module.exports = new Class
 		return false;
 	}
 	
-	,tryAutoLogin: function ()
+	,tryAutoLogin: function() {
-	{
+		var ok = this.autoLogin();
-		var ok = this.autoLogin ();
 
 		this._firstLogin = false;
 		this._isGuest.value = undefined;
@@ -238,13 +211,12 @@ module.exports = new Class
 		if (!ok)
 			return false;
 
-		this._onLogin ();
+		this._onLogin();
 		return true;
 	}
 	
-	,clearAutoLogin: function ()
+	,clearAutoLogin: function() {
-	{
+		localStorage.removeItem('hederaGuest');
-		localStorage.removeItem ('hederaGuest');
 	}
 });
 

package.json

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

rest/core/locale/mn.json

@@ -1,5 +0,0 @@
-{
-	 "InvalidAction": "Invalid action"
-
-	,"EmptyQuery": "Empty query"
-}

rest/core/log.php

@@ -2,21 +2,30 @@
 
 class Log extends Vn\Web\JsonRequest {
 	const PARAMS = [
-		 'file'
+		 'message'
+		,'file'
 		,'line'
-		,'message'
+		,'location'
+		,'agent'
 		,'stack'
 	];
 
 	function run($db) {			
 		$user = isset($_SESSION['user']) ? $_SESSION['user'] : 'guest';
-		error_log(sprintf("Javascript: User: %s: %s(%d): %s.\n%s"
+		$message   = $_REQUEST['message'];
-			,$user
+		$file      = $_REQUEST['file'];
-			,$_REQUEST['file']
+		$line      = $_REQUEST['line'];
-			,$_REQUEST['line']
+		$location  = $_REQUEST['location'];
-			,$_REQUEST['message']
+		$agent     = $_REQUEST['agent'];
-			,$_REQUEST['stack']
+		$stack     = $_REQUEST['stack'];
-		));
+
+		error_log(
+			 "Javascript: $message in $file($line)"
+			."\nLocation: $location"
+			."\nAgent: $agent"
+			."\nUser: $user"
+			."\nStack: $stack"
+		);
 		return TRUE;
 	}
 }

rest/core/login.php

@@ -4,6 +4,9 @@ include __DIR__.'/account.php';
 
 class Login extends Vn\Web\JsonRequest {
 	function run($db) {
+		if (!$_POST['user'])
+			throw new Vn\Web\BadLoginException();
+
 		try {
 			Account::trySync($db
 				,strtolower($_POST['user'])

rest/misc/contact.php

@@ -1,8 +1,9 @@
 <?php
 
-require_once('libphp-phpmailer/PHPMailerAutoload.php');
+require_once('libphp-phpmailer/autoload.php');
 
 use Vn\Lib;
+use PHPMailer\PHPMailer\PHPMailer;
 
 class Contact extends Vn\Web\JsonRequest {
 	const PARAMS = [

rest/misc/mail.php

@@ -1,6 +1,6 @@
 <?php
 
-require_once('libphp-phpmailer/PHPMailerAutoload.php');
+require_once('libphp-phpmailer/autoload.php');
 
 class Mail extends Vn\Lib\Method {
 	function run($db) {

rest/tpv/transaction.php

@@ -7,7 +7,7 @@ class Transaction extends Vn\Web\JsonRequest {
 	const PARAMS = ['amount'];
 
 	function run($db) {
-		$amount =(int) $_REQUEST['amount'];
+		$amount = (int) $_REQUEST['amount'];
 		$companyId = empty($_REQUEST['company']) ? NULL : $_REQUEST['company'];
 			
 		$row = $db->getObject('CALL myTpvTransaction_start(#, #)',

web/mailer.php

@@ -2,9 +2,10 @@
 
 namespace Vn\Web;
 
-require_once 'libphp-phpmailer/PHPMailerAutoload.php';
+require_once 'libphp-phpmailer/autoload.php';
 
 use Vn\Lib\UserException;
+use PHPMailer\PHPMailer\PHPMailer;
 
 class Mailer {
 	private $conf;
@@ -19,7 +20,7 @@ class Mailer {
 	function createObject($mailTo, $body, $subject) {
 		$conf = $this->conf;
 
-		$mail = new \PHPMailer();
+		$mail = new PHPMailer();
 		$mail->isSMTP();
 		$mail->Host = $conf->host;