Merge pull request #2224 from nextcloud/fix-visibility-switcher

Fix visibility switcher for avatar

settings/js/federationscopemenu.js

@@ -113,7 +113,7 @@
 			this._context = context;
 			var currentlyActiveValue = $('#'+context.target.closest('form').id).find('.icon-checkmark > input')[0].value;
 
-			for(var i = 0 in this._scopes) {
+			for(var i in this._scopes) {
 				this._scopes[i].active = false;
 			}
 

settings/js/federationsettingsview.js

@@ -137,7 +137,7 @@
 
 			this._config.set(field + 'Scope', scope);
 
-			$('#' + field).parent().find('span > input').val(scope);
+			$('#' + field + 'scope').val(scope);
 
 			// TODO: user loading/success feedback
 			this._config.save();

settings/templates/personal.php

@@ -60,6 +60,7 @@
 				<div class="inlineblock button primary" id="sendcropperbutton"><?php p($l->t('Choose as profile picture')); ?></div>
 			</div>
 		</div>
+		<span class="icon-checkmark hidden"/>
 		<input type="hidden" id="avatarscope" value="<?php p($_['avatarScope']) ?>">
 	</form>
 </div>