disable edition if no multi profiles

.github/workflows/github-actions-main.yml

@@ -6,7 +6,7 @@ jobs:
     strategy:
       matrix:
         # Removed windows-2016 for now - discontinued by GitHub
-        os: [macos-latest, ubuntu-latest, windows-2019]
+        os: [macos-latest, ubuntu-latest]
     steps:
 
       # Silence apt-get update errors (for example when a module doesn't

packages/app-desktop/gui/NoteListItem.tsx

@@ -110,7 +110,7 @@ function NoteListItem(props: NoteListItemProps, ref: any) {
 
 	let listItemTitleStyle = Object.assign({}, props.style.listItemTitle);
 	listItemTitleStyle.paddingLeft = !item.is_todo ? hPadding : 4;
-	if (item.is_shared) listItemTitleStyle.color = theme.colorWarn3;
+	if (item.is_shared) listItemTitleStyle.color = theme.colorWarn2;
 	if (item.is_todo && !!item.todo_completed) listItemTitleStyle = Object.assign(listItemTitleStyle, props.style.listItemTitleCompleted);
 
 	const displayTitle = Note.displayTitle(item);

packages/app-desktop/services/sortOrder/notesSortOrderUtils.test.ts

@@ -1,6 +1,5 @@
 import { notesSortOrderFieldArray, notesSortOrderNextField, setNotesSortOrder } from './notesSortOrderUtils';
 import Setting from '@joplin/lib/models/Setting';
-// import { setupDatabaseAndSynchronizer, switchClient } from '@joplin/lib/testing/test-utils';
 const { shimInit } = require('@joplin/lib/shim-init-node.js');
 
 beforeAll(() => {
@@ -10,12 +9,6 @@ beforeAll(() => {
 
 describe('notesSortOrderUtils', () => {
 
-	// beforeEach(async (done) => {
-	// 	await setupDatabaseAndSynchronizer(1);
-	// 	await switchClient(1);
-	// 	done();
-	// });
-
 	it('should always provide the same ordered fields', async () => {
 		const expected = ['user_updated_time', 'user_created_time', 'title', 'order'];
 		expect(notesSortOrderFieldArray()).toStrictEqual(expected);

packages/lib/models/Setting.test.ts

@@ -23,11 +23,6 @@ const switchToSubProfileSettings = async () => {
 	await Setting.load();
 };
 
-const restoreSettingState = async () => {
-	await setupDatabaseAndSynchronizer(1);
-	await switchClient(1);
-};
-
 describe('models/Setting', function() {
 
 	beforeEach(async (done) => {
@@ -296,8 +291,6 @@ describe('models/Setting', function() {
 		expect((await Setting.loadOne('locale')).value).toBe('fr_FR');
 		expect((await Setting.loadOne('theme')).value).toBe(Setting.THEME_DARK);
 		expect((await Setting.loadOne('sync.target')).value).toBe(undefined);
-
-		await restoreSettingState();
 	});
 
 	it('should save sub-profile settings', async () => {
@@ -334,8 +327,6 @@ describe('models/Setting', function() {
 			'$schema': 'https://joplinapp.org/schema/settings.json',
 			'sync.target': 8,
 		});
-
-		await restoreSettingState();
 	});
 
 	it('all global settings should be saved to file', async () => {

packages/lib/models/Setting.ts

@@ -1516,7 +1516,7 @@ class Setting extends BaseModel {
 
 		this.metadata_ = Object.assign(this.metadata_, this.customMetadata_);
 
-		if (this.constants_.env === Env.Dev) this.validateMetadata(this.metadata_);
+		if (this.value('env') === Env.Dev) this.validateMetadata(this.metadata_);
 
 		return this.metadata_;
 	}

packages/lib/services/interop/InteropService_Importer_Md.ts

@@ -46,6 +46,7 @@ export default class InteropService_Importer_Md extends InteropService_Importer_
 	}
 
 	async importDirectory(dirPath: string, parentFolderId: string) {
+		console.info(`Import: ${dirPath}`);
 		const supportedFileExtension = this.metadata().fileExtensions;
 		const stats = await shim.fsDriver().readDirStats(dirPath);
 		for (let i = 0; i < stats.length; i++) {
@@ -53,6 +54,7 @@ export default class InteropService_Importer_Md extends InteropService_Importer_
 
 			if (stat.isDirectory()) {
 				if (await this.isDirectoryEmpty(`${dirPath}/${stat.path}`)) {
+					console.info(`Ignoring empty directory: ${stat.path}`);
 					continue;
 				}
 				const folderTitle = await Folder.findUniqueItemTitle(basename(stat.path));

packages/lib/services/rest/Api.test.ts

@@ -379,28 +379,6 @@ describe('services_rest_Api', function() {
 		expect(resourceV2.size).toBe((await shim.fsDriver().stat(Resource.fullPath(resourceV2))).size);
 	}));
 
-	it('should allow updating a resource file only', (async () => {
-		await api.route(RequestMethod.POST, 'resources', null, JSON.stringify({
-			title: 'resource',
-		}), [{ path: `${supportDir}/photo.jpg` }]);
-
-		const resourceV1: ResourceEntity = (await Resource.all())[0];
-
-		await msleep(1);
-
-		await api.route(RequestMethod.PUT, `resources/${resourceV1.id}`, null, null, [
-			{
-				path: `${supportDir}/photo-large.png`,
-			},
-		]);
-
-		const resourceV2: ResourceEntity = (await Resource.all())[0];
-
-		// It should have updated the file content, but not the metadata
-		expect(resourceV2.title).toBe(resourceV1.title);
-		expect(resourceV2.size).toBeGreaterThan(resourceV1.size);
-	}));
-
 	it('should update resource properties', (async () => {
 		await api.route(RequestMethod.POST, 'resources', null, JSON.stringify({
 			title: 'resource',

packages/lib/services/style/themeToCss.test.ts

@@ -26,7 +26,6 @@ const input: Theme = {
 	selectedColor2: '#131313',
 	colorError2: '#ff6c6c',
 	colorWarn2: '#ffcb81',
-	colorWarn3: '#ff7626',
 
 	// Color scheme "3" is used for the config screens for example/
 	// It's dark text over gray background.
@@ -77,7 +76,6 @@ const expected = `
 	--joplin-selected-color2: #131313;
 	--joplin-color-error2: #ff6c6c;
 	--joplin-color-warn2: #ffcb81;
-	--joplin-color-warn3: #ff7626;
 	--joplin-background-color3: #F4F5F6;
 	--joplin-background-color-hover3: #CBDAF1;
 	--joplin-color3: #738598;

packages/lib/shim-init-node.js

@@ -235,8 +235,6 @@ function shimInit(options = null) {
 		const readChunk = require('read-chunk');
 		const imageType = require('image-type');
 
-		const isUpdate = !!options.destinationResourceId;
-
 		const uuid = require('./uuid').default;
 
 		if (!(await fs.pathExists(filePath))) throw new Error(_('Cannot access %s', filePath));
@@ -244,16 +242,12 @@ function shimInit(options = null) {
 		defaultProps = defaultProps ? defaultProps : {};
 
 		let resourceId = defaultProps.id ? defaultProps.id : uuid.create();
-		if (isUpdate) resourceId = options.destinationResourceId;
+		if (options.destinationResourceId) resourceId = options.destinationResourceId;
 
-		let resource = isUpdate ? {} : Resource.new();
+		let resource = options.destinationResourceId ? {} : Resource.new();
 		resource.id = resourceId;
-
-		// When this is an update we auto-update the mime type, in case the
-		// content type has changed, but we keep the title. It is still possible
-		// to modify the title on update using defaultProps.
 		resource.mime = mimeUtils.fromFilename(filePath);
-		if (!isUpdate) resource.title = basename(filePath);
+		resource.title = basename(filePath);
 
 		let fileExt = safeFileExtension(fileExtension(filePath));
 
@@ -294,7 +288,7 @@ function shimInit(options = null) {
 		const saveOptions = { isNew: true };
 		if (options.userSideValidation) saveOptions.userSideValidation = true;
 
-		if (isUpdate) {
+		if (options.destinationResourceId) {
 			saveOptions.isNew = false;
 			const tempPath = `${targetPath}.tmp`;
 			await shim.fsDriver().move(targetPath, tempPath);

packages/lib/themes/dark.ts

@@ -29,7 +29,6 @@ const theme: Theme = {
 	selectedColor2: '#013F74',
 	colorError2: '#ff6c6c',
 	colorWarn2: '#ffcb81',
-	colorWarn3: '#ffcb81',
 
 	// Color scheme "3" is used for the config screens for example/
 	// It's dark text over gray background.

packages/lib/themes/light.ts

@@ -26,7 +26,6 @@ const theme: Theme = {
 	selectedColor2: '#131313',
 	colorError2: '#ff6c6c',
 	colorWarn2: '#ffcb81',
-	colorWarn3: '#ff7626',
 
 	// Color scheme "3" is used for the config screens for example/
 	// It's dark text over gray background.

packages/lib/themes/type.ts

@@ -27,8 +27,7 @@ export interface Theme {
 	color2: string;
 	selectedColor2: string;
 	colorError2: string;
-	colorWarn2: string; // On a darker background (eg. sidebar)
-	colorWarn3: string; // On a lighter background (eg. note list)
+	colorWarn2: string;
 
 	// Color scheme "3" is used for the config screens for example/
 	// It's dark text over gray background.