Resolve issue #112

dev/Common/UtilsUser.js

@@ -1,8 +1,10 @@
-import { ComposeType, FolderType } from 'Common/EnumsUser';
+import { ComposeType/*, FolderType*/ } from 'Common/EnumsUser';
 import { EmailModel } from 'Model/Email';
 import { encodeHtml } from 'Common/Html';
 import { isArray } from 'Common/Utils';
 import { createElement } from 'Common/Globals';
+import { FolderUserStore } from 'Stores/User/Folder';
+import { SettingsUserStore } from 'Stores/User/Settings';
 
 /**
  * @param {(string|number)} value
@@ -213,24 +215,20 @@ rl.Utils = {
 
 /**
  * @param {Array} aSystem
- * @param {Array} aList
  * @param {Array=} aDisabled
  * @param {Array=} aHeaderLines
  * @param {Function=} fDisableCallback
  * @param {Function=} fRenameCallback
- * @param {boolean=} bSystem
+ * @param {boolean=} bNoSelectSelectable Used in FolderCreatePopupView
- * @param {boolean=} bBuildUnvisible
  * @returns {Array}
  */
 export function folderListOptionsBuilder(
 	aSystem,
-	aList,
 	aDisabled,
 	aHeaderLines,
-	fDisableCallback,
 	fRenameCallback,
-	bSystem,
+	fDisableCallback,
-	bBuildUnvisible
+	bNoSelectSelectable
 ) {
 	let /**
 		 * @type {?FolderModel}
@@ -238,71 +236,67 @@ export function folderListOptionsBuilder(
 		bSep = false,
 		aResult = [];
 
-	const sDeepPrefix = '\u00A0\u00A0\u00A0';
+	const sDeepPrefix = '\u00A0\u00A0\u00A0',
+		// FolderSystemPopupView should always be true
+		showUnsubscribed = (aSystem.length || bNoSelectSelectable) ? !SettingsUserStore.hideUnsubscribed() : true;
 
-	bBuildUnvisible = undefined === bBuildUnvisible ? false : !!bBuildUnvisible;
+	fDisableCallback = fDisableCallback || (() => false);
-	bSystem = null == bSystem ? 0 < aSystem.length : bSystem;
+	fRenameCallback = fRenameCallback || (oItem => oItem.name());
-	fDisableCallback = null != fDisableCallback ? fDisableCallback : null;
-	fRenameCallback = null != fRenameCallback ? fRenameCallback : null;
 
 	if (!isArray(aDisabled)) {
 		aDisabled = [];
 	}
 
-	if (!isArray(aHeaderLines)) {
+	isArray(aHeaderLines) && aHeaderLines.forEach(line =>
-		aHeaderLines = [];
-	}
-
-	aHeaderLines.forEach(line => {
 		aResult.push({
 			id: line[0],
 			name: line[1],
 			system: false,
 			dividerbar: false,
 			disabled: false
-		});
+		})
-	});
+	);
 
 	bSep = true;
 	aSystem.forEach(oItem => {
 		aResult.push({
 			id: oItem.fullNameRaw,
-			name: fRenameCallback ? fRenameCallback(oItem) : oItem.name(),
+			name: fRenameCallback(oItem),
 			system: true,
 			dividerbar: bSep,
 			disabled:
 				!oItem.selectable ||
 				aDisabled.includes(oItem.fullNameRaw) ||
-				(fDisableCallback ? fDisableCallback(oItem) : false)
+				fDisableCallback(oItem)
 		});
 		bSep = false;
 	});
 
 	bSep = true;
-	aList.forEach(oItem => {
+	FolderUserStore.folderList().forEach(oItem => {
-		// if (oItem.subscribed() || !oItem.exists || bBuildUnvisible)
+/*
-		if (
+		if ((oItem.subscribed() || !oItem.exists || bBuildUnvisible)
-			(oItem.subscribed() || !oItem.exists || bBuildUnvisible) &&
+		 && (oItem.selectable || oItem.hasSubscribedSubfolders())
-			(oItem.selectable || oItem.hasSubscribedSubfolders())
+		 && (FolderType.User === oItem.type() || !bSystem || oItem.hasSubscribedSubfolders()) {
 		) {
-			if (FolderType.User === oItem.type() || !bSystem || oItem.hasSubscribedSubfolders()) {
+*/
-				aResult.push({
+		if (showUnsubscribed || oItem.subscribed() || !oItem.exists || oItem.hasSubscribedSubfolders()) {
-					id: oItem.fullNameRaw,
+			aResult.push({
-					name:
+				id: oItem.fullNameRaw,
-						sDeepPrefix.repeat(oItem.deep + 1) +
+				name:
-						(fRenameCallback ? fRenameCallback(oItem) : oItem.name()),
+					sDeepPrefix.repeat(oItem.deep) +
-					system: false,
+					fRenameCallback(oItem),
-					dividerbar: bSep,
+				system: false,
-					disabled:
+				dividerbar: bSep,
-						!oItem.selectable ||
+				disabled: !bNoSelectSelectable && (
-						aDisabled.includes(oItem.fullNameRaw) ||
+					!oItem.selectable ||
-						(fDisableCallback ? fDisableCallback(oItem) : false)
+					aDisabled.includes(oItem.fullNameRaw) ||
-				});
+					fDisableCallback(oItem))
-				bSep = false;
+			});
-			}
+			bSep = false;
 		}
 
