Merge back Release 0.15.4

.gitignore

@@ -1,6 +1,7 @@
 /node_modules/
 /build/
 .php_cs.cache
+.php-cs-fixer.cache
 /tests/output/
 
 # Composer

CHANGELOG.md

@@ -3,8 +3,32 @@ All notable changes to this project will be documented in this file.
 
 The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
 
-## [Unreleased]
-.
+## [Unreleased] - XXXX-XX-XX
+
+
+## [0.15.4] - 2021-07-13
+
+### Fixed
+- user_ldap 0.15.3 double quote in passwords does not work [#662](https://github.com/owncloud/user_ldap/issues/662)
+- Fix display errors in combination with other apps [#660](https://github.com/owncloud/user_ldap/issues/660)
+- [QA] Frontend breaks with other auser auth apps [#659](https://github.com/owncloud/user_ldap/issues/659)
+- [QA] tab break into new line hides content [#656](https://github.com/owncloud/user_ldap/issues/656)
+- [QA] Login Attributes: LDAP Filter misagligned output [#653](https://github.com/owncloud/user_ldap/issues/653)
+
+## [0.15.3] - 2021-06-14
+
+### Fixed
+- Fix display errors in combination with other apps [#660](https://github.com/owncloud/user_ldap/issues/660)
+- Fixed read LDAP attribute value 0 returned as null [#599](https://github.com/owncloud/user_ldap/issues/599)
+- Security: filter special characters from password field [#636](https://github.com/owncloud/user_ldap/issues/636)
+- LDAP multiple base dns break pagination [#307](https://github.com/owncloud/user_ldap/issues/307)
+
+### Changed
+- Add warning for disabling email login regarding strict login check [#581](https://github.com/owncloud/user_ldap/issues/581) (Requires 10.5.0)
+- Facelift [#597](https://github.com/owncloud/user_ldap/issues/597)
+- Bump libraries
+
+
 
 ## [0.15.2] - 2020-06-16
 
@@ -30,6 +54,7 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
 ### Changed
 
 - Drop PHP 7.0 - [#474](https://github.com/owncloud/user_ldap/issues/474)
+- Requires ownCloud min-version 10.4
 
 ## [0.14.0] - 2019-11-11
 
@@ -128,7 +153,10 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
 - Add hint for max search term length - [#105](https://github.com/owncloud/user_ldap/issues/105)
 - Allow proxy to check next server - [#101](https://github.com/owncloud/user_ldap/issues/101)
 
-[Unreleased]: https://github.com/owncloud/user_ldap/compare/v0.15.2...master
+
+[Unreleased]: https://github.com/owncloud/user_ldap/compare/v0.15.4...master
+[0.15.4]: https://github.com/owncloud/user_ldap/compare/v0.15.3...v0.15.4
+[0.15.3]: https://github.com/owncloud/user_ldap/compare/v0.15.2...v0.15.3
 [0.15.2]: https://github.com/owncloud/user_ldap/compare/v0.15.1...v0.15.2
 [0.15.1]: https://github.com/owncloud/user_ldap/compare/v0.15.0...v0.15.1
 [0.15.0]: https://github.com/owncloud/user_ldap/compare/v0.14.0...v0.15.0

Makefile

@@ -129,13 +129,13 @@ test-php-phpstan: vendor-bin/phpstan/vendor
 .PHONY: test-php-style
 test-php-style: ## Run php-cs-fixer and check owncloud code-style
 test-php-style: vendor-bin/owncloud-codestyle/vendor vendor-bin/php_codesniffer/vendor
-	$(PHP_CS_FIXER) fix -v --diff --diff-format udiff --dry-run --allow-risky yes
+	$(PHP_CS_FIXER) fix -v --diff --dry-run --allow-risky yes
 	$(PHP_CODESNIFFER) --runtime-set ignore_warnings_on_exit --standard=phpcs.xml tests/acceptance
 
 .PHONY: test-php-style-fix
 test-php-style-fix: ## Run php-cs-fixer and fix code-style issues
 test-php-style-fix: vendor-bin/owncloud-codestyle/vendor
-	$(PHP_CS_FIXER) fix -v --diff --diff-format udiff --allow-risky yes
+	$(PHP_CS_FIXER) fix -v --diff --allow-risky yes
 
 .PHONY: test-php-unit
 test-php-unit: ## Run php unit tests

appinfo/info.xml

@@ -14,7 +14,7 @@ More information is available in the [LDAP User and Group Backend documentation]
 	</description>
 	<licence>AGPL</licence>
 	<author>Jörn Friedrich Dreyer, Tom Needham, Juan Pablo Villafañez Ramos, Dominik Schmidt and Arthur Schiwon</author>
-	<version>0.15.2</version>
+	<version>0.15.4</version>
 	<types>
 		<authentication/>
 	</types>

css/settings.css

@@ -1,14 +1,17 @@
-#ldap section {
-	margin-bottom: 2rem;
-	width: 100% !important;
+#ldap {
+	width: calc(100% - 40px);
 }
 
-@media all and (min-width: 768px) {
+@media all and (min-width: 1200px) {
 	#ldap section {
-		width: 60% !important;
+		width: 65% !important;
 	}
 }
 
+#ldap section {
+	margin-bottom: 2rem;
+}
+
 #ldap section h3 {
 	border-bottom: 1px solid #efefef;
 	margin-left: -41px;
@@ -75,6 +78,23 @@
 #ldap .tablerow .hint {
 	clear: both;
 	font-size: .8em;
+	position: relative;
+	padding-left: 1rem;
+}
+
+#ldap .tablerow .hint:before {
+	background: url('/core/img/actions/info.svg');
+    background-size: contain;
+	background-position: 50% 100%;
+	background-repeat: no-repeat;
+	display: inline-block;
+	height: 1.3em;
+	width: 1.3em;
+	margin-right: 0rem;
+	content: ' ';
+	position: absolute;
+	margin-left: -1rem;
+	top: 3px;
 }
 
 #ldap .tablerow .hint img {
@@ -95,7 +115,13 @@
 
 
 .ldapWizardControls {
-	text-align: right;
+	display: flex;
+	flex-direction: row;
+	justify-content: end;
+}
+
+.ldapWizardControls > div {
+	margin-right: 1rem;
 }
 
 .ldapWizardInfo {
@@ -215,14 +241,6 @@ select[multiple=multiple] + button {
 	cursor: wait;
 }
 
-#ldap .ldap_saving {
-	margin-right: 15px;
-	color: orange;
-	font-weight: bold;
-}
-
-#ldap .ldap_saving img { height: 15px; }
-
 #ldap .app-name .ldap_config_state_indicator_container {
 	display: inline-block;
 	margin-left: 2em;
@@ -267,17 +285,17 @@ select[multiple=multiple] + button {
 }
 
 #ldap .header {
-	position: fixed;
 	background: #fff;
-	width: inherit;
 	top: 5.5em;
 	padding-top: 1.7em;
 	z-index: 1;
+	width: calc(100% - 40px);
 }
 
 @media all and (min-width: 768px) {
 	#ldap .header {
 		top: 3.5em;
+		width: calc(750% - 40px);
 	}
 }
 
@@ -312,22 +330,56 @@ select[multiple=multiple] + button {
 	left: 1.3em;
 }
 
-#ldap .ui-tabs .ui-tabs-nav li.stateIndicator {
+#ldap .ui-tabs .ui-tabs-nav li.stateIndicator,
+#ldap .ui-tabs .ui-tabs-nav li.ldap_saving_wrapper {
 	color: #333333;
 	border: none;
 	background: transparent;
-	padding: 0.7em 1.3em;
+	padding: 0.2rem 1.3rem;
 	display: none;
 }
 
+#ldap .ui-tabs .ui-tabs-nav li.ldap_saving_wrapper {
+	padding: 0.5rem 1.3rem;
+	display: block; 
+}
+
+#ldap .ui-tabs .ui-tabs-nav li.ldap_saving_wrapper .ldap_saving {
+	margin-right: 15px;
+	color: orange;
+	font-weight: bold;
+}
+
+#ldap .ui-tabs .ui-tabs-nav li.ldap_saving_wrapper .ldap_saving .done {
+	display: none;
+	color: #128c12;
+}
+
+#ldap .ui-tabs .ui-tabs-nav li.ldap_saving_wrapper .ldap_saving.done img {
+	display: none;
+}
+
+#ldap .ui-tabs .ui-tabs-nav li.ldap_saving_wrapper .ldap_saving.done .done {
+	display: inline;
+}
+
+#ldap .ui-tabs .ui-tabs-nav li.ldap_saving_wrapper .ldap_saving.done .working {
+	display: none;
+}
+
+#ldap .ldap_config_state_indicator_subline {
+	font-weight: 100;
+	font-size: 75%;
+}
+
 @media all and (min-width: 768px) {
 	#ldap .ui-tabs .ui-tabs-nav li.stateIndicator {
 		display: block;
 	}
 }
 
 .ui-tabs .ui-tabs-panel {
-	margin-top: 5.3em;
+	margin-top: 1rem;
 }
 
 #ldap button {
