More fixes for scrutinizer-ci (+ enable php)

Author: RainLoop

.scrutinizer.yml

@@ -11,4 +11,5 @@ filter:
         - 'build/*'
         - 'data/*'
 checks:
-    javascript: true
+    javascript: true
+    php: true

dev/External/ko.js

@@ -135,7 +135,6 @@ ko.bindingHandlers.tooltip = {
 	init: function(oElement, fValueAccessor) {
 
 		var
-			bi18n = true,
 			sValue = '',
 			Translator = null,
 			$oEl = $(oElement),
@@ -145,7 +144,6 @@ ko.bindingHandlers.tooltip = {
 
 		if (!Globals.bMobileDevice || bMobile)
 		{
-			bi18n = 'on' === ($oEl.data('tooltip-i18n') || 'on');
 			sValue = !ko.isObservable(fValue) && _.isFunction(fValue) ? fValue() : ko.unwrap(fValue);
 
 			oElement.__opentip = new Opentip(oElement, {
@@ -171,7 +169,7 @@ ko.bindingHandlers.tooltip = {
 				oElement.__opentip.activate();
 			}
 
-			if (bi18n)
+			if ('on' === ($oEl.data('tooltip-i18n') || 'on'))
 			{
 				Translator = require('Common/Translator');
 
@@ -207,8 +205,9 @@ ko.bindingHandlers.tooltip = {
 			var sValue = !ko.isObservable(fValue) && _.isFunction(fValue) ? fValue() : ko.unwrap(fValue);
 			if (sValue)
 			{
-				var i18n = 'on' === ($oEl.data('tooltip-i18n') || 'on');
-				oElement.__opentip.setContent(i18n ? require('Common/Translator').i18n(sValue) : sValue);
+				oElement.__opentip.setContent('on' === ($oEl.data('tooltip-i18n') || 'on') ?
+					require('Common/Translator').i18n(sValue) : sValue);
+
 				oElement.__opentip.activate();
 			}
 			else

dev/Promises/AbstractAjax.js

@@ -90,7 +90,7 @@ AbstractAjaxPromises.prototype.ajaxRequest = function(sAction, bPost, iTimeOut,
 			}
 
 			// backward capability
-			var sType = Enums.StorageResultType.Error;
+			var sType = '';
 			switch (true)
 			{
 				case 'success' === sTextStatus && oData && oData.Result && sAction === oData.Action:

dev/View/Popup/AddOpenPgpKey.js

@@ -46,13 +46,11 @@ function AddOpenPgpKeyPopupView()
 			return false;
 		}
 
-		var
-			done = false,
-			match = null;
+		var done = false;
 
 		do
 		{
-			match = reg.exec(keyTrimmed);
+			var match = reg.exec(keyTrimmed);
 			if (match && 0 < count)
 			{
 				if (match[0] && match[1] && match[2] && match[1] === match[2])

dev/View/Popup/Compose.js

@@ -1380,13 +1380,9 @@ ComposePopupView.prototype.onBuild = function()
 {
 	this.initUploader();
 
-	var
-		self = this,
-		oScript = null;
+	var self = this;
 
-	key('ctrl+q, command+q, ctrl+w, command+w', Enums.KeyState.Compose, function() {
-		return false;
-	});
+	key('ctrl+q, command+q, ctrl+w, command+w', Enums.KeyState.Compose, Utils.noopFalse);
 
 	key('`', Enums.KeyState.Compose, function() {
 		if (self.oEditor && !self.oEditor.hasFocus() && !Utils.inFocus())
@@ -1437,7 +1433,7 @@ ComposePopupView.prototype.onBuild = function()
 
 	if (this.dropboxEnabled() && this.dropboxApiKey() && !window.Dropbox)
 	{
-		oScript = window.document.createElement('script');
+		var oScript = window.document.createElement('script');
 		oScript.type = 'text/javascript';
 		oScript.src = 'https://www.dropbox.com/static/api/2/dropins.js';
 		$(oScript).attr('id', 'dropboxjs').attr('data-app-key', self.dropboxApiKey());
@@ -1690,19 +1686,16 @@ ComposePopupView.prototype.initUploader = function()
 					this.dragAndDropVisible(false);
 				}, this))
 				.on('onProgress', _.bind(function(sId, iLoaded, iTotal) {
-					var oItem = null;
-					if (Utils.isUnd(oUploadCache[sId]))
+
+					var oItem = oUploadCache[sId];
+					if (!oItem)
 					{
 						oItem = this.getAttachmentById(sId);
 						if (oItem)
 						{
 							oUploadCache[sId] = oItem;
 						}
 					}
-					else
-					{
-						oItem = oUploadCache[sId];
-					}
 
 					if (oItem)
 					{
@@ -1740,21 +1733,15 @@ ComposePopupView.prototype.initUploader = function()
 				}, this))
 				.on('onStart', _.bind(function(sId) {
 
-					var
-						oItem = null;
-
-					if (Utils.isUnd(oUploadCache[sId]))
+					var oItem = oUploadCache[sId];
+					if (!oItem)
 					{
 						oItem = this.getAttachmentById(sId);
 						if (oItem)
 						{
 							oUploadCache[sId] = oItem;
 						}
 					}
-					else
-					{
-						oItem = oUploadCache[sId];
-					}
 
 					if (oItem)
 					{