-		if (oItem.subscribed() && oItem.subFolders.length) {
+		if (oItem.subFolders.length) {
 			aResult = aResult.concat(
 				folderListOptionsBuilder(
 					[],
@@ -311,8 +305,7 @@ export function folderListOptionsBuilder(
 					[],
 					fDisableCallback,
 					fRenameCallback,
-					bSystem,
+					bNoSelectSelectable
-					bBuildUnvisible
 				)
 			);
 		}

dev/Model/FolderCollection.js

@@ -257,13 +257,13 @@ export class FolderModel extends AbstractModel {
 							oFolder => (oFolder.subscribed() || oFolder.hasSubscribedSubfolders()) && !oFolder.isSystemFolder()
 						),
 
-				canBeEdited: () => FolderType.User === folder.type() && folder.exists && folder.selectable,
+				canBeEdited: () => FolderType.User === folder.type() && folder.exists/* && folder.selectable*/,
 
 				visible: () => {
 					const isSubscribed = folder.subscribed(),
 						isSubFolders = folder.hasSubscribedSubfolders();
 
-					return isSubscribed || (isSubFolders && (!folder.exists || !folder.selectable));
+					return isSubscribed || (isSubFolders && (!folder.exists/* || !folder.selectable*/));
 				},
 
 				isSystemFolder: () => FolderType.User !== folder.type(),

dev/Settings/User/Folders.js

@@ -118,25 +118,16 @@ export class FoldersUserSettings {
 		}
 	}
 
-	subscribeFolder(folder) {
+	toggleFolderSubscription(folder) {
+		let subscribe = !folder.subscribed();
 		Local.set(ClientSideKeyName.FoldersLashHash, '');
-		Remote.folderSetSubscribe(()=>0, folder.fullNameRaw, true);
+		Remote.folderSetSubscribe(()=>0, folder.fullNameRaw, subscribe);
-		folder.subscribed(true);
+		folder.subscribed(subscribe);
 	}
 
-	unSubscribeFolder(folder) {
+	toggleFolderCheckable(folder) {
-		Local.set(ClientSideKeyName.FoldersLashHash, '');
+		let checkable = !folder.checkable();
-		Remote.folderSetSubscribe(()=>0, folder.fullNameRaw, false);
+		Remote.folderSetCheckable(()=>0, folder.fullNameRaw, checkable);
-		folder.subscribed(false);
+		folder.checkable(checkable);
-	}
-
-	checkableTrueFolder(folder) {
-		Remote.folderSetCheckable(()=>0, folder.fullNameRaw, true);
-		folder.checkable(true);
-	}
-
-	checkableFalseFolder(folder) {
-		Remote.folderSetCheckable(()=>0, folder.fullNameRaw, false);
-		folder.checkable(false);
 	}
 }

dev/Stores/User/Folder.js

@@ -3,7 +3,6 @@ import ko from 'ko';
 import { FolderType, FolderSortMode } from 'Common/EnumsUser';
 import { UNUSED_OPTION_VALUE } from 'Common/Consts';
 import { addObservablesTo, addSubscribablesTo } from 'Common/Utils';
-import { folderListOptionsBuilder } from 'Common/UtilsUser';
 import { getFolderInboxName, getFolderFromCacheList } from 'Common/Cache';
 import { SettingsGet } from 'Common/Globals';
 
@@ -107,28 +106,6 @@ export const FolderUserStore = new class {
 			this.folderListSystemNames().map(name => getFolderFromCacheList(name)).filter(v => v)
 		);
 
-		this.folderMenuForMove = ko.computed(() =>
-			folderListOptionsBuilder(
-				this.folderListSystem(),
-				this.folderList(),
-				[this.currentFolderFullNameRaw()],
-				[],
-				null,
-				(item) => (item ? item.localName() : '')
-			)
-		);
-
-		this.folderMenuForFilters = ko.computed(() =>
-			folderListOptionsBuilder(
-				this.folderListSystem(),
-				this.folderList(),
-				[this.sieveAllowFileintoInbox ? '' : 'INBOX'],
-				[['', '']],
-				null,
-				(item) => (item ? item.localName() : '')
-			)
-		);
-
 		const
 			fRemoveSystemFolderType = (observable) => () => {
 				const folder = getFolderFromCacheList(observable());

dev/Styles/User/FolderList.less

@@ -111,6 +111,9 @@
 					color: var(--folders-selected-color, #eee);
 				}
 			}
+			&:not(.selectable) {
+				font-style: italic;
+			}
 
 			&.focused {
 				background-color: #888;

dev/Styles/User/SettingsFolders.less

@@ -34,6 +34,10 @@
 
 	.folder-item {
 
+		&:not(.selectable) {
+			font-style: italic;
+		}
+
 		&.system .folder-name {
 			font-weight: bold;
 		}

dev/View/Popup/Filter.js

@@ -11,6 +11,8 @@ import { SieveUserStore } from 'Stores/User/Sieve';
 import { decorateKoCommands } from 'Knoin/Knoin';
 import { AbstractViewPopup } from 'Knoin/AbstractViews';
 
+import { folderListOptionsBuilder } from 'Common/UtilsUser';
+
 class FilterPopupView extends AbstractViewPopup {
 	constructor() {
 		super('Filter');
@@ -25,7 +27,15 @@ class FilterPopupView extends AbstractViewPopup {
 		this.fTrueCallback = null;
 
 		this.defaultOptionsAfterRender = defaultOptionsAfterRender;
-		this.folderSelectList = FolderUserStore.folderMenuForFilters;
+		this.folderSelectList = ko.computed(() =>
+			folderListOptionsBuilder(
+				FolderUserStore.folderListSystem(),
+				[FolderUserStore.sieveAllowFileintoInbox ? '' : 'INBOX'],
+				[['', '']],
+				item => item ? item.localName() : ''
+			)
+		);
+
 
 		this.selectedFolderValue.subscribe(() => this.filter() && this.filter().actionValueError(false));
 

dev/View/Popup/FolderCreate.js

@@ -25,14 +25,14 @@ class FolderCreatePopupView extends AbstractViewPopup {
 		this.parentFolderSelectList = ko.computed(() =>
 			folderListOptionsBuilder(
 				[],
-				FolderUserStore.folderList(),
 				[],
 				[['', '']],
+				oItem =>
+					oItem ? (oItem.isSystemFolder() ? oItem.name() + ' ' + oItem.manageFolderSystemName() : oItem.name()) : '',
 				FolderUserStore.namespace
 					? item => FolderUserStore.namespace !== item.fullNameRaw.substr(0, FolderUserStore.namespace.length)
 					: null,
-				oItem =>
+				true
-					oItem ? (oItem.isSystemFolder() ? oItem.name() + ' ' + oItem.manageFolderSystemName() : oItem.name()) : ''
 			)
 		);
 

dev/View/Popup/FolderSystem.js

@@ -30,16 +30,11 @@ class FolderSystemPopupView extends AbstractViewPopup {
 		this.folderSelectList = ko.computed(() =>
 			folderListOptionsBuilder(
 				[],
-				FolderUserStore.folderList(),
 				FolderUserStore.folderListSystemNames(),
 				[
 					['', this.sChooseOnText],
 					[UNUSED_OPTION_VALUE, this.sUnuseText]
-				],
+				]
-				null,
-				null,
-				null,
-				true
 			)
 		);
 

dev/View/User/MailBox/MessageList.js

@@ -15,7 +15,7 @@ import { UNUSED_OPTION_VALUE } from 'Common/Consts';
 
 import { doc, leftPanelDisabled, moveAction, Settings, SettingsGet } from 'Common/Globals';
 
-import { computedPaginatorHelper, showMessageComposer } from 'Common/UtilsUser';
+import { computedPaginatorHelper, showMessageComposer, folderListOptionsBuilder } from 'Common/UtilsUser';
 import { FileInfo } from 'Common/File';
 
 import { mailBox, serverRequest } from 'Common/Links';
@@ -78,7 +78,14 @@ export class MessageListMailBoxUserView extends AbstractViewRight {
 		this.isMessageSelected = MessageUserStore.isMessageSelected;
 		this.messageListSearch = MessageUserStore.listSearch;
 		this.messageListError = MessageUserStore.listError;
-		this.folderMenuForMove = FolderUserStore.folderMenuForMove;
+		this.folderMenuForMove = ko.computed(() =>
+			folderListOptionsBuilder(
+				FolderUserStore.folderListSystem(),
+				[FolderUserStore.currentFolderFullNameRaw()],
+				[],
+				item => item ? item.localName() : ''
+			)
+		);
 
 		this.useCheckboxesInList = SettingsUserStore.useCheckboxesInList;
 

snappymail/v/0.0.0/app/libraries/MailSo/Mail/Folder.php

@@ -233,7 +233,7 @@ class Folder implements \JsonSerializable
 			'FullName' => $this->FullName(),
 			'FullNameRaw' => $this->FullNameRaw(),
 			'Delimiter' => (string) $this->Delimiter(),
-			'HasVisibleSubFolders' => $this->HasVisibleSubFolders(),
+//			'HasVisibleSubFolders' => $this->HasVisibleSubFolders(),
 			'Subscribed' => $this->bSubscribed,
 			'Exists' => $this->bExists,
 			'Selectable' => $this->IsSelectable(),

snappymail/v/0.0.0/app/templates/Views/User/SettingsFolderItem.html

@@ -14,14 +14,10 @@
 		<span class="delete-folder e-action fontastic" data-bind="visible: canBeDeleted() && !deleteAccess(), click: function (oFolder) { $root.folderForDeletion(oFolder); }">🗑</span>
 	</td>
 	<td data-i18n="[title]SETTINGS_FOLDERS/HELP_SHOW_HIDE_FOLDER">
-		<span class="unsubscribed-folder e-action fontastic" data-bind="visible: canBeSubscribed() && !subscribed(), click: function(oFolder) { $root.subscribeFolder(oFolder); }">👁</span>
+		<span class="e-action fontastic" data-bind="css: {'unsubscribed-folder':!subscribed()}, visible: canBeSubscribed(), click: function(oFolder) { $root.toggleFolderSubscription(oFolder); }">👁</span>
-		<span class="e-action fontastic" data-bind="visible: canBeSubscribed() && subscribed(), click: function(oFolder) { $root.unSubscribeFolder(oFolder); }">👁</span>
 	</td>
 	<td data-bind="visible: $root.displaySpecSetting" data-i18n="[title]SETTINGS_FOLDERS/HELP_CHECK_FOR_NEW_MESSAGES">
-		<span class="unchecked-folder e-action" data-bind="visible: canBeSelected() && subscribed() && !checkable(), click: function(oFolder) { $root.checkableTrueFolder(oFolder); }">
+		<span class="e-action" data-bind="css: {'check-folder':checkable(), 'unchecked-folder':!checkable()}, visible: canBeSelected() && subscribed(), click: function(oFolder) { $root.toggleFolderCheckable(oFolder); }">
-			<i class="icon-check-mark-circle-two"></i>
-		</span>
-		<span class="check-folder e-action" data-bind="visible: canBeSelected() && subscribed() && checkable(), click: function(oFolder) { $root.checkableFalseFolder(oFolder); }">
 			<i class="icon-check-mark-circle-two"></i>
 		</span>
 	</td>