@@ -357,4 +409,4 @@ select[multiple=multiple] + button {
 
 #ldap p {
 	margin-bottom: 1em;
-}
+}

js/wizard/configModel.js

@@ -626,6 +626,9 @@ OCA = OCA || {};
 				configPrefix: configID,
 				errorMessage: _.isUndefined(result['message']) ? '' : result['message']
 			};
+
+			$('.ldap_config_state_indicator_subline').html('')
+
 			model._broadcast('deleteConfiguration', payload);
 		}
 	};

js/wizard/view.js

@@ -55,6 +55,14 @@ OCA = OCA || {};
 			});
 
 			$('.ldap_action_test_connection').click(this.onTestButtonClick);
+
+			$('#ldap_host').on('input', function () {				
+				$('.ldap_config_state_indicator_subline').html($('#ldap_host').val() + ':' + $('#ldap_port').val())
+			})
+
+			$('#ldap_port').on('input', function () {
+				$('.ldap_config_state_indicator_subline').html($('#ldap_host').val() + ':' + $('#ldap_port').val())
+			})
 		},
 
 		/**
@@ -215,6 +223,9 @@ OCA = OCA || {};
 			view._updateStatusIndicator(view.STATUS_UNTESTED);
 			view.basicStatusCheck(view);
 			view.functionalityCheck();
+			if (view['configModel']['configuration']['ldap_host'] !== '')
+				$('.ldap_config_state_indicator_subline').html(view['configModel']['configuration']['ldap_host'] + ':' + view['configModel']['configuration']['ldap_port'])
+
 		},
 
 		/**
@@ -329,16 +340,22 @@ OCA = OCA || {};
 		 * shows a save spinner
 		 */
 		showSaveSpinner: function() {
-			this.$saveSpinners.removeClass('hidden');
+			this.$saveSpinners.show()
 			$('#ldap *').addClass('save-cursor');
 		},
 
 		/**
 		 * hides the save spinner
 		 */
 		hideSaveSpinner: function() {
-			this.$saveSpinners.addClass('hidden');
-			$('#ldap *').removeClass('save-cursor');
+			this.$saveSpinners.addClass('done')
+			setTimeout(function() {
+				$('#ldap .ldap_saving').fadeOut(1000, function () {
+					$('#ldap .ldap_saving').removeClass('done')
+				});
+				$('#ldap *').removeClass('save-cursor');
+
+			}, 1000)			
 		},
 
 		/**
@@ -468,11 +485,4 @@ OCA = OCA || {};
 	};
 
 	OCA.LDAP.Wizard.WizardView = WizardView;
-})();
-
-$(document).ready(function () {
-	$('.ui-tabs .ui-tabs-panel').css('margin-top', $('#ldap .header').outerHeight())
-    $(window).on('resize', function(){
-		$('.ui-tabs .ui-tabs-panel').css('margin-top', $('#ldap .header').outerHeight())
-  	});
-});
+})();