Merge pull request #35253 from nextcloud/bugfix/noid/remove-warnings-loginpage

core/src/components/MainMenu.js

@@ -28,6 +28,7 @@ import Vue from 'vue'
 import AppMenu from './AppMenu.vue'
 
 export const setUp = () => {
+
 	Vue.mixin({
 		methods: {
 			t,
@@ -35,8 +36,13 @@ export const setUp = () => {
 		},
 	})
 
+	const container = document.getElementById('header-left__appmenu')
+	if (!container) {
+		// no container, possibly we're on a public page
+		return
+	}
 	const AppMenuApp = Vue.extend(AppMenu)
-	const appMenu = new AppMenuApp({}).$mount('#header-left__appmenu')
+	const appMenu = new AppMenuApp({}).$mount(container)
 
 	Object.assign(OC, {
 		setNavigationCounter(id, counter) {

core/src/components/login/LoginForm.vue

@@ -168,7 +168,7 @@ export default {
 	computed: {
 		isError() {
 			return this.invalidPassword || this.userDisabled
-				|| (this.throttleDelay && this.throttleDelay > 5000)
+				|| this.throttleDelay > 5000
 		},
 		errorLabel() {
 			if (this.invalidPassword) {
@@ -177,7 +177,7 @@ export default {
 			if (this.userDisabled) {
 				return t('core', 'User disabled')
 			}
-			if (this.throttleDelay && this.throttleDelay > 5000) {
+			if (this.throttleDelay > 5000) {
 				return t('core', 'We have detected multiple invalid login attempts from your IP. Therefore your next login is throttled up to 30 seconds.')
 			}
 			return undefined